[System Tests] Wrong initialization of OpenStackActions instance in upgrade_ha test

Bug #1402537 reported by Andrey Sledzinskiy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Andrey Sledzinskiy

Bug Description

ERROR: Upgrade ha deployed cluster
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/proboscis/case.py", line 296, in testng_method_mistake_capture_func
    compatability.capture_type_error(s_func)
  File "/usr/lib/python2.7/dist-packages/proboscis/compatability/exceptions_2_6.py", line 27, in capture_type_error
    func()
  File "/usr/lib/python2.7/dist-packages/proboscis/case.py", line 350, in func
    func(test_case.state.get_state())
  File "/home/jenkins/workspace/6.0.system_test.CentOS.deploy_neutron_gre_ha.upgrade/UPGRADE/fuelweb_test/helpers/decorators.py", line 52, in wrapper
    return func(*args, **kwagrs)
  File "/home/jenkins/workspace/6.0.system_test.CentOS.deploy_neutron_gre_ha.upgrade/UPGRADE/fuelweb_test/tests/test_upgrade.py", line 165, in upgrade_ha_env
    self.fuel_web.get_public_vip(cluster_id),
  File "/home/jenkins/workspace/6.0.system_test.CentOS.deploy_neutron_gre_ha.upgrade/UPGRADE/fuelweb_test/models/fuel_web_client.py", line 1275, in get_public_vip
    return self.client.get_networks(cluster_id)['public_vip']
KeyError: 'public_vip'

In upgrade_ha_env test we have such openstack initialization
os_conn = os_actions.OpenStackActions(
            self.fuel_web.get_public_vip(cluster_id),
            data['user'], data['password'], data['tenant'])

But it's simple cluster and it should look like
os_conn = os_actions.OpenStackActions(
            self.fuel_web.get_nailgun_node_by_name('slave-06')['ip'],
            data['user'], data['password'], data['tenant'])

Tags: system-tests
Revision history for this message
Andrey Sledzinskiy (asledzinskiy) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-main (master)

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

Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Andrey Sledzinskiy (asledzinskiy)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-main (stable/6.0)

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

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

Reviewed: https://review.openstack.org/141744
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=19d64368fec69bbfe36ea2a324a6e06c6b889511
Submitter: Jenkins
Branch: master

commit 19d64368fec69bbfe36ea2a324a6e06c6b889511
Author: asledzinskiy <email address hidden>
Date: Mon Dec 15 10:43:06 2014 +0200

    Fix upgrade_ha_env test

    - OpenStackActions instance was initialized with public vip
    of controller but cluster mode is simple and there is no
    public vip. So now we are passing ip of the controller that
    is taking from devops

    Change-Id: I19eae689519979a3374c4e526940c5d953d69e44
    Closes-Bug: #1402537

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (stable/6.0)

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

commit 052e7506dd0a54a0bab1114d1fd40fb7e490ba36
Author: asledzinskiy <email address hidden>
Date: Mon Dec 15 10:43:06 2014 +0200

    Fix upgrade_ha_env test

    - OpenStackActions instance was initialized with public vip
    of controller but cluster mode is simple and there is no
    public vip. So now we are passing ip of the controller that
    is taking from devops

    Change-Id: I19eae689519979a3374c4e526940c5d953d69e44
    Closes-Bug: #1402537

Changed in fuel:
status: Fix Committed → Fix Released
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.