yahoo-eng-team team mailing list archive

[Thierry Carrez <thierry.carrez+lp@xxxxxxxxx>]

** Changed in: neutron
       Status: Fix Committed => 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/1446021

Title:
  Remove get_admin_roles in neutron/policy.py

Status in OpenStack Neutron (virtual network service):
  Fix Released

Bug description:
  There is a comment in that method that clearly states its temporary
  nature.

  
      # NOTE(salvatore-orlando): This function provides a solution for
      # populating implicit contexts with the appropriate roles so that
      # they correctly pass policy checks, and will become superseded
      # once all explicit policy checks are removed from db logic and
      # plugin modules. For backward compatibility it returns the literal
      # admin if ADMIN_CTX_POLICY is not defined

  link:
  http://git.openstack.org/cgit/openstack/neutron/tree/neutron/policy.py#n473

  It is probably time function and all associated logic is removed - as
  it constitutes a blocker for adoption of oslo_policy

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