yahoo-eng-team team mailing list archive

[Keigo Noha <1963652@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

In current Horizon, ToggleSuspend doesn't check an appropriate policy.

The current implementation is:
~~~
    def allowed(self, request, instance=None):
        if not instance:
            return False
        self.suspended = instance.status == "SUSPENDED"
        if self.suspended:
            self.current_present_action = RESUME
            policy_rules = (
                ("compute", "os_compute_api:os-rescue"),)
~~~

However, the action uses server_resume() to resume an instance.
To align the policy to the actual command, it should be os_compute_api:os-suspend-server:resume

** Affects: horizon
     Importance: Undecided
         Status: New

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

Title:
  ToggleSuspend doesn't check an appropriate policy

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  In current Horizon, ToggleSuspend doesn't check an appropriate policy.

  The current implementation is:
  ~~~
      def allowed(self, request, instance=None):
          if not instance:
              return False
          self.suspended = instance.status == "SUSPENDED"
          if self.suspended:
              self.current_present_action = RESUME
              policy_rules = (
                  ("compute", "os_compute_api:os-rescue"),)
  ~~~

  However, the action uses server_resume() to resume an instance.
  To align the policy to the actual command, it should be os_compute_api:os-suspend-server:resume

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