FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add more details for application load balancers in error state by dergeberl · Pull Request #15 · stackitcloud/application-load-balancer-controller · GitHub

Add more details for application load balancers in error state - #15

Merged
ske-prow[bot] merged 5 commits into
mainfrom
moreDetailedALBErrors
Aug 5, 2026
Merged

Add more details for application load balancers in error state#15
ske-prow[bot] merged 5 commits into
mainfrom
moreDetailedALBErrors

Conversation

Copy link
Copy Markdown
Member

How to categorize this PR?

/kind enhancement

What this PR does / why we need it:
Returns more detailed errors for LBs when available in SDK.

Same PR is also open for NLBs stackitcloud/cloud-provider-stackit#1410

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Breaking changes:

ske-prow Bot added kind/enhancement Enhancement, improvement, extension do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jul 27, 2026
dergeberl force-pushed the moreDetailedALBErrors branch from 9d96127 to 240c79f Compare July 28, 2026 06:53
ske-prow Bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jul 28, 2026
dergeberl force-pushed the moreDetailedALBErrors branch from 240c79f to d6dcaba Compare July 28, 2026 06:54
dergeberl force-pushed the moreDetailedALBErrors branch from d6dcaba to 9ec57ac Compare July 28, 2026 06:56

Copy link
Copy Markdown
Member Author

/hold
wait for feedback of LB Team
/cc @fischerman
/cc @stackitcloud/ske-infrastructure

ske-prow Bot requested review from a team and fischerman July 28, 2026 06:57
ske-prow Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 28, 2026
dergeberl changed the title WIP: Add more details for application load balancers in error state Add more details for application load balancers in error state Jul 28, 2026
ske-prow Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 28, 2026
Comment thread pkg/stackit/fake/alb.go Outdated

ske-prow Bot commented Jul 28, 2026

Copy link
Copy Markdown

@fischerman: adding LGTM is restricted to approvers and reviewers in OWNERS files.

Details

In response to this:

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.

Copy link
Copy Markdown
Member Author

/unhold

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 30, 2026

ske-prow Bot commented Aug 5, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

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

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:

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

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 Aug 5, 2026

ske-prow Bot commented Aug 5, 2026

Copy link
Copy Markdown

LGTM label has been added.

DetailsGit tree hash: a10787e0ef450a5c4ec120e9edafe731e8443cbb

ske-prow Bot merged commit 0360425 into main Aug 5, 2026
3 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/enhancement Enhancement, improvement, extension lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL