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

Allow overriding of creation timeout for LoadBalancerMachines #279

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

maboehm
Copy link
Collaborator

@maboehm maboehm commented Jan 18, 2024

The LoadbalancerSet controller (specifically the LBM status controller) currently deletes any LBM that takes more than 10 Minutes to report their first condition. This change allows users to override this duration per object by setting the annotation yawol.stackit.cloud/creationTimeout on LoadBalancerMachine objects.

Copy link
Member

@timebertt timebertt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, minor nits only :)

api/v1beta1/loadbalancerset_types.go Outdated Show resolved Hide resolved
api/v1beta1/loadbalancerset_types.go Outdated Show resolved Hide resolved
api/v1beta1/loadbalancerset_types.go Outdated Show resolved Hide resolved
@maboehm maboehm requested a review from timebertt January 19, 2024 08:50
@timebertt timebertt self-assigned this Jan 19, 2024
Copy link
Member

@timebertt timebertt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!
/lgtm

@timebertt timebertt changed the title Allow overriding of CreationTimeout Allow overriding of creation timeout for LoadBalancerMachines Jan 19, 2024
@timebertt timebertt merged commit 553b52a into main Jan 19, 2024
1 check passed
@timebertt timebertt deleted the feature/customize-lbm-deletion branch January 19, 2024 09:08
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.

3 participants