-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Spring Config support #5662
Comments
This also ties into the discussions around "dynamic config" for MP Config and SmallRye Config, and how far down that road we want to go |
@kenfinnigan have things changed on the Smallrye side with the latest big changes to configuration? |
changed in what sense? the current situation is that dynamic config is still not really supported directly |
That answers my question :). I was basically wondering if now have dynamic config support |
Depending on the
and calling If the |
Great info thanks! How about the ability to add a config source based on some property? I am thinking that for a ConfigMapConfigSource for example we need at runtime to know the Kubernete API URL. |
Do you mean enabling a |
Closer to the latter. |
Ah ok, behavior like that is currently not defined in the spec. However, we're going to explore an implementation to do something like that in SmallRye Config to hopefully get it in the spec |
That's cool, thanks for the info! I'll keep an eye out for it |
@kenfinnigan I am assuming you were referring to this issue? |
Yes |
Description
We need to see how much of Spring Config make sense. Do we want to add support for Config Server both as the server and as a client?
The idea for this came from the Micronaut vs Quarkus talk at Devoxx where the speaker mentioned that Micronaut could work well in environments where Spring Config Server is being used
For reference: https://github.com/vert-x3/vertx-config/tree/master/vertx-config-spring-config-server
Discusion
From @cescoffier
Analysis
(links to analysis docs containing architecture design work, requirements gathering, etc)
Tasks
The text was updated successfully, but these errors were encountered: