Skip to content

Commit 9b0edd8

Browse files
committed
Align resources naming convention
1 parent a0a88c2 commit 9b0edd8

File tree

2 files changed

+17
-13
lines changed

2 files changed

+17
-13
lines changed

controllers/limitador_controller_test.go

+11-11
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ var _ = Describe("Limitador controller", func() {
179179
context.TODO(),
180180
types.NamespacedName{
181181
Namespace: LimitadorNamespace,
182-
Name: limitadorObj.Name,
182+
Name: limitador.DeploymentName(limitadorObj),
183183
},
184184
&createdLimitadorDeployment)
185185

@@ -353,7 +353,7 @@ var _ = Describe("Limitador controller", func() {
353353
context.TODO(),
354354
types.NamespacedName{
355355
Namespace: LimitadorNamespace,
356-
Name: limitadorObj.Name,
356+
Name: limitador.DeploymentName(limitadorObj),
357357
},
358358
&updatedLimitadorDeployment)
359359

@@ -396,7 +396,7 @@ var _ = Describe("Limitador controller", func() {
396396
context.TODO(),
397397
types.NamespacedName{
398398
Namespace: LimitadorNamespace,
399-
Name: limitadorObj.Name,
399+
Name: limitador.DeploymentName(limitadorObj),
400400
},
401401
&updatedLimitadorDeployment)
402402

@@ -544,7 +544,7 @@ var _ = Describe("Limitador controller", func() {
544544
context.TODO(),
545545
types.NamespacedName{
546546
Namespace: LimitadorNamespace,
547-
Name: limitadorObj.Name,
547+
Name: limitador.DeploymentName(limitadorObj),
548548
},
549549
&createdLimitadorDeployment)
550550

@@ -609,7 +609,7 @@ var _ = Describe("Limitador controller", func() {
609609
context.TODO(),
610610
types.NamespacedName{
611611
Namespace: LimitadorNamespace,
612-
Name: limitadorObj.Name,
612+
Name: limitador.DeploymentName(limitadorObj),
613613
},
614614
&updatedLimitadorDeployment)
615615

@@ -672,7 +672,7 @@ var _ = Describe("Limitador controller", func() {
672672
context.TODO(),
673673
types.NamespacedName{
674674
Namespace: LimitadorNamespace,
675-
Name: limitadorObj.Name,
675+
Name: limitador.DeploymentName(limitadorObj),
676676
},
677677
&updatedLimitadorDeployment)
678678

@@ -711,7 +711,7 @@ var _ = Describe("Limitador controller", func() {
711711
context.TODO(),
712712
types.NamespacedName{
713713
Namespace: LimitadorNamespace,
714-
Name: limitadorObj.Name,
714+
Name: limitador.DeploymentName(limitadorObj),
715715
},
716716
&deploymentObj)
717717

@@ -730,7 +730,7 @@ var _ = Describe("Limitador controller", func() {
730730
context.TODO(),
731731
types.NamespacedName{
732732
Namespace: LimitadorNamespace,
733-
Name: limitadorObj.Name,
733+
Name: limitador.DeploymentName(limitadorObj),
734734
},
735735
&updateDeploymentObj)
736736

@@ -825,7 +825,7 @@ var _ = Describe("Limitador controller", func() {
825825
context.TODO(),
826826
types.NamespacedName{
827827
Namespace: LimitadorNamespace,
828-
Name: limitadorObj.Name,
828+
Name: limitador.DeploymentName(limitadorObj),
829829
},
830830
&deploymentObj)
831831

@@ -908,7 +908,7 @@ var _ = Describe("Limitador controller", func() {
908908
context.TODO(),
909909
types.NamespacedName{
910910
Namespace: LimitadorNamespace,
911-
Name: limitadorObj.Name,
911+
Name: limitador.DeploymentName(limitadorObj),
912912
},
913913
&deploymentObj)
914914

@@ -955,7 +955,7 @@ var _ = Describe("Limitador controller", func() {
955955
context.TODO(),
956956
types.NamespacedName{
957957
Namespace: LimitadorNamespace,
958-
Name: limitadorObj.Name,
958+
Name: limitador.DeploymentName(limitadorObj),
959959
},
960960
&deploymentObj)
961961

pkg/limitador/k8s_objects.go

+6-2
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ func Deployment(limitador *limitadorv1alpha1.Limitador, deploymentOptions Deploy
7171
APIVersion: "apps/v1",
7272
},
7373
ObjectMeta: metav1.ObjectMeta{
74-
Name: limitador.ObjectMeta.Name, // TODO: revisit later. For now assume same.
74+
Name: DeploymentName(limitador),
7575
Namespace: limitador.ObjectMeta.Namespace, // TODO: revisit later. For now assume same.
7676
Labels: Labels(limitador),
7777
},
@@ -164,7 +164,7 @@ func LimitsConfigMap(limitadorObj *limitadorv1alpha1.Limitador) (*v1.ConfigMap,
164164
}
165165

166166
func LimitsConfigMapName(limitadorObj *limitadorv1alpha1.Limitador) string {
167-
return fmt.Sprintf("limits-config-%s", limitadorObj.Name)
167+
return fmt.Sprintf("limitador-limits-config-%s", limitadorObj.Name)
168168
}
169169

170170
func ServiceName(limitadorObj *limitadorv1alpha1.Limitador) string {
@@ -175,6 +175,10 @@ func PVCName(limitadorObj *limitadorv1alpha1.Limitador) string {
175175
return fmt.Sprintf("limitador-%s", limitadorObj.Name)
176176
}
177177

178+
func DeploymentName(limitadorObj *limitadorv1alpha1.Limitador) string {
179+
return fmt.Sprintf("limitador-%s", limitadorObj.Name)
180+
}
181+
178182
func PodDisruptionBudget(limitadorObj *limitadorv1alpha1.Limitador) *policyv1.PodDisruptionBudget {
179183
return &policyv1.PodDisruptionBudget{
180184
ObjectMeta: metav1.ObjectMeta{

0 commit comments

Comments
 (0)