[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #6947: Allow password definition during VM deploy

[via GitHub]

DaanHoogland commented on code in PR #6947:
URL: https://github.com/apache/cloudstack/pull/6947#discussion_r1095446274


##########
server/src/main/java/com/cloud/vm/UserVmManagerImpl.java:
##########
@@ -1158,9 +1159,8 @@ private UserVm rebootVirtualMachine(long userId, long 
vmId, boolean enterSetup,
     private UserVm forceRebootVirtualMachine(long vmId, long hostId, boolean 
enterSetup) {
         try {
             if (stopVirtualMachine(vmId, false) != null) {
-                Map<VirtualMachineProfile.Param,Object> params = null;
+                Map<VirtualMachineProfile.Param,Object> params = new HashMap();

Review Comment:
   ```suggestion
                   Map<VirtualMachineProfile.Param,Object> params = new 
HashMap<>();
   ```



-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

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