add timer support about puppet-systemd HOT 6 CLOSED

Hi,
this already works fine:
https://github.com/voxpupuli/puppet-borg/blob/f2c66b51c46d57b25326524415c0e9ced683a469/manifests/service.pp#L11-L26

https://github.com/voxpupuli/puppet-borg/blob/f2c66b51c46d57b25326524415c0e9ced683a469/templates/borg-backup.timer.epp#L1-L13

from puppet-systemd.

@bastelfreak I agree that it works, but it's confusing for end users. It would be nice to have a defined type or something that just wraps it to be more clear.

from puppet-systemd.

I was just contemplating writing a systemd::timer type , will try something.

from puppet-systemd.

The only interface that makes sense would be

systemd::timer{'runoften.timer':
    timer_content   => ...
    service_content =>
    enable => true,
    active => true,
}

Essentially to specify the content of the service and timer unit.

It does not make sense to specify the individual say OnCalandar arguments since this gets tricky because you then have to define the Before, , After, multiple OnCalendar, ...

The alternative is to only support really trivial jobs. Such as

systemd::timer{'myjob':
  exec => 'rm -rf /',
  on_calendar => 'daily',
}

along with some other timer options. Anything more complicated would require to resort back to two systemd::unit_files as per the above examples.

So is the latter one still worth while.

from puppet-systemd.

@traylenator I think that it is. I also think that a wrapper that creates the rest of the bits for a generally useful OnCalendar example would be useful but could be a later addition.

from puppet-systemd.

Implemented in #138

from puppet-systemd.