Kubernetes extension should not produce env vars with slashes when mapping mp-rest client config to env vars
See original GitHub issueGiven an application.properties
like the following:
foo/mp-rest/url=http://localhost:8080
quarkus.kubernetes.env.mapping.foo/mp-rest/url.from-secret=something-parameters
quarkus.kubernetes.env.mapping.foo/mp-rest/url.with-key=foo.url
The generated kubernetes.yml
uses the FOO/MP_REST/URL
env var, which is invalid, as noted by https://github.com/eclipse/microprofile-rest-client/issues/171 and https://github.com/eclipse/microprofile-config/issues/420 in the past.
We should replace /
with _
and figure out if MP-Config actually can do the reverse mapping.
Issue Analytics
- State:
- Created 3 years ago
- Comments:6 (5 by maintainers)
Top Results From Across the Web
How To Handle Environment Variables With Kubernetes?
This post shows you the variety of ways Kuberentes helps you create and manage environment variables within kubernetes.
Read more >Invalid env value with Quarkus RestClient property when ...
I named the environment variable replacing "/" to "" and it worked. I noticed that Quarkus kubernetes extension had already changed "-" to...
Read more >MicroProfile Config properties :: Open Liberty Docs
MicroProfile Config can be used by applications as a single API that can retrieve configuration information from different sources.
Read more >Environment Variables Configuration Pattern - Weaveworks
Wanting to learn about various Kubernetes patterns? Check out our blog post regarding the environment variables configuration pattern.
Read more >Kubernetes Environment Variables - Alen Komljen
Instead of specifying each environment variable individually we can reference the whole config map. Set Environment Variables. To set the ...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
I think we should provide a dotted version of
/mp-rest/url
to improve the user experience and to not confuse people. I mean you can still use it but a dotted version will also be accepted and map to the right configuration.Well, if we didn’t want to confuse people, we would have proper Quarkus config 😃