SRIOV trunk port - multiple vlans on same VF

[Munish]

Need to implement trunk port for SRIOV ports. There is existing trunk port API, the sriov agent need to gather the vlans and configure the nic with the list of vlans that are allowed to carry traffic to the VM bound to that VF. Some of the nic vendors provide the mechanism to allow multiple vlan filters on same VF.

[Miguel Lavalle (minsel)]

Let me understand what is entailed in this change:

1) Do we need to change the trunk port API?

2) Is the work restricted to the sr-iov agent?

[Slawek Kaplonski (slaweq)]

You said that it is supported only by some vendors, so do You plan to add some discovery of this feature and validation on API level if it can be used or just documenting it properly will be enough?

[Munish (mm6021)]

mlavalle,
1) I don't see any changes to API as of now, We need to see if trunk port has any limitation to do it for SRIOV ports, which i don't think has any issues, will check again.
2) Yes work should be in sr-iov agent from neutron perspective.

Slawek,

  I don't think we need any discovery, if the sr-iov is configured in the env, it already supports sr-iov ports. We can add validation check on agent side if the nic don't support the feature it will return error at bind time.

[Miguel Lavalle (minsel)]

ok, let's bring this up to the drivers team

[Miguel Lavalle (minsel)]

This RFE was discussed and approved for implementation on October 5th

[Munish (mm6021)]

https://review.opendev.org/#/c/665467/

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ocata
Review: https://review.opendev.org/677327

[Manjeet Singh Bhatia (manjeet-s-bhatia)]

so another thing here is trunk creation would not be allowed after vms are created with sriov ports because api doesn't allow trunk creation for bound ports, could be an issue for sriov

[Slawek Kaplonski (slaweq)]

This bug has had a related patch abandoned and has been automatically un-assigned due to inactivity. Please re-assign yourself if you are continuing work or adjust the state as appropriate if it is no longer valid.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ocata
Review: https://review.opendev.org/677327
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

[Slawek Kaplonski (slaweq)]

This bug has had a related patch abandoned and has been automatically un-assigned due to inactivity. Please re-assign yourself if you are continuing work or adjust the state as appropriate if it is no longer valid.

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Slawek Kaplonski <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/665467
Reason: This review is > 4 weeks without comment, and failed Zuul jobs the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

[Rodolfo Alonso (rodolfo-alonso-hernandez)]

Bug closed due to lack of activity, please feel free to reopen if needed.