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.

Sign up for GitHub

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

Jump to bottom

GracefulNodeShutdown configuration file /etc/systemd/logind.conf.d/99-kubelet.conf is marked executable. #97949

Closed
wzshiming opened this issue Jan 12, 2021 · 2 comments · Fixed by #97950
Closed

GracefulNodeShutdown configuration file /etc/systemd/logind.conf.d/99-kubelet.conf is marked executable. #97949

wzshiming opened this issue Jan 12, 2021 · 2 comments · Fixed by #97950
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/backlog Higher priority than priority/awaiting-more-evidence. sig/node Categorizes an issue or PR as relevant to SIG Node. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@wzshiming
Copy link
Member

wzshiming commented Jan 12, 2021 •
edited

What happened:

image

kubernetes/pkg/kubelet/nodeshutdown/systemd/inhibit_linux.go

Line 181 in 9caf675

if err := ioutil.WriteFile(logindOverridePath, []byte(inhibitOverride), 0755); err != nil {

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

Environment:

  • Kubernetes version (use kubectl version): v1.20.1
  • Cloud provider or hardware configuration:
  • OS (e.g: cat /etc/os-release): 7
  • Kernel (e.g. uname -a): Linux zsm-test-2 3.10.0-1160.11.1.el7.x86_64 Unit test coverage in Kubelet is lousy. (~30%) #1 SMP Fri Dec 18 16:34:56 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
  • Install tools:
  • Network plugin and version (if this is a network-related bug):
  • Others:
@wzshiming wzshiming added the kind/bug Categorizes issue or PR as related to a bug. label Jan 12, 2021
@k8s-ci-robot k8s-ci-robot added needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Jan 12, 2021
@wzshiming
Copy link
Member Author

/sig node

@k8s-ci-robot k8s-ci-robot added sig/node Categorizes an issue or PR as relevant to SIG Node. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Jan 12, 2021
@wzshiming wzshiming mentioned this issue Jan 12, 2021
Merged
@ehashman
Copy link
Member

/triage accepted
/priority backlog

@k8s-ci-robot k8s-ci-robot added triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/backlog Higher priority than priority/awaiting-more-evidence. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/backlog Higher priority than priority/awaiting-more-evidence. sig/node Categorizes an issue or PR as relevant to SIG Node. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

GracefulNodeShutdown configuration remove executable permission bits wzshiming/kubernetes
3 participants
@ehashman @wzshiming @k8s-ci-robot