create_cluster() failed because network provider is not always set by tests

Bug #1504451 reported by Dennis Dmitriev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Dennis Dmitriev

Bug Description

If a test case doesn't set network provider, create_cluster() failed with error:

...
 File "/home/jenkins/workspace/8.0.system_test.ubuntu.ceph_ha_one_controller/fuelweb_test/models/fuel_web_client.py", line 442, in create_cluster
    data['net_provider'],
KeyError: 'net_provider'

Reproduced on: https://product-ci.infra.mirantis.net/job/8.0.system_test.ubuntu.ceph_ha_one_controller/14/testReport/%28root%29/ceph_ha_one_controller_compact/ceph_ha_one_controller_compact/

Regression came from https://review.openstack.org/#/c/197628/. Behaviour for network provider will be changed in https://review.openstack.org/#/c/232414/, that will be merged after fixing the bug https://bugs.launchpad.net/fuel/+bug/1503657. So let's not change create_cluster() yet.

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/232965

Changed in fuel:
status: New → In Progress
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/232965
Committed: https://git.openstack.org/cgit/stackforge/fuel-qa/commit/?id=55bbc908f77384d78a682352735a0af5f3b6f54c
Submitter: Jenkins
Branch: master

commit 55bbc908f77384d78a682352735a0af5f3b6f54c
Author: Dennis Dmitriev <email address hidden>
Date: Fri Oct 9 11:59:19 2015 +0300

    Network provider is not always set by tests

    create_cluster() logic for network provider will be changed in
     https://review.openstack.org/#/c/232414/, so this is quick fix
    to return old behaviour and get system tests working.

    Change-Id: I1b3efacdd9fb945ddb3c6a5e019f192dbb09d2cc
    Closes-Bug:#1504451

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
tags: added: area-qa
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.