OpenStack Compute (nova)

No 'OS-EXT-VIF-NET' extension in v2.1

Bug #1470690 reported by Ghanshyam Mann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
High
Ghanshyam Mann
OpenStack Compute (nova) 12.0.0 "liberty"
Kilo
Fix Released
Undecided
Unassigned
OpenStack Compute (nova) 2015.1.3

Bug Description

V2 APi has extension for virtual interface 'OS-EXT-VIF-NET' but it is not present in v2.1 API.

Because of this there is difference between v2 and v2.1 response of virtual interface API.

v2 List virtual interface Response (with all extension enable)

{
    "virtual_interfaces": [
        {
            "id": "%(id)s",
            "mac_address": "%(mac_addr)s",
            "OS-EXT-VIF-NET:net_id": "%(id)s"
        }
    ]
}

v2.1 List virtual interface Response

{
    "virtual_interfaces": [
        {
            "id": "%(id)s",
            "mac_address": "%(mac_addr)s"
        }
    ]
}

As v2.1 is released in kilo, we should backport this fix to kilo branch also.

Ghanshyam Mann (ghanshyammann)
Changed in nova:
assignee: nobody → Ghanshyam Mann (ghanshyammann)
Alex Xu (xuhj)
Changed in nova:
status: New → Confirmed
importance: Undecided → High
OpenStack Infra (hudson-openstack)
Changed in nova:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/197850

John Garbutt (johngarbutt)
tags: added: kilo-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to nova (stable/kilo)

Related fix proposed to branch: stable/kilo
Review: https://review.openstack.org/198944

Revision history for this message
Ghanshyam Mann (ghanshyammann) wrote :

Spec to add VIF "net_id" as microversion- https://review.openstack.org/#/c/198622/5

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

Reviewed: https://review.openstack.org/198622
Committed: https://git.openstack.org/cgit/openstack/nova-specs/commit/?id=855811672c4c53cccfcc1bb6bdbc4be8704ccd3d
Submitter: Jenkins
Branch: master

commit 855811672c4c53cccfcc1bb6bdbc4be8704ccd3d
Author: ghanshyam <email address hidden>
Date: Mon Jul 6 16:21:20 2015 +0900

    Spec to add vif 'net-id' in virtual interfaces API

    APIImpact
    Partially Implements: blueprint add-vif-net-id-in-vif-list

    Related-Bug: 1470690

    Change-Id: I3caca80a6c010b86150909126f4545425ed99e11

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Ghanshyam Mann (<email address hidden>) on branch: master
Review: https://review.openstack.org/197822
Reason: Changes will be done as microversion in https://blueprints.launchpad.net/nova/+spec/add-vif-net-id-in-vif-list

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (stable/kilo)

Change abandoned by Ghanshyam Mann (<email address hidden>) on branch: stable/kilo
Review: https://review.openstack.org/197850
Reason: Changes will be done as microversion in https://blueprints.launchpad.net/nova/+spec/add-vif-net-id-in-vif-list

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

Reviewed: https://review.openstack.org/198934
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=3e98973787710c44ac408086f21acd6bcbc91cc3
Submitter: Jenkins
Branch: master

commit 3e98973787710c44ac408086f21acd6bcbc91cc3
Author: ghanshyam <email address hidden>
Date: Tue Jul 7 10:34:17 2015 +0900

    Removing extension "OS-EXT-VIF-NET" from v2.1 extension-list

    Extension "OS-EXT-VIF-NET" is not ported to v2.1, due to that
    there is difference between v2 and v2.1 as mentioned in bug#1470690.

    But v2.1 extension-list has "OS-EXT-VIF-NET" extension which convey that
    this extension is also loaded for v2.1 and user will get the expected response
    according to "OS-EXT-VIF-NET" extension which is not true as this extension
    is not actually ported to v2.1.

    It is decided that fix for that difference should go in microversion not
    in base v2.1 as it is already released.

    This patch remove extension "OS-EXT-VIF-NET" from v2.1 extension-list which
    will actually convey that this extension is missing in v2.1.

    Partial-Bug: 1470690

    Depends-On: I3caca80a6c010b86150909126f4545425ed99e11

    Change-Id: Ic99ac1179d02d907422911fe1369b64479fd5f33

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

Reviewed: https://review.openstack.org/198944
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=9a51140ba05e09be85390520f06d4e4204be9b53
Submitter: Jenkins
Branch: stable/kilo

commit 9a51140ba05e09be85390520f06d4e4204be9b53
Author: ghanshyam <email address hidden>
Date: Tue Jul 7 11:54:51 2015 +0900

    Kilo-Removing extension "OS-EXT-VIF-NET" from v2.1 ext list

    Extension "OS-EXT-VIF-NET" is not ported to v2.1, due to that
    there is difference between v2 and v2.1 as mentioned in bug#1470690.

    But v2.1 extension-list has "OS-EXT-VIF-NET" extension which convey that
    this extension is also loaded for v2.1 and user will get the expected response
    according to "OS-EXT-VIF-NET" extension which is not true as this extension
    is not actually ported to v2.1.

    It is decided that fix for that difference should go in microversion not
    in base v2.1 as it is already released.

    This patch remove extension "OS-EXT-VIF-NET" from v2.1 extension-list which
    will actually convey that this extension is missing in v2.1.

    This is back-porting Ic99ac1179d02d907422911fe1369b64479fd5f33 in Kilo stable
    branch.

    Partial-Bug: 1470690

    Change-Id: Ic590bb5df2b2df917af9ce9941da6afba4e382a5

tags: added: in-stable-kilo
Revision history for this message
John Garbutt (johngarbutt) wrote :

seems to have been fixed as part of:
https://blueprints.launchpad.net/nova/+spec/add-vif-net-id-in-vif-list

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → liberty-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: liberty-rc1 → 12.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.