fix: fix missing status on objects

Signed-off-by: Dustin Scott <[email protected]>

config/samples/cluster/rosa_simple.yaml

@@ -4,7 +4,7 @@ metadata:
   name: rosa-simple
 spec:
   accountID: "660250927410"
-  displayName: dscott
+  displayName: dscott-test3
   tags:
     owner: dscott
   iam:

config/samples/machinepool/sample.yaml

@@ -1,15 +1,15 @@
 apiVersion: ocm.mobb.redhat.com/v1alpha1
 kind: MachinePool
 metadata:
-  name: sample
+  name: sample-missing
 spec:
   wait: false
-  clusterName: "my-cluster"
+  clusterName: "dscott-test3"
   minimumNodesPerZone: 1
   maximumNodesPerZone: 1
   instanceType: m5.xlarge
   labels:
-    owner: my-user
+    owner: dscott
   aws:
     spotInstances:
       enabled: false

controllers/request/cluster.go

@@ -6,6 +6,8 @@ import (
 	"reflect"
 
 	clustersmgmtv1 "github.com/openshift-online/ocm-sdk-go/clustersmgmt/v1"
+	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
+	"k8s.io/apimachinery/pkg/runtime"
 
 	"github.com/rh-mobb/ocm-operator/pkg/kubernetes"
 )
@@ -49,8 +51,10 @@ func GetUpstreamCluster(request Cluster, client ClusterFetcher) (cluster *cluste
 			return cluster, fmt.Errorf("%s: [%s] - %w", errRetrieveClusterMessage, request.GetClusterName(), ErrMissingClusterID)
 		}
 	} else {
+		var response *clustersmgmtv1.ClusterGetResponse
+
 		// retrieve the cluster from ocm by id
-		response, err := client.For(request.GetObject().GetClusterID()).Get().Send()
+		response, err = client.For(request.GetObject().GetClusterID()).Get().Send()
 		if err != nil {
 			if response.Status() == http.StatusNotFound {
 				return cluster, nil
@@ -61,7 +65,17 @@ func GetUpstreamCluster(request Cluster, client ClusterFetcher) (cluster *cluste
 	}
 
 	// keep track of the original object
-	original := request.GetObject()
+	object := request.GetObject().DeepCopyObject()
+
+	// convert the client object to a runtime object.  to do this,
+	// we convert to an unstructured object which satisfies both a client.Object
+	// and runtime.Object interface.
+	// TODO: this is a little sloppy and needs some attention.
+	objectMap, objectErr := runtime.DefaultUnstructuredConverter.ToUnstructured(object)
+	if objectErr != nil {
+		return cluster, fmt.Errorf("unable to convert client.Object to runtime.Object - %w", err)
+	}
+	original := &unstructured.Unstructured{Object: objectMap}
 
 	// update the object
 	request.SetClusterStatus(cluster)