OK, I'll update the other service_exists() calls in the Ubuntu module in trunk.
AC On Jul 17, 2013, at 8:56 AM, Aaron Peeler <[email protected]> wrote: > I would suggest ufw. This seems to be more inline with unbuntu the way > to manage the OS firewall. > > AP > > > > On Tue, Jul 16, 2013 at 6:08 PM, Aaron Coburn <[email protected]> wrote: >> Hi, Folks, >> I have only just started working with Ubuntu images in the VCL, so forgive >> me for any ignorance here. However, because the default Ubuntu install uses >> ufw instead of iptables, I added a local version of >> `firewall_compare_update` in OS::Linux::Ubuntu. This is in line with how >> 2.3.2 works. This local version checks for the ufw service instead of >> iptables; otherwise, the two versions are the same. I have pushed the change >> into both the 2.3-bugfix line and also on trunk. >> >> After looking at the code in trunk some more, however, it appears that, >> throughout the Ubuntu module, the code is checking for the 'iptables' >> service: >> >> $self->service_exists("iptables") >> >> instead of "ufw". Is this correct? >> >> If "iptables" is the intended service, then I will roll back my changes. >> >> -Aaron Coburn >> >> >> >> -- >> Aaron Coburn >> Academic Technology Services, Amherst College >> [email protected]<mailto:[email protected]> >> >> >> >> >> >> >> > > > > -- > Aaron Peeler > Program Manager > Virtual Computing Lab > NC State University > > All electronic mail messages in connection with State business which > are sent to or received by this account are subject to the NC Public > Records Law and may be disclosed to third parties.
