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.

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

ScientificLinux: service.enable not mapped correctly #8338

Closed
sabaini opened this issue Nov 7, 2013 · 2 comments
Closed

ScientificLinux: service.enable not mapped correctly #8338

sabaini opened this issue Nov 7, 2013 · 2 comments

Comments

@sabaini
Copy link
Contributor

sabaini commented Nov 7, 2013

Symptom: on a ScientificLinux 6.3 VM the service.enable function does nothing and returns a comment "Enable is not available on this minion, service httpd is in the desired state".

This patch fixes it for me --

diff --git a/salt/modules/rh_service.py b/salt/modules/rh_service.py
index 44a6b36..3812063 100644
--- a/salt/modules/rh_service.py
+++ b/salt/modules/rh_service.py
@@ -44,7 +44,7 @@ def __virtual__():
     enable = set((
         'RedHat',
         'CentOS',
-        'Scientific',
+        'ScientificLinux',
         'CloudLinux',
         'Amazon',
         'Fedora',

The issue should be reproducible with Salt 0.17.1 and SL 6.3 on master and minion, and this call:

salt-call state.low '{ "state": "service", "fun": "enabled", "name": "httpd" }'
@sabaini
Copy link
Contributor Author

sabaini commented Nov 8, 2013

pull request: #8345

@basepi
Copy link
Contributor

basepi commented Nov 9, 2013

Closing, as it appears the above pull request fixes the issue! Thanks for the patch!

@basepi basepi closed this as completed Nov 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants