ensure_packages unzip with 'before' breaks interoperability about puppet-consul HOT 5 CLOSED

Possible solution: "Remove management of unzip package" - danieldreier@8253cba

from puppet-consul.

Fixed by #185

from puppet-consul.

Closing since this is fixed.

from puppet-consul.

There is another solution. (just ran into this in my copy of 1.0.2)

If you put the relationship after the ensure_packages, it can work without breaking other ensure_packages.

ensure_package('unzip')
Package['unzip'] -> Staging::File['consul.zip']

This allows you to still ensure that the package is managed, but doesn't cause a Duplicate Declaration error. I'm guessing it's similar to the

include 'myclass'

vs

class:{'myclass':}

schematics.

from puppet-consul.

That will still cause the duplication error if another user declares (not via ensure_packages) unzip before consul is parsed.

from puppet-consul.