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

Fixing dependencies parameter on sensu_check type #240

Merged
merged 1 commit into from
Sep 11, 2014
Merged

Fixing dependencies parameter on sensu_check type #240

merged 1 commit into from
Sep 11, 2014

Conversation

ttmgraham
Copy link
Contributor

It appears that currently the sensu_check dependencies parameter is treating a given array as a string.

With the following configuration

sensu::check { 'mycheck':
        command      => '/etc/sensu/plugins/mycommand',
        interval     => 120,
        standalone   => false,
        subscribers  => ['mygroup'],
        dependencies => ['my-first-check', 'my-second-check']
}

Will produce a configuration file which dependencies section looks like:

"dependencies": [
  "my-first-test"
],

Adding this change seems to fix the issue. I am fairly new to this so if there is a better way to fix this please let me know.

jamtur01 added a commit that referenced this pull request Sep 11, 2014
Fixing dependencies parameter on sensu_check type
@jamtur01 jamtur01 merged commit f16bfaf into sensu:master Sep 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants