ravening commented on pull request #4200:
URL: https://github.com/apache/cloudstack/pull/4200#issuecomment-726894671


   > @ravening I am absolutely -1 on allowing admins to screw-up by not 
requiring the domain id, there is no need. Requiring it does not stop admins 
from doing anything
   > 
   > isrecursive might be nice though.
   
   @PaulAngus think about the way how virtual machine is deployed by domain 
admin. They dont pass domainid parameter while creating the VM. So according to 
you, we should disable creating vm without passing domainid as well right?
   
   
   ```
   (test1) 🐱 > list accounts filter=rolename
   {
     "account": [
       {
         "rolename": "Domain Admin"
       }
     ],
     "count": 1
   }
   (test1) 🐱 >
   (test1) 🐱 > deploy virtualmachine
   💩 Missing required parameters:  serviceofferingid, templateid, zoneid
   (test1) 🐱 >
   ```


----------------------------------------------------------------
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:
[email protected]


Reply via email to