Add application load balancer controller with ingress support by dergeberl · Pull Request #54 · stackitcloud/gardener-extension-provider-stackit

dergeberl · 2026-03-17T13:41:40Z

How to categorize this PR?

/kind enhancement

What this PR does / why we need it:

This PR adds the application load balancer controller (stackitcloud/application-load-balancer-controller) to the gardener-extension-provider-stackit.

There was already some tests merged in this repo which did not work. So it should be safe to change that parts.

This PR removes the existing config field deployALBIngressController, removes the old feature gate, name things consistently, add a validation with config option if alb controller is allowed

This PR makes use of the validation of the shoot via the admission webhook. Therefore this PR is based on #203 and has to be merged after this.

/hold
The application load balancer controller is currently still under development and this PR uses a feature build (see PR stackitcloud/application-load-balancer-controller#3). We only should merge this after we adopted the version in the imagevector to a released version.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Breaking changes:

stackit-ske · 2026-04-08T11:45:24Z

Our project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:

After 15d of inactivity, lifecycle/stale is applied
After 15d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
After 7d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

Mark this PR as fresh with /remove-lifecycle stale
Mark this PR as rotten with /lifecycle rotten
Close this PR with /close

/lifecycle stale

dergeberl · 2026-04-14T06:33:29Z

/lifecycle frozen

ske-prow · 2026-04-14T06:33:32Z

@dergeberl: The lifecycle/frozen label cannot be applied to Pull Requests.

Details

In response to this:

/lifecycle frozen

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

breuerfelix · 2026-07-15T07:14:40Z

/lgtm lets wait for a proper release

…evector

ske-prow · 2026-07-16T09:53:50Z

@dergeberl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
pull-gardener-extension-provider-stackit-infra-integration-tests	`1c5e7ac`	link	true	`/test pull-gardener-extension-provider-stackit-infra-integration-tests`

Full PR test history. Your PR dashboard. Command help for this repository.
Please help us cut down on flakes by linking this test failure to an open flake report or filing a new flake report if you can't find an existing one. Also see the gardener testing guideline for how to avoid and hunt flakes.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

breuerfelix · 2026-07-16T14:38:13Z

/override pull-gardener-extension-provider-stackit-integration-tests
only the prefix override is failing

ske-prow · 2026-07-16T14:38:18Z

@breuerfelix: Overrode contexts on behalf of breuerfelix: pull-gardener-extension-provider-stackit-integration-tests

Details

In response to this:

/override pull-gardener-extension-provider-stackit-integration-tests
only the prefix override is failing

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

ske-prow · 2026-07-16T14:38:35Z

LGTM label has been added.

Details

Git tree hash: c9db5a85ed95f9da2b01d86a5969a46a80a4bf2b

breuerfelix · 2026-07-17T12:06:01Z

/hold cancel

ske-prow · 2026-07-17T12:11:09Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: breuerfelix, nschad

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~OWNERS~~ [breuerfelix,nschad]

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

nschad · 2026-07-17T12:18:59Z

/override pull-gardener-extension-provider-stackit-integration-tests

ske-prow · 2026-07-17T12:19:04Z

@nschad: Overrode contexts on behalf of nschad: pull-gardener-extension-provider-stackit-integration-tests

Details

In response to this:

/override pull-gardener-extension-provider-stackit-integration-tests

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

ske-prow Bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. kind/enhancement Enhancement, improvement, extension size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Mar 17, 2026

jamand force-pushed the alb branch from 1c13677 to cc3a6bc Compare March 17, 2026 15:11

ske-prow Bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Mar 18, 2026

ske-prow Bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 8, 2026

dergeberl added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Apr 14, 2026

dergeberl force-pushed the alb branch from 1c5e7ac to c7a2ba3 Compare July 7, 2026 06:24

ske-prow Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 7, 2026

dergeberl force-pushed the alb branch 5 times, most recently from f53e478 to d1f39df Compare July 8, 2026 08:51

ske-prow Bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jul 8, 2026

dergeberl force-pushed the alb branch 8 times, most recently from 69f9b8e to 7c74c4c Compare July 8, 2026 13:06

dergeberl force-pushed the alb branch from 9b49c71 to 1c52e82 Compare July 13, 2026 13:15

dergeberl requested a review from a team July 13, 2026 13:16

dergeberl force-pushed the alb branch from 1c52e82 to 08957a0 Compare July 14, 2026 14:55

breuerfelix reviewed Jul 14, 2026

View reviewed changes

breuerfelix reviewed Jul 15, 2026

View reviewed changes

Comment thread imagevector/images.yaml Outdated

ske-prow Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 15, 2026

Integrate ALB Controller in gardener-extension-provider-stackit

f41683e

dergeberl force-pushed the alb branch from b0c6c83 to 6fc7b70 Compare July 15, 2026 12:42

ske-prow Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 15, 2026

dergeberl added 3 commits July 15, 2026 14:51

Add tests to validation

4353843

Add missing pdb and configmap

1b8359b

Fix wrong namings

2695efe

dergeberl force-pushed the alb branch from 6fc7b70 to 2695efe Compare July 15, 2026 12:52

Set released version for application-load-balancer-controller in imag…

4e0a844

…evector

breuerfelix approved these changes Jul 16, 2026

View reviewed changes

ske-prow Bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jul 16, 2026

ske-prow Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 17, 2026

nschad approved these changes Jul 17, 2026

View reviewed changes

ske-prow Bot merged commit a4ef6d0 into main Jul 17, 2026
4 checks passed

ske-prow Bot deleted the alb branch July 17, 2026 12:23

Uh oh!

Conversation

dergeberl commented Mar 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

stackit-ske commented Apr 8, 2026

Uh oh!

dergeberl commented Apr 14, 2026

Uh oh!

ske-prow Bot commented Apr 14, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

breuerfelix commented Jul 15, 2026

Uh oh!

ske-prow Bot commented Jul 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

breuerfelix commented Jul 16, 2026

Uh oh!

ske-prow Bot commented Jul 16, 2026

Uh oh!

ske-prow Bot commented Jul 16, 2026

Uh oh!

breuerfelix commented Jul 17, 2026

Uh oh!

ske-prow Bot commented Jul 17, 2026

Uh oh!

nschad commented Jul 17, 2026

Uh oh!

ske-prow Bot commented Jul 17, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

dergeberl commented Mar 17, 2026 •

edited

Loading

ske-prow Bot commented Jul 16, 2026 •

edited

Loading