Commit Graph

4 Commits

Author SHA1 Message Date
Zuul
38dd0a3ef8 Merge "Add Basic Authentication support for HAProxy" 2017-12-19 20:52:30 +00:00
Cédric Jeanneret
9d438cd142 Add Basic Authentication support for HAProxy
In order to get a proper support for authenticated endpoints, this patch
creates a new definition (tripleo::haproxy::userlist) and exploit it in
the dynamic endpoint (tripleo::haproxy::service_endpoints) as well as
standard tripleo::haproxy::endpoint.

It also detected a small issue with the "underscorization" of the
service name, the missing 'G' flag for regsubst, that preventend all
dashes to be replaced by underscores.

Change-Id: Ie7471155d1ef3f6adc177a468b81ac410bbfb9c0
Closes-Bug: 1736132
2017-12-16 09:37:27 +01:00
Cédric Jeanneret
2f7d622b0a correct unit tests
Moved to on_supported_os instead of hard-coded list

Change-Id: Id501931b7cb32463680cc861ab4e70230e2d98d1
2017-12-16 09:36:18 +01:00
Cédric Jeanneret
4430253701 Add unit test for tripleo::haproxy::service_endpoints
In order to get a proper unit test suits, we need to change the
separator from the "dot" to "double-semicolons".

The "." is a reserved character in YAML for hashes. In puppet world,
we commonly use "::".

Unit tests don't work if we let the "dot" separator, and apparently it's
an intended outcome with that kind of syntax.

Fact it's working in the deploy process is kind of black magic (see the
tripleo::firewall::service_rules resource for example). And the "dot"
creates something weird, as all other resources are using the standard
double-semicolon.

Change-Id: I78625a6e69f58dfb2bbad83fdd4f798cd3f4c281
Closes-Bug: 1737086
2017-12-14 09:47:43 +01:00