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

Mount sentinel-location without sentinel-command #49

Closed
wants to merge 7 commits into from

Conversation

ckotzbauer
Copy link
Member

Wait with merge until kubereboot/kured#813 is released.

Signed-off-by: Christian Kotzbauer <[email protected]>
Signed-off-by: Christian Kotzbauer <[email protected]>
Signed-off-by: Christian Kotzbauer <[email protected]>
Signed-off-by: Christian Kotzbauer <[email protected]>
Signed-off-by: Christian Kotzbauer <[email protected]>
@rptaylor
Copy link

rptaylor commented Aug 11, 2023

This works in my testing in conjunction with kubereboot/kured#813
My values:

image:
  repository: ghcr.io/ckotzbauer/kured
  tag: 1.14.0-alpha.0
configuration:
  period: "0h0m30s"
  rebootDelay: 0h1m0s
  rebootSentinel: /var/lib/example/reboot-required
extraArgs:
  reboot-method: signal
containerSecurityContext:
  readOnlyRootFilesystem: true
  privileged: false
  allowPrivilegeEscalation: false
  capabilities:
    drop: ["*"]
    add: ["CAP_KILL"]

Then I confirmed that the kured pods were deployed with:

  containers:
  - args:
    - --ds-name=kured
    - --ds-namespace=kured
    - --metrics-port=8080
    - --period=0h0m30s
    - --reboot-sentinel=/sentinel/reboot-required
    - --reboot-command=/bin/systemctl reboot
    - --reboot-delay=0h1m0s
    - --log-format=text
    - --reboot-method=signal

  volumeMounts:
    - mountPath: /sentinel
      name: sentinel
      readOnly: true

  - hostPath:
      path: /var/lib/example
      type: ""
    name: sentinel

Then on a node I touch /var/lib/example/reboot-required and the node reboooted.

charts/kured/README.md Outdated Show resolved Hide resolved
Signed-off-by: Christian Kotzbauer <[email protected]>
@ckotzbauer
Copy link
Member Author

Thanks @rptaylor for review and testing, I adjusted the PR.

@ckotzbauer ckotzbauer added this to the 1.15.0 milestone Aug 12, 2023
@ckotzbauer ckotzbauer mentioned this pull request Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants