-
-
Notifications
You must be signed in to change notification settings - Fork 236
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
add timeout_stop
to control systemd TimeoutStopSec
#499
Conversation
e4e4231
to
b2fab21
Compare
With larger WAL segments, prometheus fails to write a new checkpoint in `TimeoutStopSec` time. This change adds a new parameter to define `TimeoutStopSec` for `prometheus.service`.
b2fab21
to
9a1a206
Compare
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.
🙋🏻♀️
@@ -397,6 +397,16 @@ Currently only implemented for systemd based service. | |||
|
|||
Default value: ``undef`` | |||
|
|||
##### `timeout_stop` | |||
|
|||
Data type: `Optional[String]` |
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.
i feel like we can do better than this
but that better type would either belong here:
https://github.com/camptocamp/puppet-systemd/tree/master/types
unless a general one already exists here: https://github.com/puppetlabs/puppetlabs-stdlib/tree/main/types
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.
I totally agree that "String" is very ... liberal - and I checked both modules before.
Let's see if there's nice regex to be drafted in a puppet type before merging.
@antondollmaier can you please rebase against our latest master branch? |
@antondollmaier ping :) |
Dear @antondollmaier, thanks for the PR! This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
I have opened a new issue to change the systemd configuration, which will allow customization like this without adding new parameters. #734 |
Pull Request (PR) description
With larger WAL segments, prometheus fails to write
a new checkpoint in
TimeoutStopSec
time:This change adds a new parameter to define
TimeoutStopSec
for
prometheus.service
.This Pull Request (PR) fixes the following issues
No issue previously created.
The required systemd unitfile overwrite could have been made locally as well - but as I suppose that others could also face similar issues, I decided to implement the parameter directly.