Fix tests

ionos-cloud · Oct 9, 2024 · f618472 · f618472
1 parent bee82b0
commit f618472
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 1 deletion.
diff --git a/internal/service/vmservice/vm.go b/internal/service/vmservice/vm.go
@@ -120,7 +120,7 @@ func checkCloudInitStatus(ctx context.Context, machineScope *scope.MachineScope)
 		}
 	}
 
-	if !machineScope.SkipQemuGuestCheck() && !machineScope.SkipCloudInitCheck() {
+	if !machineScope.SkipCloudInitCheck() {
 		if running, err := machineScope.InfraCluster.ProxmoxClient.CloudInitStatus(ctx, machineScope.VirtualMachine); err != nil || running {
 			if running {
 				return true, nil

diff --git a/pkg/scope/machine.go b/pkg/scope/machine.go
@@ -255,6 +255,10 @@ func (m *MachineScope) SkipQemuGuestCheck() bool {
 
 // SkipCloudInitCheck check whether cloud-init status check is enabled.
 func (m *MachineScope) SkipCloudInitCheck() bool {
+	if m.SkipQemuGuestCheck() {
+		return false
+	}
+
 	if m.ProxmoxMachine.Spec.Checks != nil {
 		return ptr.Deref(m.ProxmoxMachine.Spec.Checks.SkipCloudInitStatus, false)
 	}

diff --git a/pkg/scope/machine_test.go b/pkg/scope/machine_test.go
@@ -174,6 +174,21 @@ func TestMachineScope_SkipCloudInit(t *testing.T) {
 	require.False(t, scope.SkipQemuGuestCheck())
 }
 
+func TestMachineScope_SkipQemuDisablesCloudInitCheck(t *testing.T) {
+	p := infrav1alpha1.ProxmoxMachine{
+		Spec: infrav1alpha1.ProxmoxMachineSpec{
+			Checks: &infrav1alpha1.ProxmoxMachineChecks{
+				SkipQemuGuestAgent: ptr.To(true),
+			},
+		},
+	}
+	scope := MachineScope{
+		ProxmoxMachine: &p,
+	}
+
+	require.False(t, scope.SkipCloudInitCheck())
+}
+
 func TestMachineScope_GetBootstrapSecret(t *testing.T) {
 	client := fake.NewClientBuilder().Build()
 	p := infrav1alpha1.ProxmoxMachine{