diff options
| author |   David Gurtner <aldavud@crimson.ch> | 2017-02-20 18:50:54 +0100 |
|---|---|---|
| committer | David Gurtner <aldavud@crimson.ch> | 2017-02-20 19:03:08 +0100 |
| commit | f8313f755c56b2cf614605588f8164c510b18956 (patch) | |
| tree | 13d6ede604c0debe83ee237fb861ede78dec76e6 | |
| parent | 47969551159a81cd1a050c3b345892aec0bf7d96 (diff) | |
Replace default to be more robust
Specifying undef as the fallback only works because the merge
function specifically checks for this:
next if arg.is_a? String and arg.empty? # empty string is synonym for puppet's undef
But the empty Hash would be a much more robust default.
Change-Id: I7e302c00ef030d75998e352d88b3ccc60b194ab7
| -rw-r--r-- | manifests/profile/base/horizon.pp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/manifests/profile/base/horizon.pp b/manifests/profile/base/horizon.pp index 1849435..278c25c 100644 --- a/manifests/profile/base/horizon.pp +++ b/manifests/profile/base/horizon.pp @@ -25,11 +25,11 @@ # # [*neutron_options*] # (Optional) A hash of parameters to enable features specific to Neutron -# Defaults to hiera('horizon::neutron_options', undef) +# Defaults to hiera('horizon::neutron_options', {}) # class tripleo::profile::base::horizon ( $step = hiera('step'), - $neutron_options = hiera('horizon::neutron_options', undef), + $neutron_options = hiera('horizon::neutron_options', {}), ) { if $step >= 4 { # Horizon |