-
-
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
(doc) Adds examples of running the service created #29
(doc) Adds examples of running the service created #29
Conversation
petems
commented
Jul 5, 2017
- Closes Add explicit ordering to README.md #24
README.md
Outdated
@@ -19,6 +19,9 @@ Let this module handle file creation and systemd reloading. | |||
::systemd::unit_file { 'foo.service': | |||
source => "puppet:///modules/${module_name}/foo.service", | |||
} | |||
~> service {'foo' |
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.
there is a : missing?
README.md
Outdated
@@ -33,6 +36,11 @@ file { '/usr/lib/systemd/system/foo.service': | |||
source => "puppet:///modules/${module_name}/foo.service", | |||
} ~> | |||
Exec['systemctl-daemon-reload'] | |||
|
|||
service {'foo' |
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.
here as well?
Hi @petems, thanks for the patch. Can you take a look at the comments? By the way, what is in your opinion better? include ::systemd or include "::systemd" |
@petems ping :) |
@petems could you address @bastelfreak's comments please? |
3a1e3a1
to
318bcc8
Compare
318bcc8
to
5765ab6
Compare
@raphink @bastelfreak Done! 👍 |
thanks! |
Comma separated values