yahoo-eng-team team mailing list archive

[Sylvain Bauza <sbauza@xxxxxxx>]

Sorry, but technically I don't see a bug here, rather some behaviour
that should be modified, right?

I mean, you're providing support for detaching an interface in the
ironic driver, that's not a bug then.

If so, please follow the existing process where you should fill in a blueprint and ask for a spec-less implementation, that should be enough I guess.
http://docs.openstack.org/developer/nova/process.html#how-do-i-get-my-code-merged


** Changed in: nova
       Status: New => Invalid

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1606231

Title:
  [RFE] Support nova virt interface attach/detach

Status in Ironic:
  Confirmed
Status in OpenStack Compute (nova):
  Invalid

Bug description:
  Steps to reproduce:
  1. Get list of attached ports of instance:
  nova interface-list 42dd8b8b-b2bc-420e-96b6-958e9295b2d4
  +------------+--------------------------------------+--------------------------------------+-----------------------------------------------+-------------------+
  | Port State | Port ID                              | Net ID                               | IP addresses                                  | MAC Addr          |
  +------------+--------------------------------------+--------------------------------------+-----------------------------------------------+-------------------+
  | ACTIVE     | 512e6c8e-3829-4bbd-8731-c03e5d7f7639 | ccd0fd43-9cc3-4544-b17c-dfacd8fa4d14 | 10.1.0.6,fdea:fd32:11ff:0:f816:3eff:fed1:8a7c | 52:54:00:85:19:89 |
  +------------+--------------------------------------+--------------------------------------+-----------------------------------------------+-------------------+
  2. Show ironic port. it has vif_port_id in extra with id of neutron port:
  ironic port-show 735fcaf5-145d-4125-8701-365c58c6b796
  +-----------------------+-----------------------------------------------------------+
  | Property              | Value                                                     |
  +-----------------------+-----------------------------------------------------------+
  | address               | 52:54:00:85:19:89                                         |
  | created_at            | 2016-07-20T13:15:23+00:00                                 |
  | extra                 | {u'vif_port_id': u'512e6c8e-3829-4bbd-8731-c03e5d7f7639'} |
  | local_link_connection |                                                           |
  | node_uuid             | 679fa8a9-066e-4166-ac1e-6e77af83e741                      |
  | pxe_enabled           |                                                           |
  | updated_at            | 2016-07-22T13:31:29+00:00                                 |
  | uuid                  | 735fcaf5-145d-4125-8701-365c58c6b796                      |
  +-----------------------+-----------------------------------------------------------+
  3. Delete neutron port:
  neutron port-delete 512e6c8e-3829-4bbd-8731-c03e5d7f7639
  Deleted port: 512e6c8e-3829-4bbd-8731-c03e5d7f7639
  4. It is done from interface list:
  nova interface-list 42dd8b8b-b2bc-420e-96b6-958e9295b2d4
  +------------+---------+--------+--------------+----------+
  | Port State | Port ID | Net ID | IP addresses | MAC Addr |
  +------------+---------+--------+--------------+----------+
  +------------+---------+--------+--------------+----------+
  5. ironic port still has vif_port_id with neutron's port id:
  ironic port-show 735fcaf5-145d-4125-8701-365c58c6b796
  +-----------------------+-----------------------------------------------------------+
  | Property              | Value                                                     |
  +-----------------------+-----------------------------------------------------------+
  | address               | 52:54:00:85:19:89                                         |
  | created_at            | 2016-07-20T13:15:23+00:00                                 |
  | extra                 | {u'vif_port_id': u'512e6c8e-3829-4bbd-8731-c03e5d7f7639'} |
  | local_link_connection |                                                           |
  | node_uuid             | 679fa8a9-066e-4166-ac1e-6e77af83e741                      |
  | pxe_enabled           |                                                           |
  | updated_at            | 2016-07-22T13:31:29+00:00                                 |
  | uuid                  | 735fcaf5-145d-4125-8701-365c58c6b796                      |
  +-----------------------+-----------------------------------------------------------+

  This can confuse when user wants to get list of unused ports of ironic node.
  vif_port_id should be removed after neutron port-delete.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ironic/+bug/1606231/+subscriptions