Nic Ordering not guaranteed with nova-network API

Bug #1170725 reported by Zhi Yan Liu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Zhi Yan Liu
Grizzly
Fix Released
Medium
Russell Bryant

Bug Description

The nova-network did not honor nic ordering, so nics were attached in db natural order. I will prepare the patch to sorts the networks honor equested nic ordering when Nova calls the nova-network API.

The related bug and fix about this for quantum api is https://bugs.launchpad.net/nova/+bug/1064524
Regarding quantum has replaced nova-network from havana, this bug and its fix is just for folsom (grizzly if needed).

Zhi Yan Liu (lzy-dev)
description: updated
Changed in nova:
assignee: nobody → Zhi Yan Liu (lzy-dev)
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

Zhi Yan Liu, Did you also see this bug? https://bugs.launchpad.net/nova/+bug/1156844

Revision history for this message
Zhi Yan Liu (lzy-dev) wrote :

@Dims, they are similar, I have not saw that before I report this. But for now, I don't think they are the same, I will fix it just like quantum fixing did (bug #1064524), that's keep "--nic net-id=<net1> --nic net-id=<net2>" order which be provided from CLI call/API call to provisioned instance vNIC order.
(actually I have already fixed it from our product internal)

Zhi Yan Liu (lzy-dev)
Changed in nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

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

Reviewed: https://review.openstack.org/27252
Committed: http://github.com/openstack/nova/commit/90d3371be6c7ec2fc85c92f4744adf4f83003522
Submitter: Jenkins
Branch: master

commit 90d3371be6c7ec2fc85c92f4744adf4f83003522
Author: Zhi Yan Liu <email address hidden>
Date: Mon Apr 22 16:18:54 2013 +0800

    Make nova-network support requested nic ordering

    The nova-network did not honor nic ordering, so nics were attached in db
    natural order. This patch sorts the networks honor requested nic
    ordering when Nova calls the nova-network API.

    Fixes bug: #1170725

    Change-Id: I7dd04ac7bdda95c626cbb0fe35be0aadacb4bd2a
    Signed-off-by: zhiyanl-cn <email address hidden>

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

Fix proposed to branch: stable/grizzly
Review: https://review.openstack.org/28312

tags: added: grizzly-backport-potential
tags: added: folsom-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/grizzly)

Reviewed: https://review.openstack.org/28312
Committed: http://github.com/openstack/nova/commit/be0a14a02428f2732c66af2b2272fa44b323787f
Submitter: Jenkins
Branch: stable/grizzly

commit be0a14a02428f2732c66af2b2272fa44b323787f
Author: Zhi Yan Liu <email address hidden>
Date: Mon Apr 22 16:18:54 2013 +0800

    Make nova-network support requested nic ordering

    The nova-network did not honor nic ordering, so nics were attached in db
    natural order. This patch sorts the networks honor requested nic
    ordering when Nova calls the nova-network API.

    Fixes bug: #1170725

    Change-Id: I7dd04ac7bdda95c626cbb0fe35be0aadacb4bd2a
    Signed-off-by: zhiyanl-cn <email address hidden>
    (cherry picked from commit 90d3371be6c7ec2fc85c92f4744adf4f83003522)

tags: added: in-stable-grizzly
Thierry Carrez (ttx)
Changed in nova:
milestone: none → havana-1
status: Fix Committed → Fix Released
tags: removed: grizzly-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/folsom)

Fix proposed to branch: stable/folsom
Review: https://review.openstack.org/31923

Alan Pevec (apevec)
tags: removed: in-stable-grizzly
Changed in nova:
importance: Undecided → Medium
Thierry Carrez (ttx)
Changed in nova:
milestone: havana-1 → 2013.2
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.