The OneView's driver is missing validate connections if using the OneView ML2 Driver.

[Hugo Nicodemos Brito]

Previously, the Ironic OneView driver validated if the node was using a correct MAC and if it was physical. Now, if the user is using the OneView ML2 Driver, a new validation of the local_link_connection field and pxe_enabled is needed.

[Vladyslav Drok (vdrok)]

The patch is here - https://review.openstack.org/377106.

Could you please comment on the questions raised there? After patch https://review.openstack.org/415881 ports without valid local_link_connection information will not be picked, and requests to do actions on such ports won't get to the ML2 driver. In that patch, we only validate that llc is not empty, because the actual content of it is validated on the api side.

[Vladyslav Drok (vdrok)]

OK, so the issue here is, the oneview ml2 driver relies on the information stored in switch_info, and you need a way to validate it. Maybe a thin wrapper around neutron network interface that would do the validation you need while not giving the impression that this is now required for all network interfaces is an option for you? I don't see other way to solve it ATM, so confirming.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Andre Aranha (<email address hidden>) on branch: master
Review: https://review.openstack.org/377106
Reason: python-oneviewclient will be migrated to hponeview, and this modification will be implemented there

[Julia Kreger (juliaashleykreger)]

Oneview was removed from Ironic.