kvstore: drop obsolete removal of legacy prefixes #35995
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When running in kvstore mode, Cilium agents attempt to remove legacy prefixes from the kvstore itself at startup. Given that this cleanup logic has been in place for almost 7 years now [1], it is fair to assume that it is no longer necessary at this point. Hence, let's drop it, for simplicity. [1]: 7ccf877 ("kvstore: New kvstore abstraction API") Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
|
/test |
marseel
approved these changes
Nov 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
When running in kvstore mode, Cilium agents attempt to remove legacy prefixes from the kvstore itself at startup. Given that this cleanup logic has been in place for almost 7 years now [1], it is fair to assume that it is no longer necessary at this point. Hence, let's drop it, for simplicity.
[1]: 7ccf877 ("kvstore: New kvstore abstraction API")