Autoscaler should not try to monitor or scale virtual nodes and virtual node pools

Author: lxuan94-pp

cluster-autoscaler/cloudprovider/oci/nodepools/consts/annotations.go

@@ -28,4 +28,7 @@ const (
 
 	// EphemeralStorageSize is the freeform tag key that would be used to determine the ephemeral-storage size of the node
 	EphemeralStorageSize = "cluster-autoscaler/node-ephemeral-storage"
+
+	// OciNodePoolResourceIdent is the string identifier in the ocid that indicates the resource is a virtual node pool
+	OciVirtualNodeResourceIdent = "virtualnode"
 )

cluster-autoscaler/cloudprovider/oci/nodepools/oci_manager.go

@@ -630,6 +630,9 @@ func (m *ociManagerImpl) GetNodePoolNodes(np NodePool) ([]cloudprovider.Instance
 
 // GetNodePoolForInstance returns NodePool to which the given instance belongs.
 func (m *ociManagerImpl) GetNodePoolForInstance(instance ocicommon.OciRef) (NodePool, error) {
+	if strings.Contains(instance.InstanceID, npconsts.OciVirtualNodeResourceIdent) {
+		return nil, nil
+	}
 	if instance.NodePoolID == "" {
 		klog.V(4).Infof("node pool id missing from reference: %+v", instance)
 

cluster-autoscaler/cloudprovider/oci/nodepools/oci_manager_test.go

@@ -97,6 +97,16 @@ func TestGetNodePoolForInstance(t *testing.T) {
 	if np.Id() != "ocid2" {
 		t.Fatalf("got unexpected ocid %q ; wanted \"ocid2\"", np.Id())
 	}
+
+	// now verify node pool can be found via lookup up by instance id in cache
+	np, err = manager.GetNodePoolForInstance(ocicommon.OciRef{InstanceID: "virtualnode"})
+	if err != nil {
+		t.Fatalf("unexpected error: %+v", err)
+	}
+
+	if np != nil {
+		t.Fatalf("got unexpected ocid %q ; wanted nil", np.Id())
+	}
 }
 
 func TestGetNodePoolNodes(t *testing.T) {