[nailgun] test_orchestrator_serializer.TestNeutronOrchestratorSerializerBonds.test_bonds_serialization fails sometimes

[Aleksey Kasatkin]

Test failed in CI:

https://fuel-jenkins.mirantis.com/job/verify-fuel-web/6459/consoleFull

Stacktrace

self = <nailgun.test.integration.test_orchestrator_serializer.TestNeutronOrchestratorSerializerBonds testMethod=test_bonds_serialization>

    def test_bonds_serialization(self):
        for mode in consts.BOND_MODES:
> self.check_bond_with_mode(mode)

nailgun/test/integration/test_orchestrator_serializer.py:2066:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
nailgun/test/integration/test_orchestrator_serializer.py:2062: in check_bond_with_mode
    self.check_add_bond_msg_non_lacp(bonds[0], mode)
nailgun/test/integration/test_orchestrator_serializer.py:2043: in check_add_bond_msg_non_lacp
    'properties': ['bond_mode={0}'.format(mode)]
E AssertionError: {'action': 'add-bond', 'bridge': u'br-ovsbond0', 'interfaces': [u'eth2', u'eth1' [truncated]... != {'action': 'add-bond', 'bridge': 'br-ovsbond0', 'interfaces': ['eth1', 'eth2'], [truncated]...
E {'action': 'add-bond',
E - 'bridge': u'br-ovsbond0',
E ? -
E
E + 'bridge': 'br-ovsbond0',
E - 'interfaces': [u'eth2', u'eth1'],
E + 'interfaces': ['eth1', 'eth2'],
E - 'name': u'ovsbond0',
E ? -
E
E + 'name': 'ovsbond0',
E 'properties': ['bond_mode=broadcast']}

[Aleksey Kasatkin (alekseyk-ru)]

Set priority to high as it failed several times today.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/166174
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=acd7adbf82135907cff35e6729596726c5b18f48
Submitter: Jenkins
Branch: master

commit acd7adbf82135907cff35e6729596726c5b18f48
Author: Sebastian Kalinowski <email address hidden>
Date: Fri Mar 20 12:24:33 2015 +0100

    Add tests and order ignoring for datadiff

    datadiff from tests now allows to ignore order of lists
    when comparing structures with it.
    Added some tests

    Change-Id: I6ef46f7cfe1a617f2b48add37a75202e386bcd89
    Related-Bug: #1433691

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/166175
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=aa2c855854dde1d97037ee5f59be33e78cc8c0ef
Submitter: Jenkins
Branch: master

commit aa2c855854dde1d97037ee5f59be33e78cc8c0ef
Author: Sebastian Kalinowski <email address hidden>
Date: Fri Mar 20 12:26:05 2015 +0100

    Fix random failing test_bonds_serialization

    Replaced simple assertEqual with datadiff util with ignoring
    order of items in lists.

    Change-Id: I4f0f8eb61311e87e35ffd2b3b0faa9f20dc59fb8
    Closes-Bug: #1433691