yahoo-eng-team team mailing list archive

[OpenStack Infra <2016144@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/neutron/+/880338
Committed: https://opendev.org/openstack/neutron/commit/c7ef8249419ecbefdc4a604a98d4b23554c78dae
Submitter: "Zuul (22348)"
Branch:    master

commit c7ef8249419ecbefdc4a604a98d4b23554c78dae
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Thu Apr 13 17:54:47 2023 +0200

    Do not check the context object in ``TestMeteringPlugin``
    
    Since [1], the admin context is not ``Context`` object created
    but the new one created in ``ContextBase.elevated``. This method
    copies the object itself (creating a new one), adds the "admin"
    role (if not present) and returns this new object.
    
    Because this method always returns a new object, it is not possible
    to check the ``Context`` object used to make the function call, as
    long as the test doesn't store the elevated context used in the call
    (apart from the reference stored in the mock object).
    
    [1]https://review.opendev.org/c/openstack/neutron-lib/+/880143
    
    Closes-Bug: #2016144
    Change-Id: I5d7eb73606428841caa24ce1e38d1bebd5db0a9b


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/2016144

Title:
  UTs "TestMeteringPlugin" failing with new neutron-lib 3.5.0 release

Status in neutron:
  Fix Released

Bug description:
  neutron-lib 3.5.0 released yesterday.

  Patch affected:
  https://review.opendev.org/c/openstack/requirements/+/880278

  Logs:
  https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_1c2/880278/1/check/cross-
  neutron-py310/1c2bbc3/testr_results.html

  Snippet: https://paste.opendev.org/show/bwRYjJB54O20iK4b8n1d/

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/2016144/+subscriptions