[library] crm commit is done using replace instead if patch

[Vladimir Kuklin]

This leads to race conditions and e.g. broken deployment when some of resources are not specified.

[Vladimir Kuklin (vkuklin)]

it looks like, we need to apply this patch for crmsh:

diff -Naur a/modules/cibconfig.py b/modules/cibconfig.py 1
   --- a/modules/cibconfig.py» 2013-02-12 16:14:02.000000000 +0400 2
   +++ b/modules/cibconfig.py» 2014-07-07 18:39:29.592779754 +0400 3
   @@ -2092,7 +2092,7 @@ 4
   return False 5
   if self.new_schema and not self.commit_schema(): 6
   return False 7
   - cibadmin_opts = force and "-R --force" or "-R" 8
   + cibadmin_opts = force and "-s -R --force" or "-R -s" 9
   rc = pipe_string("%s %s" % (cib_piped,cibadmin_opts), conf_node.toxml()) 10
   if rc != 0: 11
   update_err("cib",cibadmin_opts,conf_node.toprettyxml(), rc)

[OSCI Robot (oscirobot)]

Package crmsh has been built from changeset: http://gerrit.mirantis.com/17875
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1-stable-17875/centos
You can build an ISO with this package:
make iso EXTRA_RPM_REPOS="osci-testing,http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1-stable-17875/centos"

[OSCI Robot (oscirobot)]

Package crmsh has been built from changeset: http://gerrit.mirantis.com/17878
DEB Repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-5.1-stable-17878/ubuntu
You can build an ISO with this package:
make iso EXTRA_DEB_REPOS="http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-5.1-stable-17878/ubuntu /"

[Vladimir Kuklin (vkuklin)]

it looks the actual problem is that crm does application using replace and in this case the data can be outdated after galera or rabbitmq ocf modification. thus we need to build 1.2.6 version of crmsh along with patch that set's minimum required version of pacemaker to be 1.1.10 to support cibadmin patch option.

[Vladimir Kuklin (vkuklin)]

the latest ubuntu package is here:

https://launchpad.net/ubuntu/+source/crmsh/1.2.6+git+e77add-1.2ubuntu1

it contains all the stuff for deb package building including the patch and spec file for RPM

I modified it a little bit in order to apply patch during debian package build here:

https://gerrit.mirantis.com/#/c/17878/

I am going to test it and tell if it really helps.

[Vladimir Kuklin (vkuklin)]

this regression could be introduced by commits that disabled crm shadow usage along with merge of galera and rabbitmq ocf scripts that alter CIB attributes during deployment.

[Vladimir Kuklin (vkuklin)]

adding logs with haproxy resource failing to be committed

[OSCI Robot (oscirobot)]

Package crmsh has been built from changeset: http://gerrit.mirantis.com/17878
DEB Repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-5.1-stable-17878/ubuntu
You can build an ISO with this package:
make iso EXTRA_DEB_REPOS="http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-5.1-stable-17878/ubuntu /"

[Vladimir Kuklin (vkuklin)]

newer crmsh does not help at all. it looks like we need to do accurate crm diff application.

[Vladimir Kuklin (vkuklin)]

it should be worked around by following method:

instead of using live CIB:
1) create temporary shadow cib
2) run crm with this CIB and use configure load update
3) generate diff between live cib and this cib in corresponding scope:
   a) "resources" for resources and groups
   b) "constraints" for orders, locations and colocations
   c) "rsc_defaults" for rsc_defaults
4) generate a patch using crm_diff
5) fix xml patch by appending <cib> and <configuration> tags
6) apply the patch by using cibadmin -P --xml-file
7) clean up everything

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/105270

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/105271

[OpenStack Infra (hudson-openstack)]

Change abandoned by Vladimir Kuklin (<email address hidden>) on branch: master
Review: https://review.openstack.org/105270

[OpenStack Infra (hudson-openstack)]

Change abandoned by Vladimir Kuklin (<email address hidden>) on branch: master
Review: https://review.openstack.org/105271

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/105432

[OSCI Robot (oscirobot)]

Package crmsh has been built from changeset: http://gerrit.mirantis.com/17875
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1-stable-17875/centos
You can build an ISO with this package:
make iso EXTRA_RPM_REPOS="osci-testing,http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1-stable-17875/centos"

[OSCI Robot (oscirobot)]

Package crmsh has been built from changeset: http://gerrit.mirantis.com/17878
DEB Repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-5.1-stable/ubuntu
You can build an ISO with this package:
make iso EXTRA_DEB_REPOS="http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-5.1-stable/ubuntu /"

[OSCI Robot (oscirobot)]

Package crmsh has been built from changeset: http://gerrit.mirantis.com/17875
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1-stable/centos
You can build an ISO with this package:
make iso EXTRA_RPM_REPOS="osci-testing,http://osci-obs.vm.mirantis.net:82/centos-fuel-5.1-stable/centos"

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/105432
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=03342f53fac5cfadb44083b32f46f04b2b1ccbb3
Submitter: Jenkins
Branch: master

commit 03342f53fac5cfadb44083b32f46f04b2b1ccbb3
Author: Vladimir Kuklin <email address hidden>
Date: Tue Jul 8 15:24:07 2014 +0400

    Apply CIB changes using patch method

    Generate XML diff patch and apply it.

    1) generate config changes in shadow
    2) generate diff only for specific scope
    3) apply it

    This will not touch cluster attributes and
    other stuff it should not touch, thus not
    breaking functionality of OCF scripts using
    CIB attributes.

    Change-Id: Ibcfaa82707d1aed01b79c8307d17f4e71fd60906
    Closes-bug: #1338594

[OSCI Robot (oscirobot)]

Package crmsh has been built from changeset: http://gerrit.mirantis.com/30189
DEB Repository URL: http:///osci-obs.vm.mirantis.net:82/ubuntu-fuel-6.0-stable-30189/ubuntu