Support Launching an instance with a port with vnic_type=direct

Bug #1402959 reported by Itzik Brown
26
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Medium
Akihiro Motoki

Bug Description

To support Launching instances with 'SR-IOV' interfaces using the dashboard there is a need to:

1)Adding the ability to specify vnic_type to 'port create' operation
2)Adding option to create a port as a tenant (Right now only Admin can do this)
3)Adding the ability to launch an instance with a pre configured port

Duplicate bugs:
https://bugs.launchpad.net/horizon/+bug/1399252
https://bugs.launchpad.net/horizon/+bug/1399254

Tags: neutron
Changed in horizon:
assignee: nobody → Nikunj Aggarwal (nikunj2512)
Changed in horizon:
status: New → Invalid
Revision history for this message
Itzik Brown (itzikb1) wrote :

Hi,
Can you please explain why do you consider it invalid?

Matthias Runge (mrunge)
Changed in horizon:
assignee: Nikunj Aggarwal (nikunj2512) → nobody
Matthias Runge (mrunge)
Changed in horizon:
status: Invalid → Confirmed
Revision history for this message
Rob Cresswell (robcresswell-deactivatedaccount) wrote :

My apologies; I think I must have misclicked during cleanup of multiple bugs.

Revision history for this message
Matthias Runge (mrunge) wrote :
Changed in horizon:
importance: Undecided → Medium
Itxaka Serrano (itxaka)
Changed in horizon:
assignee: nobody → Itxaka Serrano (itxakaserrano)
status: Confirmed → In Progress
Revision history for this message
Itxaka Serrano (itxaka) wrote :

 - Adding the ability to specify vnic_type to 'port create' operation

This is done and available on master already.

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

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

Revision history for this message
Itxaka Serrano (itxaka) wrote :

 - Adding the ability to launch an instance with a pre configured port

Should be converted into a BP IMO as its much bigger than a simple bug. Plus it needs to be done on both the old create instance and the new javascript one so it needs to be properly defined on how to show it, how would it work, what is expected and so on.

Revision history for this message
Itxaka Serrano (itxaka) wrote :
Revision history for this message
Itxaka Serrano (itxaka) wrote :

Added BP for the first part as well as this seems more in line with a new feature than a bug.

https://blueprints.launchpad.net/horizon/+spec/network-ports-tenant

Revision history for this message
Itxaka Serrano (itxaka) wrote :

https://blueprints.launchpad.net/horizon/+spec/sriov-support has been superseed.

Attaching ports to an instance on launch is not SR-IOV exclusive and actually has nothing to do with attaching ports or neutron or anything, its a configuration on nova + neutron which is left to the operator.

so first lets bring attaching ports to instance launch instead.
https://blueprints.launchpad.net/horizon/+spec/allow-launching-ports

Revision history for this message
Akihiro Motoki (amotoki) wrote :

Launching an instance with pre-created neutron port is already supported.
I think the remaining issue is to allow users (non-admin) to create a port by specifying vnic_type.

tags: added: neutron
Revision history for this message
Akihiro Motoki (amotoki) wrote :

@Itxaka, are you still working on this? If not, could you clear the assignee?

Revision history for this message
Itxaka Serrano (itxaka) wrote :

@Akihiro not anymore, let me clear the assignee :)

Changed in horizon:
assignee: Itxaka Serrano (itxaka) → nobody
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in horizon:
assignee: nobody → Akihiro Motoki (amotoki)
Akihiro Motoki (amotoki)
Changed in horizon:
milestone: none → queens-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/513887
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=09efe2d432c7a17c696e7240f69b3c76b481e3f3
Submitter: Zuul
Branch: master

commit 09efe2d432c7a17c696e7240f69b3c76b481e3f3
Author: Akihiro Motoki <email address hidden>
Date: Sat Oct 21 00:48:42 2017 +0000

    Allow regular users to specify VNIC type for port

    VNIC type needs to be specified when creating a server with an SR-IOV
    port. In this case, a user first creates a port with vnic_type=direct
    and creates a server with the created port. To allow this scenario,
    horizon needs to allow users to specify VNIC type for port.

    This commit also clean up the code duplication on the port security and
    mac state fields in the port create form. The code in the admin form for
    them are completely duplicated with that in the project form.

    Change-Id: Ib6e91ed7f720e2720994025429da0dcef2fa4e25
    Closes-Bug: #1402959

Changed in horizon:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/horizon 13.0.0.0b3

This issue was fixed in the openstack/horizon 13.0.0.0b3 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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