Ubuntu can't match `iptables-persistent` package about puppetlabs-firewall HOT 13 CLOSED

Hmm, package search doesn't confirm this: http://packages.ubuntu.com/search?keywords=iptables-persistent&searchon=names&suite=all&section=all

Let me see if I have a 1210 vm somewhere. Did you use 'class { firewall: }' at all to install iptables-persistent, or was this error without the class declaration?

from puppetlabs-firewall.

Aah okay - so these are just warnings from the fact. Make sure you add:

class { 'firewall': }

On the hosts you want to use the firewall provider on, and it should only occur once. /cc @dcarley.

from puppetlabs-firewall.

right, I followed the instructions and only have firewall declared at the global level in site.pp... well not quite, but effectively so...

Um, like this https://github.com/Aethylred/dynaguppy/blob/puppetmaster_ssl/modules/dynaguppy/manifests/firewall/persist.pp

from puppetlabs-firewall.

Looks like the behaviour of dpkg changed and I didn't test Quantal or later. I'll fix up that error from the fact.

@Aethylred You can now remove the exec { 'persist-firewall': } resource and notify param from the Firewall {} resource. Persistence is now handled automatically, if you include/instantiate the firewall class as Ken suggests.

The README has some up-to-date examples of this. However it doesn't have any specific information for existing users, some maybe we need some additional information there to compliment any release notes.

from puppetlabs-firewall.

@dcarley I've added the instructions to the Changelog now, and we've released that - but yeah, not sure. Maybe we need an 'upgrading' section that covers this kind of thing in the README.

from puppetlabs-firewall.

Ok, I've made the changes you specified, but the output remains the same. Aethylred/dynaguppy@9c8ed61

Info: Loading facts in /var/lib/puppet/lib/facter/iptables_version.rb
dpkg-query: no packages found matching iptables-persistent
dpkg-query: no packages found matching iptables-persistent
dpkg-query: no packages found matching iptables-persistent

from puppetlabs-firewall.

@Aethylred Have you pulled the latest master which includes 7c1ebd1 ?

from puppetlabs-firewall.

@Aethylred Have you pulled the latest master which includes 7c1ebd1 ?

And did including the firewall class install iptables-persistent? It should have ...

from puppetlabs-firewall.

Ah, need to switch to @dcarley's fork. Time for more fun with submodules.

from puppetlabs-firewall.

Sorted.

Info: Loading facts in /var/lib/puppet/lib/facter/iptables_version.rb
Info: Caching catalog for puppet.local

from puppetlabs-firewall.

Cool, well @dcarley when you're ready - raise a PR and we'll close this issue when merged in. I can raise a 0.2.1 pretty quickly with this fix.

from puppetlabs-firewall.

Sorry, that wasn't what I meant to do. Don't think it warrants a PR. Pushed to upstream/master now.

from puppetlabs-firewall.

You can switch back to upstream now, @Aethylred. Thanks for your help.

from puppetlabs-firewall.