Send list of commands returned by hypervisor Guru instead of explicitly selecting the first one

Dingane Hlaluku · Dingane Hlaluku · commit df30c81f31bd · 2019-01-16T11:11:36.000+02:00
diff --git a/engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java b/engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java
@@ -2023,7 +2023,8 @@ private Answer[] attemptHypervisorMigration(StoragePool destPool, VMInstanceVO v
         }
 
         if(CollectionUtils.isNotEmpty(commandsToSend)) {
-            Commands commandsContainer = new Commands(commandsToSend.get(0));
+            Commands commandsContainer = new Commands(Command.OnError.Stop);
+            commandsContainer.addCommands(commandsToSend);
             try {
                 // OfflineVmwareMigration: change to the call back variety?
                 // OfflineVmwareMigration: getting a Long seq to be filled with _agentMgr.send(hostId, commandsContainer, this)

Original file line number	Diff line number	Diff line change
`@@ -2023,7 +2023,8 @@ private Answer[] attemptHypervisorMigration(StoragePool destPool, VMInstanceVO v`
`2023`	`2023`	`}`
`2024`	`2024`
`2025`	`2025`	`if(CollectionUtils.isNotEmpty(commandsToSend)) {`
`2026`		`- Commands commandsContainer = new Commands(commandsToSend.get(0));`
	`2026`	`+ Commands commandsContainer = new Commands(Command.OnError.Stop);`
	`2027`	`+ commandsContainer.addCommands(commandsToSend);`
`2027`	`2028`	`try {`
`2028`	`2029`	`// OfflineVmwareMigration: change to the call back variety?`
`2029`	`2030`	`// OfflineVmwareMigration: getting a Long seq to be filled with _agentMgr.send(hostId, commandsContainer, this)`