Chapter 6: quarkus-rest-config: application.properties Questions

[edeandrea]

Sub-issue of quarkus-for-spring-developers/project-management#31

In src/main/resources/application.properties is

quarkus.kubernetes.ingress.host=chapter-6-quarkus-rest-config.127.0.0.1.nip.io
quarkus.kubernetes.ports.http.host-port=8080
quarkus.kubernetes.ports.http.container-port=8080

necessary?

[edeandrea]

@cmoulliard I don't know the answer to this. If these are required then we can simply close out this ticket.

• quarkus.kubernetes.ports.http.container-port: 8080 seems to be the default on the generated Service and Deployment if this property is not specified.
• quarkus.kubernetes.ports.http.host-port: This defaults to 80 if not specified. Is there a particular reason that the Service should be exposed on 8080 rather than 80?

• If quarkus.kubernetes.ingress.host isn't set, the resulting Ingress.spec looks like

  spec:
   rules:
     - http:
         paths:
           - backend:
               service:
                 name: chapter-6-quarkus-rest-config
                 port:
                   name: http
             pathType: Prefix
     - http:
         paths:
           - backend:
               service:
                 name: chapter-6-quarkus-rest-config
                 port:
                   name: http
             pathType: Prefix

  instead of

  spec:
    rules:
      - host: chapter-6-quarkus-rest-config.127.0.0.1.nip.io
        http:
          paths:
            - backend:
                service:
                  name: chapter-6-quarkus-rest-config
                  port:
                    name: http
              path: /
              pathType: Prefix

[cmoulliard]

quarkus.kubernetes.ingress.host=chapter-6-quarkus-rest-config.127.0.0.1.nip.io

This property is necessary to configure properly the ingress controller in order to access the service using the host address

[cmoulliard]

quarkus.kubernetes.ports.http.host-port=8080
quarkus.kubernetes.ports.http.container-port=8080

No. I will remove them ;-)

[cmoulliard]

Issues reported fixed. Can you check https://github.com/quarkus-for-spring-developers/examples/blob/main/chapter-6/chapter-6-quarkus-rest-config/README.md and close it when done please ? @edeandrea