tempest-sahara-tests job is broken again

Bug #1501347 reported by Andrey Pavlov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Sahara
Fix Released
Medium
Andrey Pavlov
Liberty
Fix Released
Medium
Andrey Pavlov
Mitaka
Fix Released
Medium
Andrey Pavlov

Bug Description

tempest-sahara-tests fails with the following trace:

Captured traceback:
~~~~~~~~~~~~~~~~~~~
    Traceback (most recent call last):
      File "tempest/test.py", line 274, in setUpClass
        six.reraise(etype, value, trace)
      File "tempest/test.py", line 267, in setUpClass
        cls.resource_setup()
      File "/usr/local/lib/python2.7/dist-packages/sahara/tests/tempest/scenario/data_processing/client_tests/base.py", line 59, in resource_setup
        cls.floating_ip_pool = cls.get_floating_ip_pool_id_for_neutron()
      File "/usr/local/lib/python2.7/dist-packages/sahara/tests/tempest/scenario/data_processing/client_tests/base.py", line 117, in get_floating_ip_pool_id_for_neutron
        TEMPEST_CONF.network.floating_network_name)
      File "/usr/local/lib/python2.7/dist-packages/sahara/tests/tempest/scenario/data_processing/client_tests/base.py", line 137, in _find_network_by_name
        if network['label'] == network_name:
    TypeError: string indices must be integers

Changed in sahara:
assignee: nobody → Andrey Pavlov (apavlov-n)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to sahara (master)

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

Changed in sahara:
status: New → In Progress
Changed in sahara:
importance: Undecided → Medium
milestone: none → mitaka-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to sahara (master)

Reviewed: https://review.openstack.org/229442
Committed: https://git.openstack.org/cgit/openstack/sahara/commit/?id=8ad5858052e1d5d7179f8d9c2a8b8ffa98676ab4
Submitter: Jenkins
Branch: master

commit 8ad5858052e1d5d7179f8d9c2a8b8ffa98676ab4
Author: Andrey Pavlov <email address hidden>
Date: Wed Sep 30 16:56:32 2015 +0300

    Fix of client tests in tempest

    python-saharaclient tests in tempest were fixed
    and documentation updated

    Change-Id: I6a5964b8ecd1e343c427ae7aefdadac5cfa26ad9
    Closes-bug: #1501347

Changed in sahara:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to sahara (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/241199

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to sahara (stable/liberty)

Reviewed: https://review.openstack.org/241199
Committed: https://git.openstack.org/cgit/openstack/sahara/commit/?id=a94404701555c01bc1d41f20b5682ad9ce556717
Submitter: Jenkins
Branch: stable/liberty

commit a94404701555c01bc1d41f20b5682ad9ce556717
Author: Andrey Pavlov <email address hidden>
Date: Wed Sep 30 16:56:32 2015 +0300

    Fix of client tests in tempest

    python-saharaclient tests in tempest were fixed
    and documentation updated

    Change-Id: I6a5964b8ecd1e343c427ae7aefdadac5cfa26ad9
    Closes-bug: #1501347
    (cherry picked from commit 8ad5858052e1d5d7179f8d9c2a8b8ffa98676ab4)

Changed in sahara:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to sahara-tests (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to sahara-tests (master)
Download full text (9.0 KiB)

Reviewed: https://review.openstack.org/357664
Committed: https://git.openstack.org/cgit/openstack/sahara-tests/commit/?id=e163f7925974ce7a74e29cbf83db7fb00103e470
Submitter: Jenkins
Branch: master

commit 87bae8585033bf371a936d2de330a89d791244ad
Author: Matthew Treinish <email address hidden>
Date: Tue Jun 28 11:53:22 2016 -0400

    Remove unecessary decorators from private interface

    This commit removes the usage of the services decorators from the
    tempest plugin tests. That decorator is an internal interface and
    not meant for external consumption, and is now causing failures
    because of recent tempest changes. Plugins should only use documented
    stable interfaces. [1]

    Also the use of the decorator here actually adds nothing to the tests.
    They do 2 things, add a test attr to filter by service and add a skip
    check on the service availability. However both of these are already
    done, data_processing is already part of the test_id so filtering
    will just work and the skip check for sahara is baked into the base
    test class. So using this decorator does nothing except break the
    plugin.

    [1] http://docs.openstack.org/developer/tempest/plugin.html#stable-tempest-apis-plugins-may-use

    Change-Id: I45587d065c1b7b7f28feb29e8845f8f0f8405b8c

commit f4db8224967267e91abb05dd9c526318589f383b
Author: ZhiQiang Fan <email address hidden>
Date: Mon Apr 18 22:00:47 2016 +0800

    update options mentioned in tempest readme

    Some options have already been deprecated but still used in tempest
    README.rst, let's use the new one.

    Change-Id: Ic3bfb66b0898869149126c21aef0b2fd673385e2

commit f82c7db143bb7bf3c358e735f9646a17684ba5b6
Author: Luigi Toscano <email address hidden>
Date: Fri Feb 26 11:24:32 2016 +0100

    Use the integrated tempest.lib module

    Tempest-lib has been reintegrated into tempest (as code, but it still provides
    a stable API). The old library is still available, but tempest tests are
    failing because when sahara is installed and exports the scenario tests
    for python clients, python-lib is not installed (it's correcly in
    test-requirements.txt, not requirements.txt), and tempest does not
    install tempest-lib anymore. Hence, fix the tests to use the new
    official location.

    Change-Id: Iaded03ee387b4a0bbabb7bfe9e54d38e010d9c4f

commit 85ce728189d8f8a63843ebde5950a0c56233fa3a
Author: luhuichun <email address hidden>
Date: Tue Dec 1 14:09:22 2015 +0800

    Replacing all hard coded cluster status using cluster_utils

    Change-Id: I9550e98553da9f7bfc57923e44a789e07a912ff4
    Closes-bug: 1517061

commit 0abdc18a94089bf698582daa051a96fd645935f3
Author: Evgeny Sikachev <email address hidden>
Date: Thu Nov 26 13:40:51 2015 +0300

    Add missed checks for testing update method

    In tempest tests for saharaclient we have missed methods
    for checking update of templates.
    Closes-bug: #1520140

    Change-Id: I8191e3344ecffd7a2ed442ba3d561741701c45e2

commit 80bfb9a469d2a289e903b19488214855b8a7fa94
Author: Evgeny Sikachev <email address hidden>
Date: Tue Nov 17 16:05:04 2015 +0300

    Adding t...

Read more...

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/sahara-tests 0.3.0

This issue was fixed in the openstack/sahara-tests 0.3.0 release.

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.