Skip to content

Align networking extensions to provider extensions #547

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

Conversation

ScheererJ
Copy link
Member

How to categorize this PR?

/area networking
/kind technical-debt

What this PR does / why we need it:

Align networking extensions to provider extensions. This change adapts the extension and admission components to be more in line with the provider extensions.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

I used provider aws as the source for the comparison.

Release note:

The networking-cilium extension now uses the same helm values as the provider extensions.

@ScheererJ ScheererJ requested review from a team as code owners April 10, 2025 13:49
@gardener-robot gardener-robot added needs/review area/networking Networking related kind/technical-debt Something that is only solved on the surface, but requires more (re)work to be done properly size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Apr 10, 2025
@ScheererJ ScheererJ force-pushed the enhancement/align-networking-extensions-to-provider-extensions branch from fd05a84 to 43c1d4b Compare April 14, 2025 13:46
@ScheererJ ScheererJ force-pushed the enhancement/align-networking-extensions-to-provider-extensions branch from 43c1d4b to 032949a Compare April 15, 2025 07:57
@axel7born
Copy link
Contributor

/test pull-extension-networking-cilium-e2e-kind

@axel7born
Copy link
Contributor

/lgtm

@axel7born axel7born merged commit 548d36a into gardener:master Apr 15, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/networking Networking related kind/technical-debt Something that is only solved on the surface, but requires more (re)work to be done properly size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants