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

Improve static pod probing #3204

Merged
merged 2 commits into from
Aug 3, 2022
Merged

Conversation

brandond
Copy link
Member

@brandond brandond commented Aug 2, 2022

Proposed Changes

Improve static pod probing

  • Add Startup/Readiness probes
  • Sync default probe thresholds and timing with kubadm
  • Make probe thresholds and timing configurable from the CLI, using an interface similar to control-plane resources and requests:
    --control-plane-probe-configuration=etcd-startup-initial-delay-seconds=42,kube-apiserver-readiness-period-seconds=15

Types of Changes

Enhancement

Verification

See linked issue

Linked Issues

#3104

User-Facing Change

RKE2 static pods now include readiness, liveness, and startup probes with defaults that match those configured by kubeadm.
RKE2 static pod probe timings and thresholds can be customized with the `--control-plane-probe-configuration` flag.

Further Comments

@brandond brandond requested a review from a team as a code owner August 2, 2022 00:30
@codecov-commenter

This comment was marked as outdated.

@brandond brandond force-pushed the static_pod_readiness branch 2 times, most recently from 93ea56f to b79814e Compare August 2, 2022 22:16
@brandond brandond changed the title Add Startup/Readiness probes; sync thresholds and timing with kubadm Improve static pod probing Aug 2, 2022
pkg/rke2/rke2_linux.go Show resolved Hide resolved
@brandond brandond requested a review from dereknola August 3, 2022 09:15
@brandond brandond force-pushed the static_pod_readiness branch 2 times, most recently from aae4f00 to b1753a2 Compare August 3, 2022 09:42
pkg/rke2/rke2_linux_test.go Outdated Show resolved Hide resolved
@brandond brandond force-pushed the static_pod_readiness branch from b1753a2 to b365276 Compare August 3, 2022 17:57
@brandond brandond force-pushed the static_pod_readiness branch 2 times, most recently from 1aca096 to af82e7b Compare August 3, 2022 18:02
@brandond brandond requested a review from a team August 3, 2022 20:11
@brandond brandond force-pushed the static_pod_readiness branch from af82e7b to 3b78f3c Compare August 3, 2022 20:42
* Add Startup/Readiness probes
* Sync default probne thresholds and timing with kubadm
* Make probe thresholds and timing configurable from the CLI,
  using an interface similar to control-plane resources and requests

Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
@brandond brandond merged commit 2b59f33 into rancher:master Aug 3, 2022
@brandond brandond deleted the static_pod_readiness branch June 6, 2024 23:11
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.

4 participants