* fix syntax

* import constructor

* fix syntax

* fix pillar path

* test partially streamlined haproxy config

* add missing parenthesis

* whitespace check

* test

* test

* test

* test

* comment jinja

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test new constructor function with integrated yaml_encode

* remove jinja comment

* add missing port colons

* add mysql support to haproxy constructor

* whitespace control

* additional whitespace control

* whitespace control

* remove stray endif

* whitespace control

* whitespace control

* fix port assignment

* auto-append semicolon if its missing in haproxy constructor

* whitespace fix

* test whitespace

* test

* more whitespace test

* more test

* test

* test

* test

* test

* test

* test

* test slightly improved indentation

* add additional comment explaining whitespace

* force specification of port numbers on backend of haproxy since they aren't user configurable

* add missing parenthesis

* add missing parenthesis

* remove seamless reload variable

* move openstack_services conf to dynamic pillar since it currently does not support user configuration.  I will create an issue to allow for overrides in a future update

* allow for default empty dict if file does not exist yet

* update pillar calls to have default empty dicts

* test

* add default placeholder

* fix requisite

* fix keystone connection string

* fix path

* add constructors to heat configuration

* fix constructor kwarg

* make heat conf align with rest of os nomenclature

* make db sync stateful

* streamline horizon config and add constructor

* test making memcached url address

* fix syntax

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* force to string

* test

* test

* fix syntax

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test printf style

* test

* test

* test

* test

* test

* test

* test

* test

* fix typo

* fix sql connection string for keystone domain

* clean up keystone-ldap configuration

* adjust dashboard configurations to make them more comparable - these should be merged at some point

* normalize magnum configuration

* add missing imports

* fix jinja variable

* streamline manila config

* change user/db

* use constructor for keystone

* update mds, mcd, mysql, network, neutron

* re-add missing tag

* remove superfluous import

* fix designate endpoint

* fix db user

* update to correct pillar call for keystone endpoint

* test updated ovn configuration

* update config options in ovn

* use base endpoint instead of full endpoint for designate

* update nova configuration with constructors

* improve statefulness check in nova

* partially update ovsdb

* properly import constructor

* streamline assignment of cluster remote - first draft

* drop extraneous quotation

* update commit comment

* fix spacing

* update placement configuration

* update rmq sahara share and swift

* fix variable name

* import constructor

* update volume configuration

* update constructor for glance IPs

* add kuryr user init to keystone, update zun and webssh

* update zun conf

* fix imports

* fix user name

* test condensed endpoint constructor function

* remove openstack admin credendtials from service endpoints - this will break post-conf items for now

* test new consolidated constructor

* finish removing old macro

* switch from keystone service user to swift service user

* update pillar top to reflect use of swift user rather than keystone in ceph conf

* re-add openstack admin creds to manila/neutron for post-conf stuff

* add missing base url arg

* add explicit kwarg assignments

* test orchestration macro

* add additional macros to orch for testing

* fix typo

* fix typo

* add orch macro and remove duplicative code

* fix requisite name

* fix requisite name for real

* add comment

* temporary workaround for non-existent database

* drop mysql precondition

* add webssh2 user to avoid failed systemd

* update webssh2 user shell and home

* drop home assignment

* add python3-cffi package due to

raise Exception("Version mismatch: this is the 'cffi' package version 
%s, located in %r.  When we import the top-level '_cffi_backend' 
extension module, we get version %s, located in %r.  The two versions 
should be equal; check your installation." % (

* revert to original license