<% unless @interfaces.empty? -%>
define external-interfaces = { <%= @interfaces.sort.uniq.join(", ") %> }
<% end -%>
-
-define ip-private-addresses = { 0.0.0.0, 10.0.0.0/8, 127.0.0.0/8, 169.254.0.0/16, 172.16.0.0/12, 192.0.2.0/24, 192.168.0.0/16 }
+# Exclude 169.254.169.0/24 from ip-private-addresses as is widely by cloud providers.
+define ip-private-addresses = { 0.0.0.0, 10.0.0.0/8, 127.0.0.0/8, 169.254.0.0-169.254.168.255, 169.254.170.0-169.254.255.255, 172.16.0.0/12, 192.0.2.0/24, 192.168.0.0/16 }
define ip-multicast-addresses = { 224.0.0.0/4 }
define ip6-private-addresses = { 2001:db8::/32, fc00::/7 }
define ip6-multicast-addresses = { ff00::/8 }
table inet chef-filter {
set ip-osm-addresses {
type ipv4_addr
-<% unless Array(@hosts["inet"]).empty? -%>
- elements = { <%= Array(@hosts["inet"]).sort.join(", ") %> }
+<% unless Array(@hosts[:inet]).empty? -%>
+ elements = { <%= Array(@hosts[:inet]).sort.join(", ") %> }
<% end -%>
}
set ip6-osm-addresses {
type ipv6_addr
-<% unless Array(@hosts["inet"]).empty? -%>
- elements = { <%= Array(@hosts["inet6"]).sort.join(", ") %> }
+<% unless Array(@hosts[:inet6]).empty? -%>
+ elements = { <%= Array(@hosts[:inet6]).sort.join(", ") %> }
<% end -%>
}
}
<% node[:networking][:firewall][:sets].each do |set| -%>
- set <%= set %> {
-<% if set.end_with?("-ip") -%>
- type ipv4_addr
-<% elsif set.end_with?("-ip6") -%>
- type ipv6_addr
+ set <%= set[:name] %> {
+ type <%= set[:type] %>
+<% if set[:flags] -%>
+ flags <%= set[:flags].join(", ") %>
<% end -%>
- flags dynamic
-<% unless set.start_with?("connlimit-") -%>
- timeout 120s
+<% if set[:timeout] -%>
+ timeout <%= set[:timeout] %>s
<% end -%>
}
<% end -%>
chain log-and-drop {
- limit rate 1/second log
+ limit rate 1/second log level notice
drop
}
chain log-and-reject {
- limit rate 1/second log
+ limit rate 1/second log level notice
reject
}
chain postrouting {
type nat hook postrouting priority srcnat;
-<% node.interfaces(:role => :external, :family => :inet).each do |external| -%>
-<% node.interfaces(:role => :internal, :family => :inet).each do |internal| -%>
- oifname { <%= external[:interface] %> } ip saddr { <%= internal[:network] %>/<%= internal[:prefix] %> } snat <%= external[:address] %>
+<% node.interfaces(:role => :external).each do |external| -%>
+<% node.ipaddresses(:role => :internal, :family => :inet).each do |internal| -%>
+ oifname { <%= external[:interface] %> } ip saddr { <%= internal.subnet %> } snat <%= external[:inet][:address] %>
<% end -%>
<% end -%>
}