firewall: stop using stdlib stages
authorEmilien Macchi <emilien@redhat.com>
Mon, 21 Nov 2016 14:57:09 +0000 (09:57 -0500)
committerEmilien Macchi <emilien@redhat.com>
Mon, 21 Nov 2016 19:24:20 +0000 (14:24 -0500)
commit2ca3cb03ad5f05469e5ae181981e559ccc77371f
treeb86f7d60996869343e9a052b9975d222052836cd
parent92f9c952d6cbb0c11c8457de0e49748f3adf667e
firewall: stop using stdlib stages

Using Puppet stdlib in TripleO is risky because it exposes deployments
to dependency cycles in the catalog.
We should rather use native functions to make orchestrations, like
ordering and dependencies management.

This patch:

- removes usage of stages from stdlib
- use ordering to make sure we run pre rules before post
- use ordering to make sure we start all Services in catalog before post
  rules. It ensure that we don't drop all traffic before starting the
  services, which could lead to services errors (e.g. trying to reach database
  or amqp)

Change-Id: Iec4705d6b785a40ccf6f43809b94b726ccd47fef
Closes-Bug: #1643575
manifests/firewall.pp