justicel / puppet-couchbase

Puppet couchbase module for auto-scaling of couchbase with puppet
9 stars 30 forks source link

Fix duplicate declaration of openssl package #41

Closed MattiasGees closed 5 years ago

MattiasGees commented 7 years ago

I started moving our puppetmaster to puppet server 4. During testing I came into the problem that openssl package was declared twice. I have openssl installed with my base install and after that I install couchbase.

After some investigation I noticed it was because it just skipped over the if structure and went directly to the ensure_packages definition. I fixed this by using the stdlib module and splitting out the array.

abegosum commented 7 years ago

I'm also seeing this behavior problem in 3.8 using the future parser. Please consider merging this pull request.

justicel commented 5 years ago

This is finally resolved and I'll be releasing a puppet 5 compatible module