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
Allows multi-level KV prefixes #6664
Conversation
d9b9827
to
87bb1bd
Compare
It is possible to review and merge this PR? Will be helpful for us using Zookeeper as backend config store |
30151ca
to
1be56ac
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
61c7e7b
to
744e395
Compare
I can't see this in the latest release: v2.2.11. Any chance of seeing it soon? Thanks! |
What does this PR do?
Allows slashes in root key of KV-storages.
Motivation
We've got single KV storage shared across many of teams. Each team have its own directory in it. My team soon to have some separated Traefiks. So it would be cool, if we can separate configurations as:
There is no information about inadmissibility of slashes neither in documentation nor in logs, only message: "invalid root label". I see two possibilities of resolving this situation: add this constraint to documentation or add this feature.
As it's technically not too complex to implement, I chose 2nd variant.
More