This module is not indempotent about puppet-systemd HOT 12 CLOSED

Hi @raphink. Let me know if you want to transfer it. You currently have the github permissions to do it.

from puppet-systemd.

Pretty sure this is resolved. Is anyone still having issues with this?

from puppet-systemd.

as discussed on freenode/#puppet with @roidelapluie the proposed change is to replace the Exec[daemon-reload] with a defined type. Now #2 is an open discussion on wether to replace the Exec with a Class.

So I guess this boils down to: should we use a Class or a Define?
I'll update the discussion in #2

from puppet-systemd.

https://gist.github.com/roidelapluie/069223753a576983de39

from puppet-systemd.

@raphink @mcanevet @mfournier can we have your input?

from puppet-systemd.

Is this and #2 still being worked on? There was some interest from puppet/r10k in using this module but this issue came up as a potential issue.

from puppet-systemd.

ping @raphink @mcanevet. Do you have any objections on this? Let us know if there isn't any active work on this module. If so, we could transfer it to Vox Pupuli?

from puppet-systemd.

No work ATM on my side

from puppet-systemd.

I would like to know if this module is still maintained or if it should be migrated to Vox Pupuli as suggested by @bastelfreak

from puppet-systemd.

We actually only use the main class of this module, which is why we're not very reactive on adding features to it. Transfering to Vox Pupuli is probably a good option.

from puppet-systemd.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

from puppet-systemd.

I believe this is a non-issue, File[foo.service] should still notify Exec['systemctl-daemon-reload'] even if other resources, which also notify the Exec, fail. If that isn't the case, than it is a bug in the puppet core.

from puppet-systemd.