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

Replace '.' with '_' when generating env var in Kubernetes extension #14292

Merged
merged 2 commits into from Jan 14, 2021

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Jan 14, 2021

Fixes: #14291

Copy link
Contributor

@iocanel iocanel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -1,4 +1,5 @@
quarkus.kubernetes.env.fields.fromfield=metadata.name
quarkus.kubernetes.env.vars.envvar=value
quarkus.kubernetes.env.vars."quarkus.kubernetes-config.enabled"=true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow! We must document this!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The problem is, where? :)

@geoand geoand added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jan 14, 2021
@geoand geoand merged commit bce536f into quarkusio:master Jan 14, 2021
@ghost ghost added this to the 1.12 - master milestone Jan 14, 2021
@geoand geoand deleted the k8s-config-env-var branch January 14, 2021 10:35
@gsmet gsmet modified the milestones: 1.12 - master, 1.11.1.Final Jan 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kubernetes triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting quarkus properties using the Kubernetes extension results in incorrect env var name
3 participants