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
Allow the actual inhibit delay to be greater than the expected inhibit delay #103137
Allow the actual inhibit delay to be greater than the expected inhibit delay #103137
Conversation
/cc @bobbypage |
f1993e9
to
97bcfbd
Compare
This change makes sense, but I don't think it will fix #102818 If I understood the issue in #102818 it was that ubuntu installs a Kubelet also writes a config to The systemd docs (https://www.freedesktop.org/software/systemd/man/logind.conf.html) note:
So based on that it looks like the kubelet file should take precedence? Perhaps it might be issue with the lexicographic ordering... |
Yes, I also think this is a lexicographic ordering issue. But it does not make sense to preempt and eventually overwrite other configurations here. If there are other applications that rely on suppression locks but require more time, I think they should be allowed to exist. |
I agree, that makes sense. However in the ubuntu case, if the config file for In other words, I don't think kubelet should ever decrease the overall |
I agree. This PR is done in this way |
Yup, agree! The only thing I'm not clear on is in PR description:
As we discussed, it seems like #102818 is an unrelated issue (i.e. possibly due to lexicographic sorting of logind override files). Is that correct? |
Thanks! LGTM for this change Can we discuss further the logind issue on the corresponding issue #102818? I'm still unclear if issue is lexicographic sorting of files or something else... |
/assign @mrunalp |
/priority important-soon |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mrunalp, wzshiming The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind bug
/sig node
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: