-
-
Notifications
You must be signed in to change notification settings - Fork 503
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
Fixes for Archlinux and modulesync 1.8 #685
Conversation
@@ -254,6 +254,27 @@ | |||
notify => Class['Rabbitmq::Service'], | |||
} | |||
} | |||
'Archlinux': { |
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.
this isn't the cleanest solution, but it works good. Later on we can think about unifying this setup for all distributions with systemd. Personally I prefer the camptocamp/systemd module for this, but maybe we need to wait for the discussion in voxpupuli/puppet-prometheus#90
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.
Since this isn't part of the modulesync or 'spec test fixes', does it belong in a separate PR (so that is makes it into the changelog)?
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.
that's why I didn't label it with modulesync. It will be in the changelog.
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.
To stop anyone else labelling it later, I've renamed the PR to make it a bit more obvious.
No description provided.