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

Add functionality to configure mutators #230

Closed
max-koehler opened this issue Aug 22, 2014 · 9 comments
Closed

Add functionality to configure mutators #230

max-koehler opened this issue Aug 22, 2014 · 9 comments

Comments

@max-koehler
Copy link
Contributor

If i see it right, currently we can't create configurations for mutators with the puppet modul.
Checks, config, handlers, subscriptions, ... everything can be configured except the mutators.

Is there any possibility to configure it with puppet?
@jamtur01
Copy link
Contributor

You can configure it via the handler type - https://github.com/sensu/sensu-puppet/blob/master/lib/puppet/type/sensu_handler.rb#L67. Thanks!

@max-koehler
Copy link
Contributor Author

Ok I thought this field is used to reference the mutator. 

If a hash could be used to configure the actual mutator within the handler than this will result in duplicated configurations of the same. 

Or am I wrong

@aamerik
Copy link

aamerik commented Nov 6, 2014

@jamtur01 could you please clarify "You can configure it via the handler type - https://github.com/sensu/sensu-puppet/blob/master/lib/puppet/type/sensu_handler.rb#L67. Thanks!"

@confiq
Copy link
Contributor

confiq commented Feb 24, 2015

You can configure it via the handler type - https://github.com/sensu/sensu-puppet/blob/master/lib/puppet/type/sensu_handler.rb#L67. Thanks!

@jamtur01 : I think he was referring to configuring mutator and not a handler (ex: http://sensuapp.org/docs/0.16/mutators).
With sensu_handler you are able to configure only the handler but not the mutator. This issue should be open.

CC: @max-koehler @aamerik

@max-koehler
Copy link
Contributor Author

👍

@jamtur01 jamtur01 reopened this Feb 24, 2015
@jamtur01
Copy link
Contributor

PR welcomed. Thanks!

@confiq
Copy link
Contributor

confiq commented Feb 25, 2015

I'm not sure if by PR you meant Public Relationship in this case..../s

If nobody started to work on this issue you might assign it to me and I'll try to make PR.
It should not be hard to develop it.

@jlambert121
Copy link
Contributor

PR meaning Pull Request. With the github model if someone forks the repo, makes changes/improvements/fixes they can send those changes back to the upstream repo through a Pull Request.

gsalisbury added a commit to gsalisbury/sensu-puppet that referenced this issue Oct 18, 2015
@gsalisbury
Add functionality to configure mutators sensu#230
e61469b
gsalisbury added a commit to gsalisbury/sensu-puppet that referenced this issue Oct 21, 2015
@gsalisbury
Add functionality to configure mutators sensu#230
9d754eb
jlambert121 added a commit that referenced this issue Oct 22, 2015
@jlambert121
Merge pull request #435 from gsalisbury/add_mutator_provider_230
6cf20fe 
Add functionality to configure mutators #230
Chealion pushed a commit to cybera/sensu-puppet that referenced this issue Oct 29, 2015
Merge branch 'master' of https://github.com/sensu/sensu-puppet
8cb2654 
* 'master' of https://github.com/sensu/sensu-puppet: (48 commits)
  Added subdue attribute to sensu_check type
  Adding option to manage the mutators dir
  add ttl to check provider
  Add functionality to configure mutators sensu#230
  update supported puppet versions
  Adding spec tests for manage_handlers param
  Adding option to manage the handlers dir
  set $pkg_requires to undef for the redhat osfamily as with it not set the run fails on centos/redhat when strict_variables is enabled
  version bump: 2.0.0
  set sysconfig parameters when defined
  allow setting of path
  Add require on apt::update for puppetlabs-apt 2.x
  Correcting issue sensu#318 - Changing the default value of sensu_plugin_version from absent to installed
  allow handle and handlers together
  Make the sensu enterprise dashboard not show up unbidden
  Move file_ensure out of conditional
  Add tests for sensu-plugins provider
  update apt module dep
  fix db type
  Fix Sensu Enterprise services when not using enterprise
  ...
salimane added a commit to MeltwaterArchive/sensu-puppet that referenced this issue Nov 2, 2015
@salimane
Merge remote-tracking branch 'upstream/master'
945467d 
* upstream/master: (286 commits)
  Added subdue attribute to sensu_check type
  Adding option to manage the mutators dir
  add ttl to check provider
  Add functionality to configure mutators sensu#230
  update supported puppet versions
  Adding spec tests for manage_handlers param
  Adding option to manage the handlers dir
  set $pkg_requires to undef for the redhat osfamily as with it not set the run fails on centos/redhat when strict_variables is enabled
  version bump: 2.0.0
  set sysconfig parameters when defined
  allow setting of path
  Add require on apt::update for puppetlabs-apt 2.x
  Correcting issue sensu#318 - Changing the default value of sensu_plugin_version from absent to installed
  allow handle and handlers together
  Make the sensu enterprise dashboard not show up unbidden
  Move file_ensure out of conditional
  Add tests for sensu-plugins provider
  update apt module dep
  fix db type
  Fix Sensu Enterprise services when not using enterprise
  ...
thanandorn pushed a commit to thanandorn/sensu-puppet that referenced this issue Dec 6, 2015
@gsalisbury
Add functionality to configure mutators sensu#230
09306c3
@jaxxstorm
Copy link
Contributor

I believe this is merged and resolved, so closing. Please reopen if required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@jamtur01 @max-koehler @confiq @aamerik @jlambert121 @jaxxstorm