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.45.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.46.0
Choose a head ref
  • 6 commits
  • 7 files changed
  • 3 contributors

Commits on Sep 21, 2017

  1. Add dependencies for sensu_gem plugins

    Previously, on upgrades to Sensu that installed a new embedded Ruby
    installation, there was a chance that plugins installed with the `sensu_gem`
    package provider would be evaluated BEFORE the Sensu package was upgraded
    (which would mean that a second run of Puppet was necessary to get the plugins
    installed into the new embedded Ruby installation). This commit adds a resource
    chain ensuring that a change to the Sensu package will come before any package
    resources using the `sensu_gem` provider (which would in turn notify the Sensu
    services of their change).
    glarizza committed Sep 21, 2017
    Configuration menu
    Copy the full SHA
    16d22a9 View commit details
    Browse the repository at this point in the history
  2. Add script to check for plugin install order

    This commit adds a script that will allow you to replicate and check for the
    problem where plugins may get installed before a sensu package change. Note
    that I have hardcoded version '0.23.0' as an 'older' version of Sensu because
    I know it uses a different Ruby MRI than Latest. This may need adjusted later.
    glarizza committed Sep 21, 2017
    Configuration menu
    Copy the full SHA
    ebeeeef View commit details
    Browse the repository at this point in the history

Commits on Sep 22, 2017

  1. Correct reference to services

    Tests were failing because of this stanza (and specifically the last line):
    
    ```puppet
    Package['sensu'] ~>
    Package<| provider == 'sensu_gem' |> ~>
    Service['sensu-client', 'sensu-server', 'sensu-api']
    ```
    
    On cases where the api or client hasn't been installed there won't be any
    resources for the sensu-api or sensu-client service. In its place is this line:
    
    ```puppet
    ~> Service<| title == 'sensu-client' or title == 'sensu-server' or title == 'sensu-api'|>
    ```
    
    That collector will discover the resources if they're there and setup the
    dependencies, or exit silently if they're not in the catalog. It's not the
    prettiest, but it will do.
    glarizza committed Sep 22, 2017
    Configuration menu
    Copy the full SHA
    b1be918 View commit details
    Browse the repository at this point in the history

Commits on Jan 15, 2018

  1. Merge pull request #817 from glarizza/542_sensu_gem_dependencies

    [542] Add dependencies for sensu_gem plugins
    ghoneycutt authored Jan 15, 2018
    Configuration menu
    Copy the full SHA
    337bba4 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    0abee09 View commit details
    Browse the repository at this point in the history
  3. Release v2.46.0

    ghoneycutt committed Jan 15, 2018
    Configuration menu
    Copy the full SHA
    0093adf View commit details
    Browse the repository at this point in the history
Loading