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.

Sign up for GitHub

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

Jump to bottom

sensu::check resources should support cron scheduling #737

Closed
cwjohnston opened this issue Jul 12, 2017 · 1 comment
Closed

sensu::check resources should support cron scheduling #737

cwjohnston opened this issue Jul 12, 2017 · 1 comment
Assignees
@jeffmccune

Comments

@cwjohnston
Copy link
Contributor

Description of problem

In Sensu 0.28 the cron attribute was added to the check definition specification as an alternative to interval scheduling.

When present, the value of this attribute supersedes the interval attribute.

When cron is configured with a value, interval should not be required nor present in the configuration written to disk.
@jeffmccune jeffmccune self-assigned this Jul 13, 2017
jeffmccune added a commit to jeffmccune/sensu-puppet that referenced this issue Jul 13, 2017
@jeffmccune
(sensu#737) Add cron attribute to sensu::check type
3fe3b18 
Without this patch the sensu::check type does not manage the
[cron](https://sensuapp.org/docs/latest/reference/checks#check-attributes)
attribute.  This patch adds the `cron` parameter.  If specified, the interval
parameter will be set to `'absent'` which will remove the interval attribute
from the JSON check configuration if present.

Resolves sensu#737
jeffmccune added a commit to jeffmccune/sensu-puppet that referenced this issue Jul 13, 2017
@jeffmccune
(sensu#737) Add cron attribute to sensu::check type
3c07549 
Without this patch the sensu::check type does not manage the
[cron](https://sensuapp.org/docs/latest/reference/checks#check-attributes)
attribute.  This patch adds the `cron` parameter.  If specified, the interval
parameter will be set to `'absent'` which will remove the interval attribute
from the JSON check configuration if present.

Resolves sensu#737
@jeffmccune jeffmccune mentioned this issue Jul 13, 2017
Merged
ghoneycutt added a commit that referenced this issue Jul 13, 2017
@ghoneycutt
Merge pull request #743 from jeffmccune/737_sensu_check_cron
7cf9e3a 
(#737) Add cron attribute to sensu::check type
@ghoneycutt
Copy link
Collaborator

Implemented in v2.23.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeffmccune jeffmccune

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jeffmccune @cwjohnston @ghoneycutt