Merge pull request #4393 from MartinForReal/shafan/virtualnode

InstanceExists should return true when node is unmanaged ones

pkg/provider/azure_instances.go

@@ -231,7 +231,7 @@ func (az *Cloud) InstanceExists(ctx context.Context, node *v1.Node) (bool, error
 	}
 	if unmanaged {
 		klog.V(4).Infof("InstanceExists: omitting unmanaged node %q", node.Name)
-		return false, nil
+		return true, nil
 	}
 
 	providerID := node.Spec.ProviderID

pkg/provider/azure_instances_test.go

@@ -989,4 +989,21 @@ func TestCloud_InstanceExists(t *testing.T) {
 		assert.NoError(t, err)
 		assert.False(t, exist)
 	})
+	t.Run("should return true when instance is not managed by azure", func(t *testing.T) {
+		ctx := context.Background()
+		cloud := GetTestCloud(ctrl)
+		cloud.unmanagedNodes = sets.New("foo")
+		node := &v1.Node{
+			ObjectMeta: metav1.ObjectMeta{
+				Name: "foo",
+				Labels: map[string]string{
+					"kubernetes.azure.com/managed": "false",
+				},
+			},
+		}
+
+		exist, err := cloud.InstanceExists(ctx, node)
+		assert.NoError(t, err)
+		assert.True(t, exist)
+	})
 }