-
-
Notifications
You must be signed in to change notification settings - Fork 146
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
Add loginctl_linger_users fact #410
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am willing to let it pass with the tests written the way they are, but i would suggest we find someone with better rspec knowledge to go over it too before we merge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally not in favor of adding facts like this, particularly on widely used modules. A function or resource may be more appropriate.
That said, what's an sample use case?
My overall plan is to feed this value back into
So my users with Pat |
I'm confused, isn't that redundant if touching a /var/lib/systemd/linger/ file is the same as adding to KillExcludeUsers in the logind conf file? |
Maybe a slight difference: I guess enable linger will stop things being killed that are running in the I'm kind of worried about this fact as it would be pretty large in our case. |
If this fact looks to be too much, I can always run it out of one of my private modules. Just figured I'd default to "share with the community". |
Agree. My basic objection to a fact like this. |
No worries, I'll close this out and run the fact privately. |
I also have some "private" facts, mostly because they are not related to a specific module and very site-specific or just not needed by the module to operate and only provide filtering help for tools like choria (e.g. I wonder if it would make sense to have a section of the voxpupuli website with such custom facts that are listed with a short descriptions on a page in a similar way we have PQL queries? |
If I was going to do this and am actually thinking about it I would not rely on puppet to maintain that list. I'd set up a path unit and service and sync |
Pull Request (PR) description
Add a fact listing users with linger enabled.
As near as I can tell from the systemd source,
root
is always able to linger even it not setup explicitly.This Pull Request (PR) fixes the following issues