[Starlingx-discuss] Analysis of patch 88b7bc7 for StartlingX upstreaming

[Xu, Chenjie]

Hi Matt,
I think that there exists a bug in patch 88b7bc7. Function _get_fdb_entries will call function _create_agent_fdb by following line:
https://github.com/starlingx-staging/stx-neutron/blob/master/neutron/plugins/ml2/drivers/l2pop/mech_driver.py#L441
The variable "host" will be used to call _create_agent_fdb. However function _create_agent_fdb expects variable "agent":
https://github.com/starlingx-staging/stx-neutron/blob/master/neutron/plugins/ml2/drivers/l2pop/mech_driver.py#L264
I think this should be a bug. Could you please help review and comment?

Best Regards,
Xu, Chenjie
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.starlingx.io/pipermail/starlingx-discuss/attachments/20181128/269bd9b4/attachment.html>