Add ingress class name property part of Ingress config

Fix quarkusio#29858
Sgitario · Dec 21, 2022 · 3f28a4b · 3f28a4b
1 parent 3e68178
commit 3f28a4b
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 13 deletions.
diff --git a/...nt/src/main/java/io/quarkus/kubernetes/deployment/ApplyKubernetesIngressConfigurator.java b/...nt/src/main/java/io/quarkus/kubernetes/deployment/ApplyKubernetesIngressConfigurator.java
@@ -17,6 +17,7 @@ public void visit(KubernetesConfigFluent config) {
             KubernetesConfigFluent.IngressNested ingressConfigBuilder = config.withNewIngress();
             ingressConfigBuilder.withExpose(true);
             ingressConfig.host.ifPresent(ingressConfigBuilder::withHost);
+            ingressConfig.ingressClassName.ifPresent(ingressConfigBuilder::withIngressClassName);
 
             ingressConfigBuilder.endIngress();
         }

diff --git a/...etes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/IngressConfig.java b/...etes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/IngressConfig.java
@@ -21,6 +21,12 @@ public class IngressConfig {
     @ConfigItem
     Optional<String> host;
 
+    /**
+     * The class of the Ingress. If the ingressClassName is omitted, a default Ingress class is used.
+     */
+    @ConfigItem
+    Optional<String> ingressClassName;
+
     /**
      * Custom annotations to add to exposition (route or ingress) resources
      */

diff --git a/...uarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithIngressTest.java b/...uarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithIngressTest.java
@@ -21,12 +21,14 @@
 
 public class KubernetesWithIngressTest {
 
+    private static final String APP_NAME = "kubernetes-with-ingress";
+
     @RegisterExtension
     static final QuarkusProdModeTest config = new QuarkusProdModeTest()
             .withApplicationRoot((jar) -> jar.addClasses(GreetingResource.class))
-            .setApplicationName("kubernetes-with-ingress")
+            .setApplicationName(APP_NAME)
             .setApplicationVersion("0.1-SNAPSHOT")
-            .withConfigurationResource("kubernetes-with-ingress.properties")
+            .withConfigurationResource(APP_NAME + ".properties")
             .setLogFileName("k8s.log")
             .setForcedDependencies(List.of(Dependency.of("io.quarkus", "quarkus-kubernetes", Version.getVersion())));
 
@@ -44,23 +46,19 @@ public void assertGeneratedResources() throws IOException {
 
         assertThat(kubernetesList.get(0)).isInstanceOfSatisfying(Deployment.class, d -> {
             assertThat(d.getMetadata()).satisfies(m -> {
-                assertThat(m.getName()).isEqualTo("kubernetes-with-ingress");
-            });
-
-            assertThat(d.getSpec()).satisfies(deploymentSpec -> {
-                assertThat(deploymentSpec.getTemplate()).satisfies(t -> {
-                    assertThat(t.getSpec()).satisfies(podSpec -> {
-
-                    });
-                });
+                assertThat(m.getName()).isEqualTo(APP_NAME);
             });
         });
 
         assertThat(kubernetesList).filteredOn(i -> "Ingress".equals(i.getKind())).singleElement().satisfies(item -> {
             assertThat(item).isInstanceOfSatisfying(Ingress.class, ingress -> {
                 //Check that labels and annotations are also applied to Routes (#10260)
                 assertThat(ingress.getMetadata()).satisfies(m -> {
-                    assertThat(m.getName()).isEqualTo("kubernetes-with-ingress");
+                    assertThat(m.getName()).isEqualTo(APP_NAME);
+                });
+
+                assertThat(ingress.getSpec()).satisfies(spec -> {
+                    assertThat(spec.getIngressClassName()).isEqualTo("Nginx");
                 });
 
                 assertThat(ingress.getSpec().getRules()).allSatisfy(rule -> {

diff --git a/...sts/kubernetes/quarkus-standard-way/src/test/resources/kubernetes-with-ingress.properties b/...sts/kubernetes/quarkus-standard-way/src/test/resources/kubernetes-with-ingress.properties
@@ -1 +1,2 @@
-quarkus.kubernetes.ingress.expose=true
+quarkus.kubernetes.ingress.expose=true
+quarkus.kubernetes.ingress.ingress-class-name=Nginx