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

pkg/endpoint: clean up DatapathRegenerationLevel #32604

Merged
merged 1 commit into from
May 22, 2024

Conversation

lmb
Copy link
Contributor

@lmb lmb commented May 17, 2024

regeneration.RegenerateWithDatapathLoad is now equivalent to RegenerateWithDatapathRewrite since the loader itself decides if and when it has to recompile programs.

Rename RegenerateWithDatapathRewrite to RegenerateWithDatapath and do other misc cleanups.

regeneration.RegenerateWithDatapathLoad is now equivalent to
RegenerateWithDatapathRewrite since the loader itself decides if
and when it has to recompile programs.

Rename RegenerateWithDatapathRewrite to RegenerateWithDatapath and
do other misc cleanups.

Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
@lmb lmb added sig/policy Impacts whether traffic is allowed or denied based on user-defined policies. release-note/misc This PR makes changes that have no direct user impact. sig/agent Cilium agent related. labels May 17, 2024
@lmb
Copy link
Contributor Author

lmb commented May 17, 2024

/test

@lmb lmb marked this pull request as ready for review May 17, 2024 14:42
@lmb lmb requested review from a team as code owners May 17, 2024 14:42
@lmb lmb requested a review from tommyp1ckles May 17, 2024 14:42
@lmb lmb enabled auto-merge May 22, 2024 08:42
@lmb lmb added this pull request to the merge queue May 22, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label May 22, 2024
Merged via the queue into cilium:main with commit e71a1e1 May 22, 2024
65 of 66 checks passed
@lmb lmb deleted the pr/lmb/regen-level-cleanup branch May 22, 2024 09:37
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. sig/agent Cilium agent related. sig/policy Impacts whether traffic is allowed or denied based on user-defined policies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants