rafaelweingartner commented on pull request #4257:
URL: https://github.com/apache/cloudstack/pull/4257#issuecomment-696452264


   > the check is removed for everybody and not only for admin. Is this 
intentional? it does not seem right
   
   Can normal users execute the migrate volume command? As far as I remember 
they should not be able to do so. Of course, with the dynamic roles, we removed 
the restriction on hard-coded roles, and the restrictions are executed on the 
fly according to the allowed/denied APIs configured for a given role. 
Therefore, if the user has access to the API, it should be able to execute it 
and migrate the volume as he/she wishes.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to