Set kernel.softlockup_panic =1 based on the flag. #38001
Conversation
k8s-ci-robot
added
the
cncf-cla: yes
|
This change is |
|
I created the cluster and checked the configs on the node: |
|
cc/ @saad-ali |
k8s-github-robot
added
size/S
| @@ -261,6 +261,20 @@ net.ipv4.ip_forward: | |||
| sysctl.present: | |||
| - value: 1 | |||
|
|
|||
| {% if pillar.get('softlockup_panic', '').lower() == 'true' %} | |||
There was a problem hiding this comment.
nit: Why add this to docker salt scripts?
There was a problem hiding this comment.
Another place is kubelet salt scripts. It doesn't matter at kubelet or docker salt script, it just need to run on every CVM node.
There was a problem hiding this comment.
Also this is a workaround to mitigate the issue until we build a new image.
|
Jenkins kops AWS e2e failed for commit 38a63e3. Full PR test history. The magic incantation to run this job again is |
|
/lgtm |
k8s-ci-robot
added
the
lgtm
k8s-github-robot
added
the
do-not-merge
saad-ali
added
release-note
|
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
|
Automatic merge from submit-queue |
saad-ali
added
the
cherry-pick-approved
|
Commit found in the "release-1.5" branch appears to be this PR. Removing the "cherrypick-candidate" label. If this is an error find help to get your PR picked. |
ref: #37853