Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing port numbers at runtime leads to deployment errors when using K8s/OpenShift extensions #32882

Closed
Sgitario opened this issue Apr 25, 2023 · 1 comment · Fixed by #33789
Labels
area/kubernetes kind/bug Something isn't working
Milestone

Comments

@Sgitario
Copy link
Contributor

Describe the bug

Something else I saw today is that the port configuration (for Management and HTTP configuration) are runtime properties.

This means that if users change the port number at runtime, the generated manifests for OpenShift / Kubernetes will stop working.

I would say this is a very major issue, though the fix (changing these properties to be build-time-only properties) implies adding a strong limitation for users (since users won't be able to update the port to use any longer).

Not sure if we can / or make sense to allow changing these properties if and only if the Kubernetes extension is not present.

Relates #32225 (comment)

Expected behavior

No response

Actual behavior

No response

How to Reproduce?

No response

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response

@Sgitario Sgitario added the kind/bug Something isn't working label Apr 25, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Apr 25, 2023

/cc @geoand (kubernetes,openshift), @iocanel (kubernetes,openshift)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kubernetes kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant