-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
support amazon linux 2 for service module #45764
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 have one small request
salt/modules/rh_service.py
Outdated
else: | ||
return ( | ||
False, | ||
'Amazon Linux >= version 2 use systemd, will not ' |
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.
use
--> uses
and then maybe let's split the message up into 2 sentences with systemd. Will not
.
Edited the copy/pasted bits! |
@rallytime I think this should be added to the RC branch. |
Back-port #45764 to oxygen.rc1
What does this PR do?
Same as #45758, but for the 2017.7 branch. Disables rh_service.py on Amazon Linux 2
Previous Behavior
service.running states ran via rh_service.py rather than systemd and failed.
New Behavior
Systemd is used to manage service calls and runs properly.
Tests written?
No
Commits signed with GPG?
No