[openstack-dev] [puppet] [tripleo] [fuel] Puppet pacemaker module

Sofer Athlan-Guyot sathlang at redhat.com
Wed May 25 11:24:06 UTC 2016


Hi,

The merge of the Fuel and Tripleo puppet-pacemaker is ready for review,
as the green CI shows: https://review.openstack.org/#/c/296440/

The great news is that all the legacy code has been kept untouched.

Basically, reviewers should only ensure that this is indeed the case.  No need
to review the ~18K loc[1].  All new features are under a new namespace:
 - new manifests are under manifests/new namespace;
 - new provider are named pacemaker_* (instead of pcmk_*)

So moving to the new pacemaker means that the consumers has to
explicitly change its own manifests, giving them a upgrade path[2][3]

The new module is very well documented, so gives the README.md[4] a go.  It
has a whole lot of new features: service provider for pacemaker; much
more better test coverage, both unit and acceptance; a whole new array
of providers, ...

The above patch only include the xml providers and some pcs.  Resource,
colocation and order pcs providers are currently being ported in this
patch[5].  Those are the one used by the tripleo consumer.  It may also
has a port of the service provider depending on the tests I'm doing.

I would like to thanks Dmitry Ilyin for its outstanding work on this
one.

Enjoy,

[1] those are mainly under the lib/pacemaker which is the xml library
used by fuel as an interface to the pacemaker cluster (equivalent to the
pcs command)

[2] this is what is done in https://review.openstack.org/302409 and
https://review.openstack.org/309069 for tripleo

[3] fuel integration will come later, so don't worry about red CI for
fuel.

[4] https://review.openstack.org/gitweb?p=openstack/puppet-pacemaker.git;a=blob;f=README.md;h=446fc3f6b3c91924e98fbcf0e78923b29e0fb3f5;hb=4d2e554f687f525ad54c0b0c41a610aa68f50e4d

[5] https://review.openstack.org/310713

--
Sofer Athlan-Guyot



More information about the OpenStack-dev mailing list