[GitHub] sateesh-chodapuneedi commented on a change in pull request #1664: CLOUDSTACK-8676 Deploy user instance from vm snapshot for VMware hypervisor

[gi...@git.apache.org]

sateesh-chodapuneedi commented on a change in pull request #1664: CLOUDSTACK-8676 Deploy user instance from vm snapshot for VMware hypervisor
URL: https://github.com/apache/cloudstack/pull/1664#discussion_r113517524
 
 

 ##########
 File path: server/src/com/cloud/vm/UserVmManagerImpl.java
 ##########
 @@ -3242,8 +3283,9 @@ protected UserVm createVirtualMachine(DataCenter zone, ServiceOffering serviceOf
             throw new InvalidParameterValueException("Installing from ISO requires an ISO that is bootable: " + template.getId());
         }
 
-        // Check templates permissions
-        _accountMgr.checkAccess(owner, AccessType.UseEntry, false, template);
+        // Check templates permissions when the create vm is not from snapshot
 
 Review comment:
   @yvsubhash Here is the code snippet picked up from PR https://github.com/apache/cloudstack/pull/1721
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services