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

WIP: Add cross-cluster traffic protection #10504

Draft
wants to merge 46 commits into
base: main
Choose a base branch
from

Conversation

aknuds1
Copy link
Contributor

@aknuds1 aknuds1 commented Jan 23, 2025

What this PR does

Add cross-cluster traffic protection functionality. This is controlled through the following flags:

  • -server.cluster-verification-label -> Define the cluster verification label, which is sent with HTTP and gRPC requests
  • -server.cluster-verification-label-check -> Enable checking of the cluster verification label when receiving HTTP and gRPC requests
  • -server.auxiliary-url-paths -> Auxiliary HTTP URL paths that should be exempt from checking of the cluster verification label (e.g. k8s webhooks)
TODO

Which issue(s) this PR fixes or relates to

Fixes #

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

Copy link
Contributor

github-actions bot commented Jan 23, 2025

@aknuds1 aknuds1 force-pushed the arve/distributor-validate-cluster-name branch 10 times, most recently from a4256c8 to e3dfcd4 Compare January 30, 2025 08:20
@aknuds1 aknuds1 force-pushed the arve/distributor-validate-cluster-name branch 2 times, most recently from 34bf241 to 5dfcf69 Compare January 30, 2025 15:13
Signed-off-by: Arve Knudsen <[email protected]>
Signed-off-by: Arve Knudsen <[email protected]>
Signed-off-by: Arve Knudsen <[email protected]>
@aknuds1 aknuds1 force-pushed the arve/distributor-validate-cluster-name branch from 5dfcf69 to f17900e Compare January 30, 2025 19:43
@duricanikolic duricanikolic force-pushed the arve/distributor-validate-cluster-name branch from b72c109 to 2665a46 Compare February 2, 2025 08:38
@duricanikolic duricanikolic force-pushed the arve/distributor-validate-cluster-name branch from 177249c to 68cff50 Compare February 3, 2025 23:17
@duricanikolic duricanikolic force-pushed the arve/distributor-validate-cluster-name branch from a5db9ce to ccc6449 Compare February 7, 2025 16:01
@aknuds1 aknuds1 force-pushed the arve/distributor-validate-cluster-name branch from 9c7e204 to 583f16f Compare February 7, 2025 16:56
@aknuds1 aknuds1 force-pushed the arve/distributor-validate-cluster-name branch from 583f16f to b27c983 Compare February 7, 2025 17:29
@aknuds1 aknuds1 force-pushed the arve/distributor-validate-cluster-name branch from cb499c7 to fa29d71 Compare February 10, 2025 15:48
@aknuds1 aknuds1 changed the title WIP: Add -server.cluster flag WIP: Add cross-cluster traffic protection Feb 12, 2025
@duricanikolic duricanikolic force-pushed the arve/distributor-validate-cluster-name branch from e78bddc to bb6f4af Compare February 17, 2025 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants