tun_5_ctrl_ceph_ephemeral test failed trying to update public network cidr with list index out of range

Bug #1571543 reported by Andrey Sledzinskiy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Volodymyr Shypyguzov
6.1.x
Fix Released
High
Vladimir Jigulin
7.0.x
Confirmed
High
Vladimir Jigulin
8.0.x
Fix Released
High
Vladimir Jigulin
Mitaka
Fix Released
High
Volodymyr Shypyguzov

Bug Description

Steps:
1. Run fuel-qa tests with group tun_5_ctrl_ceph_ephemeral

Expected - test passed
Actual - test failed on updating public cidr
https://github.com/openstack/fuel-qa/blob/master/fuelweb_test/tests/tests_deployments/tests_neutron_tun/test_ha_tun_group_3.py#L202

Traceback (most recent call last):
  File "/usr/lib/python2.7/unittest/case.py", line 331, in run
    testMethod()
  File "/usr/lib/python2.7/unittest/case.py", line 1043, in runTest
    self._testFunc()
  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/9.0.acceptance.ubuntu.ha_tun_group_3/fuelweb_test/helpers/decorators.py", line 119, in wrapper
    result = func(*args, **kwargs)
  File "/home/jenkins/workspace/9.0.acceptance.ubuntu.ha_tun_group_3/fuelweb_test/tests/tests_deployments/tests_neutron_tun/test_ha_tun_group_3.py", line 196, in tun_5_ctrl_ceph_ephemeral
    self.fuel_web.update_network_cidr(cluster_id, 'public')
  File "/home/jenkins/workspace/9.0.acceptance.ubuntu.ha_tun_group_3/fuelweb_test/__init__.py", line 59, in wrapped
    result = func(*args, **kwargs)
  File "/home/jenkins/workspace/9.0.acceptance.ubuntu.ha_tun_group_3/fuelweb_test/models/fuel_web_client.py", line 2646, in update_network_cidr
    new_cidr = list(old_cidr.subnet(1))[0]
IndexError: list index out of range

Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Volodymyr Shypyguzov (vshypyguzov)
Changed in fuel:
status: New → Confirmed
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/307636

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/308154

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

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

commit e3d38cef19bbebee8e2a3ccb311a60bc1b6ad17c
Author: Volodymyr Shypyguzov <email address hidden>
Date: Tue Apr 19 11:30:20 2016 +0300

    Fix update_network_cidr method

    subnet() method of netaddr.IPNetwork expects size of subnets to be returned
    so we should pass to it subnet of the smaller size, not 1

    Change-Id: Id6b7e8e43419d224cc1c3202ffd253664245d5d4
    Closes-Bug: #1571543

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

Reviewed: https://review.openstack.org/308154
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=16a176232e5d465e3211a2a7bc48b2e87dd51c57
Submitter: Jenkins
Branch: stable/mitaka

commit 16a176232e5d465e3211a2a7bc48b2e87dd51c57
Author: Volodymyr Shypyguzov <email address hidden>
Date: Tue Apr 19 11:30:20 2016 +0300

    Fix update_network_cidr method

    subnet() method of netaddr.IPNetwork expects size of subnets to be returned
    so we should pass to it subnet of the smaller size, not 1

    Change-Id: Id6b7e8e43419d224cc1c3202ffd253664245d5d4
    Closes-Bug: #1571543

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-qa (master)

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

commit e35c3d8a9b6615f2a08e4db37f3af3283e9b2c58
Author: Volodymyr Shypyguzov <email address hidden>
Date: Fri Apr 22 10:25:13 2016 +0300

    Use prefixlen instead of parsing it from string

    +use builtin prefixlen method instead of parsing it from string

    Change-Id: I30f49e2c22ca6bdc5334848061054c6d8ac4fd51
    Related-bug: #1571543

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

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

Reviewed: https://review.openstack.org/344770
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=2fc4fcab0a42b6074fddc648cae6973bdc7c361f
Submitter: Jenkins
Branch: stable/8.0

commit 2fc4fcab0a42b6074fddc648cae6973bdc7c361f
Author: Volodymyr Shypyguzov <email address hidden>
Date: Tue Apr 19 11:30:20 2016 +0300

    Fix update_network_cidr method

    subnet() method of netaddr.IPNetwork expects size of subnets to be returned
    so we should pass to it subnet of the smaller size, not 1

    Change-Id: Id6b7e8e43419d224cc1c3202ffd253664245d5d4
    Closes-Bug: #1571543

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

Fix proposed to branch: stable/6.1
Review: https://review.openstack.org/347286

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-qa (stable/6.1)

Change abandoned by Vladimir Jigulin (<email address hidden>) on branch: stable/6.1
Review: https://review.openstack.org/347286
Reason: https://review.openstack.org/#/c/343296 cover this situation

tags: added: non-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.