Skip to content

Commit

Permalink
Merge pull request #81 from mshitrit/remove_hostPID
Browse files Browse the repository at this point in the history
remove hostPID
  • Loading branch information
openshift-merge-robot authored Sep 26, 2023
2 parents 2750538 + c50daf4 commit eea79f7
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -434,7 +434,7 @@ spec:
- email: [email protected]
name: Medik8s Team
maturity: alpha
minKubeVersion: 1.20.0
minKubeVersion: 1.25.0
provider:
name: Medik8s
url: https://www.medik8s.io/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ spec:
- email: [email protected]
name: Medik8s Team
maturity: alpha
minKubeVersion: 1.20.0
minKubeVersion: 1.25.0
provider:
name: Medik8s
url: https://www.medik8s.io/
Expand Down
2 changes: 0 additions & 2 deletions install/self-node-remediation-deamonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ spec:
type: Directory
serviceAccountName: self-node-remediation-controller-manager
priorityClassName: system-node-critical
hostPID: true
containers:
- args:
- --is-manager=false
Expand Down Expand Up @@ -72,7 +71,6 @@ spec:
mountPath: /dev
securityContext:
privileged: true
hostPID: true
name: manager
ports:
- containerPort: {{.HostPort}}
Expand Down
2 changes: 1 addition & 1 deletion pkg/reboot/rebooter.go
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ func (r *watchdogRebooter) Reboot() error {
// softwareReboot performs software reboot by running systemctl reboot
func (r *watchdogRebooter) softwareReboot() error {
r.log.Info("about to try software reboot")
// hostPID: true and privileged:true required to run this
// privileged:true required to run this
rebootCmd := exec.Command("/usr/bin/nsenter", "-m/proc/1/ns/mnt", "/bin/bash", "-c", "echo b > /proc/sysrq-trigger")

if err := rebootCmd.Run(); err != nil {
Expand Down

0 comments on commit eea79f7

Please sign in to comment.