Skip to content

Conversation

@sbernauer
Copy link
Member

@sbernauer sbernauer commented Oct 9, 2023

Description

Part of stackabletech/issues#357

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes
# Author - [ ] Changes are OpenShift compatible - [ ] CRD changes approved - [ ] Integration tests passed (for non trivial changes) 
# Reviewer - [x] Code contains useful comments - [ ] (Integration-)Test cases added - [x] Documentation added or updated - [x] Changelog updated - [x] Cargo.toml only contains references to git tags (not specific commits or branches) 
# Acceptance - [ ] Feature Tracker has been updated - [ ] Proper release label has been added 
@sbernauer sbernauer changed the title feat: Add gracefulShutdownTimeout to GenericRoleConfig with a default of 15 minutes feat: Add gracefulShutdownTimeout to GenericRoleConfig Oct 9, 2023
Copy link
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

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

LGTM!


/// Time period the Pods of this role have to gracefully shut down, e.g. `1h`, `30m` or `2d`.
/// Consult the individual operator documentation for details on how the graceful shutdown
/// mechanism works.
Copy link
Member

Choose a reason for hiding this comment

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

Maybe mention the default value here.

@sbernauer
Copy link
Member Author

Not needed any more, as we track gracefulShutdownTimeout in the product specific config now

@sbernauer sbernauer closed this Oct 12, 2023
@sbernauer sbernauer deleted the feat/add-graceful-shutdown branch October 27, 2023 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants