Wrong naming of nova extension

Bug #1525387 reported by Sergey Kraynev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Won't Fix
Undecided
Unassigned

Bug Description

I use Heat for creating VMs, and one part of new Heat code (in stable/liberty and in master) uses nova extensions:

https://github.com/openstack/heat/blob/master/heat/engine/resources/openstack/nova/server_network_mixin.py#L188

I wanted to find this extension, but faced with following issue:
1. All nova documentation tells about "os-interface" extension instead of "os-attach-interfaces"
2. Also I founded, that nova code has some misunderstanding between alias and name:

https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/legacy_v2/contrib/attach_interfaces.py#L196

it has: alias = "os-attach-interfaces"

But in the same time uses different key for creating extension:

https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/legacy_v2/contrib/attach_interfaces.py#L203

os-interface

The same issue is in name option - it's AttachInterfaces

It looks like bug in naming, because another nova extension has clear naming without such confusion:
https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/legacy_v2/contrib/server_password.py#L54-L64

I'm not sure where it should be fixed: doc vs nova code.

P.s. if you will fix it in code, don't forget to backport it in stable/liberty too, please.

Revision history for this message
Andrew Laski (alaski) wrote :

The code can't really be changed without breaking users that rely on the current name. The docs should be updated to match the code.

Changed in nova:
status: New → Confirmed
Revision history for this message
Sean Dague (sdague) wrote :

This is a doc fix not a nova code fix.

Changed in nova:
status: Confirmed → Won't Fix
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.