mitigate runtime panic when adopting cluster

Author: michaelhtm

apis/v1alpha1/ack-generate-metadata.yaml

@@ -1,5 +1,5 @@
 ack_generate_info:
-  build_date: "2024-09-19T16:42:31Z"
+  build_date: "2024-09-30T17:24:49Z"
   build_hash: f8f98563404066ac3340db0a049d2e530e5c51cc
   go_version: go1.23.0
   version: v0.38.1

pkg/resource/cluster/hook.go

@@ -192,7 +192,9 @@ func (rm *resourceManager) customUpdate(
 		return updatedRes, requeueWaitUntilCanModify(latest)
 	}
 
-	if delta.DifferentAt("Spec.Tags") {
+	// Ensure ACKResourceMetadata and ARN are not nil before derefrencing
+	// This can occur when adopting a resource
+	if delta.DifferentAt("Spec.Tags") && desired.ko.Status.ACKResourceMetadata != nil && desired.ko.Status.ACKResourceMetadata.ARN != nil {
 		if err := tags.SyncTags(
 			ctx, rm.sdkapi, rm.metrics,
 			string(*desired.ko.Status.ACKResourceMetadata.ARN),

pkg/resource/cluster/sdk.go

@@ -1,8 +1,8 @@
-	if r.ko.Spec.ResourcesVPCConfig.SubnetRefs != nil {
+	if r.ko.Spec.ResourcesVPCConfig != nil && r.ko.Spec.ResourcesVPCConfig.SubnetRefs != nil {
 		ko.Spec.ResourcesVPCConfig.SubnetRefs = r.ko.Spec.ResourcesVPCConfig.SubnetRefs
 	}
 
-	if r.ko.Spec.ResourcesVPCConfig.SecurityGroupRefs != nil {
+	if r.ko.Spec.ResourcesVPCConfig != nil && r.ko.Spec.ResourcesVPCConfig.SecurityGroupRefs != nil {
 		ko.Spec.ResourcesVPCConfig.SecurityGroupRefs = r.ko.Spec.ResourcesVPCConfig.SecurityGroupRefs
 	}