Skip to content

Conversation

salami738
Copy link

Hi, i've added two new node pool settings: drain_timeout_in_minutes and node_soak_duration_in_minutes

@salami738
Copy link
Author

Please wait with the merge. I would like to test this locally. I'll let you know here as soon as this is done.

@salami738
Copy link
Author

I tested the new code locally. All green.

@timdeluxe timdeluxe requested review from dploeger and timdeluxe March 6, 2025 12:11
Copy link
Member

@timdeluxe timdeluxe left a comment

Choose a reason for hiding this comment

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

Thanks 🎁
Will wait for an additional approval by @dploeger and then i can merge and release.

Copy link
Member

@dploeger dploeger left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks! 🎁

@timdeluxe timdeluxe merged commit 329451d into dodevops:main Mar 6, 2025
2 checks passed
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