Reverse search order for ide disks for cloud-init allocation. (#56)

jeff-voight · web-flow · commit e68ae2ff7125 · 2021-12-21T15:56:47.000+01:00
* Reverse search order for ide disks for cloud-init allocation.

* Update finalize_template_config_test to look for ide0 for cloud_init.
diff --git a/builder/proxmox/common/step_finalize_template_config.go b/builder/proxmox/common/step_finalize_template_config.go
@@ -55,7 +55,7 @@ func (s *stepFinalizeTemplateConfig) Run(ctx context.Context, state multistep.St
 			}
 		}
 		if cloudInitStoragePool != "" {
-			ideControllers := []string{"ide3", "ide2", "ide1", "ide0"}
+			ideControllers := []string{"ide0", "ide1", "ide2", "ide3"}
 			cloudInitAttached := false
 			// find a free ide controller
 			for _, controller := range ideControllers {
diff --git a/builder/proxmox/common/step_finalize_template_config_test.go b/builder/proxmox/common/step_finalize_template_config_test.go
@@ -85,7 +85,7 @@ func TestTemplateFinalize(t *testing.T) {
 			expectedVMConfig: map[string]interface{}{
 				"name":        "my-template",
 				"description": "some-description",
-				"ide3":        "ceph01:cloudinit",
+				"ide0":        "ceph01:cloudinit",
 			},
 			expectedAction: multistep.ActionContinue,
 		},

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ func (s *stepFinalizeTemplateConfig) Run(ctx context.Context, state multistep.St`
`55`	`55`	`}`
`56`	`56`	`}`
`57`	`57`	`if cloudInitStoragePool != "" {`
`58`		`- ideControllers := []string{"ide3", "ide2", "ide1", "ide0"}`
	`58`	`+ ideControllers := []string{"ide0", "ide1", "ide2", "ide3"}`
`59`	`59`	`cloudInitAttached := false`
`60`	`60`	`// find a free ide controller`
`61`	`61`	`for _, controller := range ideControllers {`