[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