MAAS

Code review comment for ~cgrabowski/maas:do_not_create_duplicate_vlan_on_controller_report_neighbors

  1. Git
  2. lp:~cgrabowski/maas
  3. do_not_create_duplicate_vlan_on_controller_report_neighbors
  4. Merge into master
Revision history for this message
Christian Grabowski (cgrabowski) wrote :

> I think the other testcases in TestReportVID must be updated also, because it
> looks like `neighbour["ip"]` won't be None usually. If you do this change, I
> think test_creates_vlan_if_necessary() will fail

Ah good point, updated.

« Back to merge proposal