diff --git a/controllers/limitador_controller_test.go b/controllers/limitador_controller_test.go index b11071a3..7ec0a6c3 100644 --- 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 index 914329d1..a7d676f3 100644 --- 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{