Align resources naming convention

Kuadrant · Nov 20, 2023 · 9b0edd8 · 9b0edd8
1 parent a0a88c2
commit 9b0edd8
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 13 deletions.
diff --git a/controllers/limitador_controller_test.go b/controllers/limitador_controller_test.go
@@ -179,7 +179,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&createdLimitadorDeployment)
 
@@ -353,7 +353,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&updatedLimitadorDeployment)
 
@@ -396,7 +396,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&updatedLimitadorDeployment)
 
@@ -544,7 +544,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&createdLimitadorDeployment)
 
@@ -609,7 +609,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&updatedLimitadorDeployment)
 
@@ -672,7 +672,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&updatedLimitadorDeployment)
 
@@ -711,7 +711,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&deploymentObj)
 
@@ -730,7 +730,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&updateDeploymentObj)
 
@@ -825,7 +825,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&deploymentObj)
 
@@ -908,7 +908,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&deploymentObj)
 
@@ -955,7 +955,7 @@ var _ = Describe("Limitador controller", func() {
 					context.TODO(),
 					types.NamespacedName{
 						Namespace: LimitadorNamespace,
-						Name:      limitadorObj.Name,
+						Name:      limitador.DeploymentName(limitadorObj),
 					},
 					&deploymentObj)
 

diff --git a/pkg/limitador/k8s_objects.go b/pkg/limitador/k8s_objects.go
@@ -71,7 +71,7 @@ func Deployment(limitador *limitadorv1alpha1.Limitador, deploymentOptions Deploy
 			APIVersion: "apps/v1",
 		},
 		ObjectMeta: metav1.ObjectMeta{
-			Name:      limitador.ObjectMeta.Name,      // TODO: revisit later. For now assume same.
+			Name:      DeploymentName(limitador),
 			Namespace: limitador.ObjectMeta.Namespace, // TODO: revisit later. For now assume same.
 			Labels:    Labels(limitador),
 		},
@@ -164,7 +164,7 @@ func LimitsConfigMap(limitadorObj *limitadorv1alpha1.Limitador) (*v1.ConfigMap,
 }
 
 func LimitsConfigMapName(limitadorObj *limitadorv1alpha1.Limitador) string {
-	return fmt.Sprintf("limits-config-%s", limitadorObj.Name)
+	return fmt.Sprintf("limitador-limits-config-%s", limitadorObj.Name)
 }
 
 func ServiceName(limitadorObj *limitadorv1alpha1.Limitador) string {
@@ -175,6 +175,10 @@ func PVCName(limitadorObj *limitadorv1alpha1.Limitador) string {
 	return fmt.Sprintf("limitador-%s", limitadorObj.Name)
 }
 
+func DeploymentName(limitadorObj *limitadorv1alpha1.Limitador) string {
+	return fmt.Sprintf("limitador-%s", limitadorObj.Name)
+}
+
 func PodDisruptionBudget(limitadorObj *limitadorv1alpha1.Limitador) *policyv1.PodDisruptionBudget {
 	return &policyv1.PodDisruptionBudget{
 		ObjectMeta: metav1.ObjectMeta{