Skip to content

Commit

Permalink
Adds logic for managing redis-sentinel package
Browse files Browse the repository at this point in the history
* redis-sentinel is present in upstream repos
* It's also present for 16.04/xenial onwards
* So lets only manage that package if we're on
Ubuntu newer or if we're managing the repo
  • Loading branch information
petems committed May 3, 2017
1 parent 3d28af9 commit c858422
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions manifests/sentinel.pp
Original file line number Diff line number Diff line change
Expand Up @@ -192,11 +192,14 @@

require ::redis

# Debian flavour machines have a dedicated redis-sentinel package
# See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=775414 for context
if $::osfamily == 'Debian' {
package { $package_name:
ensure => $package_ensure,
# Debian flavour machines have a dedicated redis-sentinel package
# This is default in Xenial onwards, unstable debian or PPA/other upstream
# See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=775414 for context
if (versioncmp($::operatingsystemmajrelease, '16.04') >= 0 or $::redis::manage_repo) {
package { $package_name:
ensure => $package_ensure,
}
}
}

Expand Down

0 comments on commit c858422

Please sign in to comment.