)]}'
{"nova/tests/unit/virt/ironic/test_driver.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"963fcf9ac060a3496756bf380aee5f997e633fdd","unresolved":false,"context_lines":[{"line_number":1391,"context_line":"        mock_lp.assert_called_once_with(node_uuid)"},{"line_number":1392,"context_line":"        calls \u003d (mock.call(first_ironic_port_uuid,"},{"line_number":1393,"context_line":"                           [{\u0027op\u0027: \u0027add\u0027, \u0027path\u0027: \u0027/extra/vif_port_id\u0027,"},{"line_number":1394,"context_line":"                             \u0027value\u0027: first_vif_id}]),"},{"line_number":1395,"context_line":"                 mock.call(second_ironic_port_uuid,"},{"line_number":1396,"context_line":"                           [{\u0027op\u0027: \u0027add\u0027, \u0027path\u0027: \u0027/extra/vif_port_id\u0027,"},{"line_number":1397,"context_line":"                             \u0027value\u0027: second_vif_id}]))"}],"source_content_type":"text/x-python","patch_set":5,"id":"5aef4532_ad107b02","line":1394,"range":{"start_line":1394,"start_character":38,"end_line":1394,"end_character":50},"updated":"2016-03-09 02:02:43.000000000","message":"first_vif_id has an address of 22:FF:FF:FF:FF:FF which matches the address in second_port, so why would this call be using the first port and the first vif?  Seems it should be first vif and second_port.","commit_id":"e147a63cb396ebaadd7c841a305cf3a52be1d373"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6f974ece6ac9f2767fefc20b4d1ec466e7ba0a1f","unresolved":false,"context_lines":[{"line_number":1391,"context_line":"        mock_lp.assert_called_once_with(node_uuid)"},{"line_number":1392,"context_line":"        calls \u003d (mock.call(first_ironic_port_uuid,"},{"line_number":1393,"context_line":"                           [{\u0027op\u0027: \u0027add\u0027, \u0027path\u0027: \u0027/extra/vif_port_id\u0027,"},{"line_number":1394,"context_line":"                             \u0027value\u0027: first_vif_id}]),"},{"line_number":1395,"context_line":"                 mock.call(second_ironic_port_uuid,"},{"line_number":1396,"context_line":"                           [{\u0027op\u0027: \u0027add\u0027, \u0027path\u0027: \u0027/extra/vif_port_id\u0027,"},{"line_number":1397,"context_line":"                             \u0027value\u0027: second_vif_id}]))"}],"source_content_type":"text/x-python","patch_set":5,"id":"5aef4532_4dc0d7e0","line":1394,"range":{"start_line":1394,"start_character":38,"end_line":1394,"end_character":50},"in_reply_to":"5aef4532_2d50eb5c","updated":"2016-03-09 02:21:43.000000000","message":"Yeah, I saw that later, it\u0027s confusing, but I guess it works.","commit_id":"e147a63cb396ebaadd7c841a305cf3a52be1d373"},{"author":{"_account_id":6610,"name":"Zhenguo Niu","email":"Niu.ZGlinux@gmail.com","username":"niu-zglinux"},"change_message_id":"c178a1d3fb6e08878c55899398a7aa42c6953912","unresolved":false,"context_lines":[{"line_number":1391,"context_line":"        mock_lp.assert_called_once_with(node_uuid)"},{"line_number":1392,"context_line":"        calls \u003d (mock.call(first_ironic_port_uuid,"},{"line_number":1393,"context_line":"                           [{\u0027op\u0027: \u0027add\u0027, \u0027path\u0027: \u0027/extra/vif_port_id\u0027,"},{"line_number":1394,"context_line":"                             \u0027value\u0027: first_vif_id}]),"},{"line_number":1395,"context_line":"                 mock.call(second_ironic_port_uuid,"},{"line_number":1396,"context_line":"                           [{\u0027op\u0027: \u0027add\u0027, \u0027path\u0027: \u0027/extra/vif_port_id\u0027,"},{"line_number":1397,"context_line":"                             \u0027value\u0027: second_vif_id}]))"}],"source_content_type":"text/x-python","patch_set":5,"id":"5aef4532_2d50eb5c","line":1394,"range":{"start_line":1394,"start_character":38,"end_line":1394,"end_character":50},"in_reply_to":"5aef4532_ad107b02","updated":"2016-03-09 02:17:55.000000000","message":"hi Matt, please see the port creation parameters, we create first_vif with address \u002722:FF:FF:FF:FF:FF\u0027 but second_vif_id, so in fact fist_vif_id has an address of \u002711:FF:FF:FF:FF:FF\u0027.","commit_id":"e147a63cb396ebaadd7c841a305cf3a52be1d373"}]}