Skip to content

[WIP] add NetworkPolicy for MultiCluster Components - #3839

Open
khrm wants to merge 1 commit into
tektoncd:mainfrom
khrm:feat/networkpolicy-scheduler-proxyaae-syncerservice
Open

[WIP] add NetworkPolicy for MultiCluster Components#3839
khrm wants to merge 1 commit into
tektoncd:mainfrom
khrm:feat/networkpolicy-scheduler-proxyaae-syncerservice

Conversation

@khrm

@khrm khrm commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Add reconciler-based NetworkPolicy management for three components:

  • TektonScheduler: default-deny + controller (Prometheus 8443) + webhook (9443, Prometheus 8443) policies in tekton-kueue namespace
  • TektonMulticlusterProxyAAE: default-deny + proxy (ingress 8080) policies in proxy-aae namespace
  • SyncerService (OpenShift only): default-deny + controller (egress-only) policies in syncer-service namespace

Each component gets spec.networkPolicy (NetworkPolicyConfig) wired from TektonConfig, with validation, CustomSet-based InstallerSet management, and FinalizeKind cleanup.

Assisted-by: Claude Code (claude-opus-4-6)

Changes

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

See the contribution guide for more details.

Release Notes

NONE

@tekton-robot tekton-robot added the release-note-none Denotes a PR that doesnt merit a release note. label Jul 29, 2026
@tekton-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please ask for approval from khrm after the PR has been reviewed.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jul 29, 2026
@codecov

codecov Bot commented Jul 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 25.59%. Comparing base (ddea504) to head (8eb9955).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3839      +/-   ##
==========================================
+ Coverage   25.40%   25.59%   +0.19%     
==========================================
  Files         449      452       +3     
  Lines       23477    23657     +180     
==========================================
+ Hits         5964     6055      +91     
- Misses      16822    16910      +88     
- Partials      691      692       +1     
Flag Coverage Δ
unit-tests 25.59% <ø> (+0.19%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@khrm khrm changed the title feat: add NetworkPolicy for MultiCluster Compo… [WIP] add NetworkPolicy for MultiCluster Components Jul 30, 2026
@tekton-robot tekton-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 30, 2026
Add reconciler-based NetworkPolicy management for three components:

- TektonScheduler: default-deny + controller (Prometheus 8443) + webhook
  (9443, Prometheus 8443) policies in tekton-kueue namespace
- TektonMulticlusterProxyAAE: default-deny + proxy (ingress 8080) policies
  in proxy-aae namespace
- SyncerService (OpenShift only): default-deny + controller (egress-only)
  policies in syncer-service namespace

Each component gets spec.networkPolicy (NetworkPolicyConfig) wired from
TektonConfig, with validation, CustomSet-based InstallerSet management,
and FinalizeKind cleanup.

Signed-off-by: Khurram Baig <kbaig@redhat.com>
Assisted-by: Claude Code (claude-opus-4-6)
@khrm
khrm force-pushed the feat/networkpolicy-scheduler-proxyaae-syncerservice branch from de4a20c to 8eb9955 Compare July 30, 2026 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. release-note-none Denotes a PR that doesnt merit a release note. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants