Merge pull request quarkusio#27825 from Sgitario/kubernetes_fix_stateful

Kubernetes: Add default template.metadata in StatefulSet resource
Dairdevil · Sep 9, 2022 · cf0cbca · cf0cbca
2 parents 83c210b + f92c4ea
commit cf0cbca
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/...yment/src/main/java/io/quarkus/kubernetes/deployment/AddStatefulSetResourceDecorator.java b/...yment/src/main/java/io/quarkus/kubernetes/deployment/AddStatefulSetResourceDecorator.java
@@ -51,6 +51,8 @@ private void initStatefulSetResourceWithDefaults(StatefulSetBuilder builder) {
                 .editOrNewTemplate()
                 .editOrNewSpec()
                 .endSpec()
+                .editOrNewMetadata()
+                .endMetadata()
                 .endTemplate();
 
         // defaults for:

diff --git a/...ard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithStatefulSetResourceTest.java b/...ard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithStatefulSetResourceTest.java
@@ -57,6 +57,9 @@ public void assertGeneratedResources() throws IOException {
                             assertThat(podSpec.getTerminationGracePeriodSeconds()).isEqualTo(10);
                             assertThat(podSpec.getContainers()).allMatch(c -> APP_NAME.equals(c.getName()));
                         });
+                        assertThat(t.getMetadata()).satisfies(metadata -> {
+                            assertThat(metadata.getLabels()).containsEntry("app.kubernetes.io/name", APP_NAME);
+                        });
                     });
                     assertThat(statefulSetSpec.getSelector()).satisfies(ls -> {
                         assertThat(ls.getMatchLabels()).containsEntry("app.kubernetes.io/name", APP_NAME);