diff options
author | Steven Hardy <shardy@redhat.com> | 2016-12-22 16:42:12 +0000 |
---|---|---|
committer | Steven Hardy <shardy@redhat.com> | 2016-12-22 16:42:12 +0000 |
commit | 3d8dfa171437502721b599fd67e4fbd2951b7f4c (patch) | |
tree | 4c7940e25e54b1e1fcfbb1aca79a2453dbefe356 | |
parent | 918bed5db00df194a317dc61f94c521340dab010 (diff) |
Ensure package updates don't happen unexpectedly
I'm seeing this run yum update on deploy, even though hiera
tripleo::packages::enable_upgrade says false.
I assume these are needed because we're getting "false", but I'm
unclear if this is a recently introduced problem (I only noticed it
today as my image has outdated centos packages and it thus hung on
step2 of the deploy.
Change-Id: If09cdde9883f2674dbbc40944be5fe4445caa08e
Closes-Bug: #1652107
-rw-r--r-- | manifests/packages.pp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/manifests/packages.pp b/manifests/packages.pp index ec2635a..147c76a 100644 --- a/manifests/packages.pp +++ b/manifests/packages.pp @@ -35,7 +35,7 @@ class tripleo::packages ( # required for stages include ::stdlib - if !$enable_install and !$enable_upgrade { + if !str2bool($enable_install) and !str2bool($enable_upgrade) { case $::osfamily { 'RedHat': { Package <| |> { provider => 'norpm' } @@ -46,7 +46,7 @@ class tripleo::packages ( } } - if $enable_upgrade { + if str2bool($enable_upgrade) { Package <| |> { ensure => 'latest' } # Running the package upgrade before managing Services in the main stage. # So we're sure that services will be able to restart with the new version |