Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: sensu/sensu-puppet
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v2.23.0
Choose a base ref
...
head repository: sensu/sensu-puppet
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v2.24.0
Choose a head ref
  • 5 commits
  • 11 files changed
  • 3 contributors

Commits on Jul 13, 2017

  1. (#598) Improve rabbitmq clustering robustness 

    Without this patch, Puppet errors out when both `sensu::rabbitmq_password`, or
any of the `sensu::rabbitmq_*` parameters are specified along with
`sensu::rabbitmq_cluster`.  The error is:

    Error: /Stage[main]/Sensu::Rabbitmq::Config/Sensu_rabbitmq_config[sensu-server.example.com]: Could not evaluate: no implicit conversion of String into Integer
    /vagrant/lib/puppet/provider/sensu_rabbitmq_config/json.rb:137:in `[]'
    /vagrant/lib/puppet/provider/sensu_rabbitmq_config/json.rb:137:in `password'

This patch addresses the problem by changing sensu::rabbitmq::config to ignore
sensu::rabbitmq_* class parameters when the cluster configuration is specified
via `sensu::rabbitmq_cluster`.

Resolves #598
    @jeffmccune
    jeffmccune committed Jul 13, 2017
    Configuration menu
    Copy the full SHA
    516e798 View commit details
    Browse the repository at this point in the history

  2. (#727) Add sensu::spawn_limit class parameter 

    Without this patch the Sensu spawn limit setting cannot easily be configured by
this module.  This patch addresses the problem by adding a sensu::spawn_limit
class parameter.  If unset, the module behavior defers to the default behavior
of Sensu which uses an effective spawn limit of 12 in version 0.29.0.

Resolves #727
    @jeffmccune
    jeffmccune committed Jul 13, 2017
    Configuration menu
    Copy the full SHA
    8224cc3 View commit details
    Browse the repository at this point in the history

  3. Merge pull request #742 from jeffmccune/598_fix_rabbitmq_cluster 

    (#598) Improve rabbitmq clustering robustness
    @ghoneycutt
    ghoneycutt authored Jul 13, 2017
    Configuration menu
    Copy the full SHA
    d7c8ab4 View commit details
    Browse the repository at this point in the history

  4. Merge pull request #744 from jeffmccune/727_spawn_limit 

    (#727) Add sensu::spawn_limit class parameter
    @ghoneycutt
    ghoneycutt authored Jul 13, 2017
    Configuration menu
    Copy the full SHA
    92424e8 View commit details
    Browse the repository at this point in the history

  5. Release v2.24.0

    @ghoneycutt
    ghoneycutt committed Jul 13, 2017
    Configuration menu
    Copy the full SHA
    7887ed8 View commit details
    Browse the repository at this point in the history
Loading