Optimize PowerVS method to fetch bucket region to avoid duplicacy

Author: ChandrikaHarish

cloud/scope/powervs/powervs_cluster.go

@@ -2931,15 +2931,7 @@ func (s *ClusterScope) isResourceCreatedByController(resourceType infrav1.Resour
 	return false
 }
 
-// TODO: duplicate function, optimize it.
+// bucketRegion returns the region to use for COS bucket for the ClusterScope.
 func (s *ClusterScope) bucketRegion() string {
-	if s.COSInstance() != nil && s.COSInstance().BucketRegion != "" {
-		return s.COSInstance().BucketRegion
-	}
-	// if the bucket region is not set, use vpc region
-	vpcDetails := s.VPC()
-	if vpcDetails != nil && vpcDetails.Region != nil {
-		return *vpcDetails.Region
-	}
-	return ""
+	return bucketRegionFrom(s.COSInstance(), s.VPC())
 }

cloud/scope/powervs/powervs_machine.go

@@ -1146,15 +1146,7 @@ func (m *MachineScope) bucketName() string {
 	return fmt.Sprintf("%s-%s", m.IBMPowerVSCluster.GetName(), "cosbucket")
 }
 
-// TODO: duplicate function, optimize it.
+// bucketRegion returns the region to use for COS bucket for the MachineScope.
 func (m *MachineScope) bucketRegion() string {
-	if m.IBMPowerVSCluster.Spec.CosInstance != nil && m.IBMPowerVSCluster.Spec.CosInstance.BucketRegion != "" {
-		return m.IBMPowerVSCluster.Spec.CosInstance.BucketRegion
-	}
-	// if the bucket region is not set, use vpc region
-	vpcDetails := m.IBMPowerVSCluster.Spec.VPC
-	if vpcDetails != nil && vpcDetails.Region != nil {
-		return *vpcDetails.Region
-	}
-	return ""
+	return bucketRegionFrom(m.IBMPowerVSCluster.Spec.CosInstance, m.IBMPowerVSCluster.Spec.VPC)
 }

cloud/scope/powervs/util.go

@@ -57,3 +57,14 @@ func CheckCreateInfraAnnotation(cluster infrav1.IBMPowerVSCluster) bool {
 	}
 	return createInfra
 }
+
+func bucketRegionFrom(cos *infrav1.CosInstance, vpc *infrav1.VPCResourceReference) string {
+	if cos != nil && cos.BucketRegion != "" {
+		return cos.BucketRegion
+	}
+	// if the bucket region is not set, use vpc region
+	if vpc != nil && vpc.Region != nil {
+		return *vpc.Region
+	}
+	return ""
+}