-
Notifications
You must be signed in to change notification settings - Fork 12
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
Support YAML in Consul #46
Comments
@radcortez as per our discussions, the Config Consul extension has been moved here. I thought Terraform would drop the other repo but it didn't, I will remove it manually. |
I remember about it, but I saw the other repo and I thought it was still active. |
Thanks! |
Hi @radcortez, I'm trying to upgrade my Quarkus App to the latest version(
logs
yml
|
I've tried it, and it works for me. Can you provide me with a reproducer? Thanks! |
Can somebody please share what the fix is to get this working. Yaml and Consul Config do not seem to work currently. |
I currently do not have enough knowledge of Quarkus Internals to provide a PR with the required changes to implement the YAML support for Consul. However, I would like to explain why this feature is essential. As a solutions architect, I will not sacrifice the human-readable ease of operations for DevOps by removing that. Furthermore, a spring boot config server will introduce additional maintenance, complexity, compute resources, and a layer of possible failure. I already have a complete Consul cluster with a Key/Value store synced across its cluster, being entirely reliant. I would ask the community, what is needed and what kind of pre-work can I provide without an in-depth knowledge of Quarkus to have the |
Yes, YAML is not supported with Consul (YAML is supported with Well, as you can see, this issue is opened for more than an year with little interest, so this was not in our priorities. All the YAML support for Config is already avalable. I guess what we need is a way to determine the payload type coming from Consul (in properties or YAML format) and use the proper reader to convert that to a |
From quarkusio/quarkus#19127
The text was updated successfully, but these errors were encountered: