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

feat: add a cron like wrapper for timers #419

Merged
merged 3 commits into from
Mar 4, 2024

Conversation

TheMeier
Copy link
Contributor

@TheMeier TheMeier commented Mar 2, 2024

Pull Request (PR) description

This helper will allow to define a timer in a cron like way, and will setup a corresponding timer and service.
In basic usage it only requires a command, user, timerspec and an ensure parameter.
For additional feaures the parameter service_overrides, timer_overrides, service_unit_overrides and timer_unit_overrides can be used to define additional aspects of the create units.
The correct order of the units is also ensured.

This Pull Request (PR) fixes the following issues

Fixes #374
Related to #287

@TheMeier TheMeier added the enhancement New feature or request label Mar 2, 2024
@TheMeier TheMeier requested review from traylenator and ekohl March 2, 2024 20:56
types/unit/timer/timerspec.pp Outdated Show resolved Hide resolved
manifests/timer_wrapper.pp Outdated Show resolved Hide resolved
manifests/timer_wrapper.pp Show resolved Hide resolved
@TheMeier TheMeier force-pushed the issues/374 branch 2 times, most recently from c24e1a4 to 4c5ad0d Compare March 3, 2024 08:52
@TheMeier TheMeier requested a review from kenyon March 3, 2024 08:53
manifests/timer_wrapper.pp Outdated Show resolved Hide resolved
manifests/timer_wrapper.pp Show resolved Hide resolved
manifests/timer_wrapper.pp Show resolved Hide resolved
manifests/timer_wrapper.pp Outdated Show resolved Hide resolved
manifests/timer_wrapper.pp Show resolved Hide resolved
manifests/timer_wrapper.pp Outdated Show resolved Hide resolved
manifests/timer_wrapper.pp Show resolved Hide resolved
spec/defines/timer_wrapper_spec.rb Outdated Show resolved Hide resolved
types/unit/timer/timerspec.pp Outdated Show resolved Hide resolved
@TheMeier TheMeier force-pushed the issues/374 branch 8 times, most recently from 8b92ffd to 206bd65 Compare March 3, 2024 19:25
@TheMeier TheMeier requested a review from traylenator March 3, 2024 19:27
This helper will allow to define a timer in a cron like way, and will
setup a corresponding timer and service.
In basic usage it only requires a command, user, timerspec and an ensure
parameter.
For additional features the parameter `service_overrides`, `timer_overrides`,
`service_unit_overrides` and `timer_unit_overrides` can be used to define
additional aspects of the create units.
The correct order of the units is also ensured.

fixes voxpupuli#374
related to voxpupuli#287
Copy link
Contributor

@traylenator traylenator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Useful.

types/unit/timespan.pp Outdated Show resolved Hide resolved
types/unit/timespan.pp Outdated Show resolved Hide resolved
TheMeier and others added 2 commits March 4, 2024 18:47
Co-authored-by: Steve Traylen <steve.traylen@cern.ch>
Co-authored-by: Steve Traylen <steve.traylen@cern.ch>
@TheMeier TheMeier merged commit d7a0907 into voxpupuli:master Mar 4, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add a cron-like systemd::timer interface
3 participants