Small typo and improvement on the k8s guide #29213
Conversation
| ---- | ||
| quarkus.kubernetes.ingress.expose=true | ||
| # To change the Ingress host. By default, it's empty. | ||
| quarkus.kubernetes.ingress.host=prod.svc.url | ||
| # To change the Ingress path of the generated Ingress rule. By default, it's "/". | ||
| quarkus.kubernetes.ports.http.path=/prod | ||
| quarkus.kubernetes.ingress.path=/prod |
There was a problem hiding this comment.
This property does not exist, the right one is quarkus.kubernetes.ports.http.path.
There was a problem hiding this comment.
Oh yes, I messed up with rule configuration.
By the way, it's not very easy to understand why a Kubernetes port have a path ?
There was a problem hiding this comment.
That's very true... We should address this not-very user-friendly property sooner than later.
There was a problem hiding this comment.
OK, I can open an issue.
Should I close the PR or just remove this change ? Other change are cosmetic.
There was a problem hiding this comment.
For me, it's fine to just remove this change and proceed with the rest.
And yes, feel free to open an issue to improve this configuration.
There was a problem hiding this comment.
Line removed and issue create: #29279
loicmathieu
force-pushed
the
k8s-guide
branch
from
cd1617b to
bca34af
Compare
gsmet
added
triage/backport?
triage/backport-2.13
and removed
triage/invalid
No description provided.