diff options
| author |   Emilien Macchi <emilien@redhat.com> | 2016-07-21 09:50:56 -0400 |
|---|---|---|
| committer | Emilien Macchi <emilien@redhat.com> | 2016-07-22 09:02:13 -0400 |
| commit | 95b432e9bc6295ed9f544db6077143c2f0660165 (patch) | |
| tree | 38ebbc444634965054ba4dff3919690ba1ecfea9 /manifests/profile/base/pacemaker.pp | |
| parent | 964526ba48fb8135f283dcd29d0c8a1ae3bec364 (diff) | |
use parameter to lookup the step instead of hiera again
In some profiles, we were looking up the $step by using Hiera
again, while we already do it in the parameter definition.
When using this class outside THT, it will fail but with this patch, we
could use just set the $step parameter and the rest of the manifest will
work.
Change-Id: I7082f47204fb4e529b164e4c4f1032e7bdd88f02
Diffstat (limited to 'manifests/profile/base/pacemaker.pp')
| -rw-r--r-- | manifests/profile/base/pacemaker.pp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/manifests/profile/base/pacemaker.pp b/manifests/profile/base/pacemaker.pp index de107a9..1785656 100644 --- a/manifests/profile/base/pacemaker.pp +++ b/manifests/profile/base/pacemaker.pp @@ -38,9 +38,9 @@ class tripleo::profile::base::pacemaker ( $pacemaker_master = false } - $enable_fencing = str2bool(hiera('enable_fencing', false)) and hiera('step') >= 5 + $enable_fencing = str2bool(hiera('enable_fencing', false)) and $step >= 5 - if hiera('step') >= 1 { + if $step >= 1 { $pacemaker_cluster_members = downcase(regsubst(hiera('controller_node_names'), ',', ' ', 'G')) $corosync_ipv6 = str2bool(hiera('corosync_ipv6', false)) if $corosync_ipv6 { @@ -78,7 +78,7 @@ class tripleo::profile::base::pacemaker ( } } - if hiera('step') >= 2 { + if $step >= 2 { if $pacemaker_master { include ::pacemaker::resource_defaults |