Skip to content

[Merged by Bors] - Add startup probe and fine-tune values in liveness and readiness probes#193

Closed
siegfriedweber wants to merge 1 commit intomainfrom
adapt_probes
Closed

[Merged by Bors] - Add startup probe and fine-tune values in liveness and readiness probes#193
siegfriedweber wants to merge 1 commit intomainfrom
adapt_probes

Conversation

@siegfriedweber
Copy link
Copy Markdown
Member

@siegfriedweber siegfriedweber commented Jun 21, 2022

Description

Add startup probe and fine-tune values in liveness and readiness probes

Integration tests

  • K3s on Hetzner Cloud, v1.23
    Build Status

  • Microsoft Azure AKS, 1.24.0
    Build Status

  • IONOS managed K8s, 1.23.8
    Build Status

Review Checklist

  • Code contains useful comments
  • (Integration-)Test cases added (or not applicable)
  • Documentation added (or not applicable)
  • Changelog updated (or not applicable)
  • Cargo.toml only contains references to git tags (not specific commits or branches)
  • Helm chart can be installed and deployed operator works (or not applicable)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

@siegfriedweber siegfriedweber self-assigned this Jun 21, 2022
Copy link
Copy Markdown
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

LGTM
It would make sense to add a startup-probe for exactly this problem but fine for me for now

@siegfriedweber siegfriedweber force-pushed the adapt_probes branch 2 times, most recently from 1329b62 to 12f99e8 Compare July 5, 2022 12:15
@siegfriedweber siegfriedweber requested a review from sbernauer July 5, 2022 12:16
Copy link
Copy Markdown
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

Don't forget to remove the [patch."https://github.com/stackabletech/operator-rs.git"]

@siegfriedweber siegfriedweber changed the title Increase failure thresholds in probes Add startup probe and fine-tune values in liveness and readiness probes Jul 5, 2022
@siegfriedweber siegfriedweber marked this pull request as ready for review July 5, 2022 15:24
@siegfriedweber
Copy link
Copy Markdown
Member Author

bors merge

bors bot pushed a commit that referenced this pull request Jul 6, 2022
…es (#193)

# Description

Add startup probe and fine-tune values in liveness and readiness probes
@bors
Copy link
Copy Markdown
Contributor

bors bot commented Jul 6, 2022

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title Add startup probe and fine-tune values in liveness and readiness probes [Merged by Bors] - Add startup probe and fine-tune values in liveness and readiness probes Jul 6, 2022
@bors bors bot closed this Jul 6, 2022
@bors bors bot deleted the adapt_probes branch July 6, 2022 10:51
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.

2 participants