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

CiliumLocalRedirectPolicies support for experimental control-plane #35652

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

joamaki
Copy link
Contributor

@joamaki joamaki commented Oct 30, 2024

No description provided.

@joamaki joamaki added the release-note/misc This PR makes changes that have no direct user impact. label Oct 30, 2024
@joamaki joamaki force-pushed the pr/joamaki/experimental-lrp branch from 75ad90f to d44d141 Compare November 21, 2024 13:55

// hash of the podInfo. Used to detect relevant pod changes with minimal
// state.
func (pi *podInfo) hash() uint64 {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we use more bits here? e.g. sha256?

Copy link

github-actions bot commented Jan 4, 2025

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Jan 4, 2025
@joamaki joamaki removed the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Jan 13, 2025
@julianwiedmann julianwiedmann added the area/lrp Impacts Local Redirect Policy. label Jan 14, 2025
This is a port of the CiliumLocalRedirectPolicy handling to the
experimental control-plane.

This initial implementation is missing the following features:

- 'SkipRedirectFromBackend' flag / SkipLBMap
- tracking of netnscookie (for the skip functionality)
- adapting the "cilium-dbg lrp" command to dump Table[LocalRedirectPolicy]

Signed-off-by: Jussi Maki <jussi@isovalent.com>
@joamaki joamaki force-pushed the pr/joamaki/experimental-lrp branch from d44d141 to 55898f2 Compare January 31, 2025 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/lrp Impacts Local Redirect Policy. 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.

2 participants