Invalid procedure to assign networks in case of enabled bonding

Bug #1670360 reported by Sergey Novikov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Ilya Bumarskov
Newton
Fix Committed
High
Ilya Bumarskov
Ocata
Fix Committed
High
Ilya Bumarskov

Bug Description

Detailed bug description:

All tests which are executed with injected env variable BONDING = true are affected by the issue.

As I understand, the root cause is in https://github.com/openstack/fuel-qa/blob/stable/newton/fuelweb_test/models/fuel_web_client.py#L3369-L3442 . These methods create invalid requiest's body, for example http://paste.openstack.org/show/601556/ , there is network "fuelweb_admin" that is assigned a twice to different ifaces

Changed in fuel:
assignee: nobody → Fuel QA Team (fuel-qa)
Changed in fuel:
status: New → Confirmed
tags: added: area-qa
summary: - [9.x swarm] Invalid procedure to assign networks in case of enabled
- bonding
+ Invalid procedure to assign networks in case of enabled bonding
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/445426

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

Fix proposed to branch: stable/ocata
Review: https://review.openstack.org/445450

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

Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

@snovikov, you are right, networks assignment is wrong here. The network should be assigned to bond itself, not to its slaves. And every network can be assigned just once on each node.

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

Reviewed: https://review.openstack.org/445426
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=199bcb1be6655711bdc68b7b431f8e4035e1d5e0
Submitter: Jenkins
Branch: stable/newton

commit 199bcb1be6655711bdc68b7b431f8e4035e1d5e0
Author: ibumarskov <email address hidden>
Date: Tue Mar 14 14:49:23 2017 +0400

    Fix vlan assignment for bond interface

    Change-Id: I44eac0c8a3ff9db3fb97d1f02de34cf9deb889e0
    Closes-bug: #1670360

Changed in fuel:
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/445451
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=682675c33e08377c1333fb33c80875865121c694
Submitter: Jenkins
Branch: master

commit 682675c33e08377c1333fb33c80875865121c694
Author: ibumarskov <email address hidden>
Date: Tue Mar 14 14:49:23 2017 +0400

    Fix vlan assignment for bond interface

    Change-Id: I44eac0c8a3ff9db3fb97d1f02de34cf9deb889e0
    Closes-bug: #1670360

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

Reviewed: https://review.openstack.org/445450
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=e02a4c0435d5ce6e503896e3540c4f776462b22a
Submitter: Jenkins
Branch: stable/ocata

commit e02a4c0435d5ce6e503896e3540c4f776462b22a
Author: ibumarskov <email address hidden>
Date: Tue Mar 14 14:49:23 2017 +0400

    Fix vlan assignment for bond interface

    Change-Id: I44eac0c8a3ff9db3fb97d1f02de34cf9deb889e0
    Closes-bug: #1670360

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-qa ocata-eol

This issue was fixed in the openstack/fuel-qa ocata-eol 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.