search

rackspace-cookbooks / platformstack

Rackspace Base Stack via Chef
Other
4 stars 6 forks source link

Confusing naming on acl and iptables #181

Open martinb3 opened 9 years ago

martinb3 commented 9 years ago

In https://github.com/rackspace-cookbooks/platformstack/pull/180, we have two recipes ::iptables and ::acl, but if node['platformstack']['iptables']['enabled'] is true, it includes the ::acl recipe. That's very confusing.

/CC @prometheanfire @cybermerc

prometheanfire commented 9 years ago

the acl recipe can probably be moved into attributes in a platformstack namespace that get merged with the correct namespace if needed. Like demo attributes.

prometheanfire commented 9 years ago

a master switch could probably be made in rackspace_iptables as a fix