adding '/' to consul kv del path to avoid greedy pattern matching and… #182
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
… remove desired key/subtree
Issue - having files:
After both are added and populated to consul, when I modify test.yaml only then test.yaml is updated in consul, but test_dev.yaml is removed at the same time.
I think it is related to not having '/' at del path. See: https://www.consul.io/docs/commands/kv/delete.html ("Trailing slashes are important" section)
Local modification fixed problem, but I am not aware of other consequences - would be happy to hear. Also there is second dev reference at the top of file I am not sure I should also modify.