search

redhat-openstack / puppet-pacemaker

Puppet modules to manage pacemaker with corosync
9 stars 25 forks source link

wrong default value #64

Closed pulecp closed 8 years ago

Spredzy commented 8 years ago

Not sure why is this considered 'wrong default value'. Could you let us know the uses cases here.

As per its intentional attempts[1], the default value aims to be an hash. Also the behavior of the inline[2] confirms that.

[1] https://github.com/redhat-openstack/puppet-pacemaker/commit/733c1088ccfa24739929af7a8612ff0ade46bc6f#diff-04c6e90faac2675aa89e2176d2eec7d8R54 [2] https://github.com/redhat-openstack/puppet-pacemaker/commit/733c1088ccfa24739929af7a8612ff0ade46bc6f#diff-ae0c67c94033f671663fcda614e2c80aR86

pulecp commented 8 years ago

Ahh, the issue is that I use old version of ruby where the method flatten is not implemented. So this pull request can be cancelled.

irb(main):001:0> RUBY_VERSION
=> "1.8.7"
irb(main):002:0> {}.flatten.join(" ")
NoMethodError: undefined method `flatten' for {}:Hash
        from (irb):2
        from :0