yahoo-eng-team team mailing list archive

Thread
Date

[Bug 1177413] Re: evacuate should not hard code admin authorisation check

To: yahoo-eng-team@xxxxxxxxxxxxxxxxxxx
From: Thierry Carrez <thierry.carrez+lp@xxxxxxxxx>
Date: Wed, 29 May 2013 10:28:25 -0000
Reply-to: Bug 1177413 <1177413@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

** Changed in: nova
       Status: Fix Committed => Fix Released

** Changed in: nova
    Milestone: None => havana-1

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1177413

Title:
  evacuate should not hard code admin authorisation check

Status in OpenStack Compute (Nova):
  Fix Released

Bug description:
  From code review, it looks like evacuate is hard coded to check for an admin context:
  https://review.openstack.org/#/c/25991/13/nova/api/openstack/compute/contrib/evacuate.py

  There are policy checks in place, but looks like there are extra
  checks that are not really needed.

  Seems like better unit tests are also needed, just to double check the
  policy checks work correctly.

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