Skip to content
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

endpoint / ApplyPolicyMapChanges: fix incorrect comment #31790

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

squeed
Copy link
Contributor

@squeed squeed commented Apr 5, 2024

This comment incorrectly referenced some now-reverted behavior. Remove this now so we don't confuse ourselves.

Fixes: #31775

This comment incorrectly referenced some now-reverted behavior. Remove
this now so we don't confuse ourselves.

Signed-off-by: Casey Callendrello <cdc@isovalent.com>
@squeed squeed added the release-note/misc This PR makes changes that have no direct user impact. label Apr 5, 2024
@squeed squeed requested a review from a team as a code owner April 5, 2024 09:03
@squeed squeed requested a review from aditighag April 5, 2024 09:03
@squeed
Copy link
Contributor Author

squeed commented Apr 15, 2024

This just changes a code comment, marking as ready-to-merge.

@squeed squeed added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Apr 15, 2024
@aanm aanm disabled auto-merge April 23, 2024 09:15
@aanm aanm merged commit d1e774b into cilium:main Apr 23, 2024
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants