network_data.json shouldn't include internal device name and type

Bug #1588017 reported by Mathieu Gagné on 2016-06-01
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Wishlist
Unassigned

Bug Description

Nova currently injects in network_data.json the internal device name in the link id field and the device "type" in the link type field. This is wrong and could mislead users about how they should use and interpret those values.

The nova-specs for network_data.json [1] specifies that:
* the link id should be a "Generic, generated ID".
* the link type should be "vif" if it's a virtual interface and "phy" for physical interface.

The values currently used come from internal implementation details and shouldn't be passed on to the end-user. The implementation should be updated to respect the spirit of the spec. This will reduce confusion to initialization agent developers.

[1] http://specs.openstack.org/openstack/nova-specs/specs/liberty/implemented/metadata-service-network-info.html#rest-api-impact

Mathieu Gagné (mgagne) on 2016-06-01
Changed in nova:
assignee: nobody → Mathieu Gagné (mgagne)

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

Changed in nova:
status: New → In Progress

Change abandoned by Michael Still (<email address hidden>) on branch: master
Review: https://review.openstack.org/324054
Reason: This patch has been sitting unchanged for more than 12 weeks. I am therefore going to abandon it to keep the nova review queue sane. Please feel free to restore the change if you're still working on it.

Sean Dague (sdague) wrote :

Automatically discovered version liberty in description. If this is incorrect, please update the description to include 'nova version: ...'

tags: added: openstack-version.liberty
Sean Dague (sdague) wrote :

There are no currently open reviews on this bug, changing the status back to the previous state and unassigning. If there are active reviews related to this bug, please include links in comments.

Changed in nova:
status: In Progress → New
assignee: Mathieu Gagné (mgagne) → nobody
Sean Dague (sdague) 21 hours ago
Changed in nova:
status: New → Confirmed
importance: Undecided → Wishlist
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers