'ha_flat_scalability' failed on Ubuntu because of unexpected pacemaker status format

[Dennis Dmitriev]

Jenkins job: http://jenkins-product.srt.mirantis.net:8080/view/6.0_swarm/job/6.0_fuelmain.system_test.ubuntu.thread_4/43/
System test: 'ha_flat_scalability'

In the method fuel_web_client/assert_pacemaker() the following string is prepared:

# Assert online nodes list
        online = 'Online: [ {0} ]'.format(' '.join(fqdn_names(online_nodes)))

, which contains as result (for particular test for example) :
Online: [ node-1.test.domain.local node-2.test.domain.local node-5.test.domain.local ]

This string is compared to pacemaker status 'crm_mon -1' which on Ubuntu returns:

Online: [ node-1 node-2 node-5 ]

The strings never match so the test is failed.

A bug was added regarding the output of pacemaker status: https://bugs.launchpad.net/fuel/+bug/1400161

assert_pacemaker() must take into account this issue.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/140173

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/6.0
Review: https://review.openstack.org/140631

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/140169
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=81594361fd573e430a5b4add5b5bf1ed15f33c2e
Submitter: Jenkins
Branch: master

commit 81594361fd573e430a5b4add5b5bf1ed15f33c2e
Author: Dennis Dmitriev <email address hidden>
Date: Tue Dec 9 00:22:55 2014 +0200

    assert_pacemaker() transforms node names to FQDN before comparison

    Pacemaker returns it's status using FQDN on CentOS and short node
    names on Ubuntu. It makes hard for comparison different node
    name formats with always FQDN from nailgun.

    Now output of pacemaker status converts into FQDN before comparison.

    Change-Id: I73bdb208111e993ab0135cd7aded47e90a0f1b34
    Closes-Bug:#1400169

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/140631
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=a78bbb136dcf4dc2164d79f3203d88d93932ee0e
Submitter: Jenkins
Branch: stable/6.0

commit a78bbb136dcf4dc2164d79f3203d88d93932ee0e
Author: Dennis Dmitriev <email address hidden>
Date: Tue Dec 9 00:22:55 2014 +0200

    assert_pacemaker() transforms node names to FQDN before comparison

    Pacemaker returns it's status using FQDN on CentOS and short node
    names on Ubuntu. It makes hard for comparison different node
    name formats with always FQDN from nailgun.

    Now output of pacemaker status converts into FQDN before comparison.

    Change-Id: I73bdb208111e993ab0135cd7aded47e90a0f1b34
    Closes-Bug:#1400169
    (cherry picked from commit 81594361fd573e430a5b4add5b5bf1ed15f33c2e)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/140173
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=97c32d3e8e4dec718d08606cfcbcfe43530c3882
Submitter: Jenkins
Branch: stable/5.1

commit 97c32d3e8e4dec718d08606cfcbcfe43530c3882
Author: Dennis Dmitriev <email address hidden>
Date: Tue Dec 9 00:45:55 2014 +0200

    assert_pacemaker() transforms node names to FQDN before comparison

    Pacemaker returns it's status using FQDN on CentOS and short node
    names on Ubuntu. It makes hard for comparison different node
    name formats with always FQDN from nailgun.

    Now output of pacemaker status converts into FQDN before comparison.

    Change-Id: I3395a18da6b14b05d592cf5b0d54d5cf951efd57
    Closes-Bug:#1400169

[Dennis Dmitriev (ddmitriev)]

Fix released for 6.0/6.1:
ISO version: {u'build_id': u'2014-12-18_01-32-01', u'ostf_sha': u'a9afb68710d809570460c29d6c3293219d3624d4', u'build_number': u'56', u'auth_required': True, u'nailgun_sha': u'5f91157daa6798ff522ca9f6d34e7e135f150a90', u'production': u'docker', u'api': u'1.0', u'fuelmain_sha': u'45caacadb878abfbd9d60e134d72229698b469c9', u'astute_sha': u'16b252d93be6aaa73030b8100cf8c5ca6a970a91', u'feature_groups': [u'mirantis'], u'release': u'6.0', u'release_versions': {u'2014.2-6.0': {u'VERSION': {u'build_id': u'2014-12-18_01-32-01', u'ostf_sha': u'a9afb68710d809570460c29d6c3293219d3624d4', u'build_number': u'56', u'api': u'1.0', u'nailgun_sha': u'5f91157daa6798ff522ca9f6d34e7e135f150a90', u'production': u'docker', u'fuelmain_sha': u'45caacadb878abfbd9d60e134d72229698b469c9', u'astute_sha': u'16b252d93be6aaa73030b8100cf8c5ca6a970a91', u'feature_groups': [u'mirantis'], u'release': u'6.0', u'fuellib_sha': u'73332192a257ea02c40a39885c502ad1ebdf3eda'}}}, u'fuellib_sha': u'73332192a257ea02c40a39885c502ad1ebdf3eda'}