[systest] List on devops nodes is not sorted in prepare_slaves_* tests

Bug #1528858 reported by Artem Panchenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Artem Panchenko
8.0.x
Fix Released
Critical
Artem Panchenko
Mitaka
Fix Released
Critical
Artem Panchenko

Bug Description

After the following patch was merged we started to use unsorted list of nodes in system tests which prepare slaves:

https://github.com/openstack/fuel-qa/commit/04693990cb186a50fca817ff701160c924347983

This cause tests failure, because unexpected slaves are bootstrapped while in deployment cases we use predefined names:

https://github.com/openstack/fuel-qa/blob/master/fuelweb_test/tests/test_neutron_tun.py#L74

                                                       <<< 2. Start 3 slave nodes >>>
2015-12-23 12:35:20,795 - INFO environment.py:124 -- Bootstrapping node: slave-02
2015-12-23 12:35:26,775 - INFO environment.py:124 -- Bootstrapping node: slave-03
2015-12-23 12:35:32,708 - INFO environment.py:124 -- Bootstrapping node: slave-04

======================================================================
ERROR: Deploy cluster in ha mode with 1 controller and Neutron VXLAN
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jenkins/venv-nailgun-tests-2.9/local/lib/python2.7/site-packages/proboscis/case.py", line 296, in testng_method_mistake_capture_func
    compatability.capture_type_error(s_func)
  File "/home/jenkins/venv-nailgun-tests-2.9/local/lib/python2.7/site-packages/proboscis/compatability/exceptions_2_6.py", line 27, in capture_type_error
    func()
  File "/home/jenkins/venv-nailgun-tests-2.9/local/lib/python2.7/site-packages/proboscis/case.py", line 350, in func
    func(test_case.state.get_state())
  File "/home/jenkins/workspace/8.0.ubuntu.smoke_neutron/fuelweb_test/helpers/decorators.py", line 81, in wrapper
    result = func(*args, **kwargs)
  File "/home/jenkins/workspace/8.0.ubuntu.smoke_neutron/fuelweb_test/tests/test_neutron_tun.py", line 76, in deploy_neutron_tun
    'slave-03': ['compute', 'cinder']
  File "/home/jenkins/workspace/8.0.ubuntu.smoke_neutron/fuelweb_test/__init__.py", line 57, in wrapped
    result = func(*args, **kwargs)
  File "/home/jenkins/workspace/8.0.ubuntu.smoke_neutron/fuelweb_test/models/fuel_web_client.py", line 1119, in update_nodes
    timeout=60 * 2)
  File "/home/jenkins/venv-nailgun-tests-2.9/local/lib/python2.7/site-packages/devops/helpers/helpers.py", line 92, in wait
    while not predicate():
  File "/home/jenkins/workspace/8.0.ubuntu.smoke_neutron/fuelweb_test/models/fuel_web_client.py", line 1118, in <lambda>
    self.get_nailgun_node_by_devops_node(devops_node)['online'],
TypeError: 'NoneType' object has no attribute '__getitem__'

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (master)

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

Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Artem Panchenko (apanchenko-8)
status: New → In Progress
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/260983
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=7e4be4fac9841b85b14b2bcfffea672bf683f288
Submitter: Jenkins
Branch: master

commit 7e4be4fac9841b85b14b2bcfffea672bf683f288
Author: Artem Panchenko <email address hidden>
Date: Wed Dec 23 13:52:23 2015 +0000

    Revert "Test for postdeploy plugin install and enable."

    This reverts commit 04693990cb186a50fca817ff701160c924347983.

    Change-Id: I367076c393e4a7cb356c66624d4402aa4d646f3d
    Closes-bug: #1528858

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/8.0)

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/260988

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/8.0)

Reviewed: https://review.openstack.org/260988
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=97143e36af04f9a79a3ea94564824543ff174fe7
Submitter: Jenkins
Branch: stable/8.0

commit 97143e36af04f9a79a3ea94564824543ff174fe7
Author: Artem Panchenko <email address hidden>
Date: Wed Dec 23 13:52:23 2015 +0000

    Revert "Test for postdeploy plugin install and enable."

    This reverts commit 04693990cb186a50fca817ff701160c924347983.

    Change-Id: I367076c393e4a7cb356c66624d4402aa4d646f3d
    Closes-bug: #1528858
    (cherry picked from commit 7e4be4fac9841b85b14b2bcfffea672bf683f288)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.