Projet

Général

Profil

Révision e53053ce

IDe53053ce3df771f5fcc24f87dd86f5c49db91b46
Parent 9e5b8bf0
Enfant 61f03b47

Ajouté par Steve Traylen il y a plus de 4 ans

Add comments for all the nftable::rules entries

For each nftable::rule this adds an extra concat fragment to
add a comment containing the name and order number for the rule.

The motivation here is to make the mapping from resulting rules back
to puppet code more obvious. When adding a new rule it should be more
obvious to understand what order to choose.

An example resulting table ends up reading as:

```
HASH Start of fragment order:00 default_in header
chain default_in {
HASH Start of fragment order:50 rulename:default_in-dhcpv6_client
ip6 saddr fe80::/10 ip6 daddr fe80::/10 udp sport 547 udp dport 546 accept
HASH Start of fragment order:50 rulename:default_in-ssh
tcp dport {22} accept
HASH Start of fragment order:90 rulename:default_in-drop_broadcasts
meta pkttype broadcast counter drop
HASH Start of fragment order:99 default_in footer
}

```

In addition there is a new test `nftables::rule`. This includes
a pending test since I would assume setting source and content
on a rule should be an error however this currently not the case.

Voir les différences:

manifests/rule.pp
25 25
      $fragment = "nftables-${table}-chain-${data[0]}-rule-${data[1]}"
26 26
    }
27 27

  
28
    concat::fragment{"${fragment}_header":
29
      content => "#   Start of fragment order:${order} rulename:${rulename}",
30
      order   => "${order}${fragment}a",
31
      target  => "nftables-${table}-chain-${data[0]}",
32
    }
33

  
28 34
    concat::fragment{
29 35
      $fragment:
30
        order  => $order,
36
        order  => "${order}${fragment}b",
31 37
        target => "nftables-${table}-chain-${data[0]}",
32 38
    }
33 39

  

Formats disponibles : Unified diff