)]}'
{"neutron/tests/unit/plugins/ml2/test_plugin.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fe1405d0283aeb03417b0b32452503cd9d5d33f5","unresolved":false,"context_lines":[{"line_number":404,"context_line":""},{"line_number":405,"context_line":""},{"line_number":406,"context_line":"class TestMl2NetworksWithVlanTransparencyFailure(TestMl2NetworksV2):"},{"line_number":407,"context_line":"    _mechanism_drivers \u003d [\u0027openvswitch\u0027]"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    def setUp(self, plugin\u003dNone):"},{"line_number":410,"context_line":"        config.cfg.CONF.set_override(\u0027vlan_transparent\u0027, True)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_4b97aa47","line":407,"updated":"2016-07-13 16:01:50.000000000","message":"It wasn\u0027t clear initially why you did this, but after testing it\u0027s that the openvswitch driver doesn\u0027t support VLAN transparency.  Should add a comment to that effect.","commit_id":"6a3604e4d39193f43da13bad4900f34f531bd337"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fe1405d0283aeb03417b0b32452503cd9d5d33f5","unresolved":false,"context_lines":[{"line_number":416,"context_line":"                            [{pnet.NETWORK_TYPE: \u0027vlan\u0027,"},{"line_number":417,"context_line":"                              pnet.PHYSICAL_NETWORK: \u0027physnet1\u0027}],"},{"line_number":418,"context_line":"                            \u0027tenant_id\u0027: \u0027tenant_one\u0027,"},{"line_number":419,"context_line":"                            \u0027vlan_transparent\u0027: \u0027True\u0027}}"},{"line_number":420,"context_line":"        network_req \u003d self.new_create_request(\u0027networks\u0027, data)"},{"line_number":421,"context_line":"        res \u003d network_req.get_response(self.api)"},{"line_number":422,"context_line":"        self.assertEqual(500, res.status_int)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_eb0f76dd","line":419,"updated":"2016-07-13 16:01:50.000000000","message":"I think between this and the above (and below) classes we can reduce code duplication a bit.  Mind if I upload a new version?","commit_id":"6a3604e4d39193f43da13bad4900f34f531bd337"},{"author":{"_account_id":10652,"name":"Erik Colnick","email":"erikcolnick@gmail.com","username":"ecolnick"},"change_message_id":"6c188effac27c5c7fce5f31d78d784753f7a1f02","unresolved":false,"context_lines":[{"line_number":416,"context_line":"                            [{pnet.NETWORK_TYPE: \u0027vlan\u0027,"},{"line_number":417,"context_line":"                              pnet.PHYSICAL_NETWORK: \u0027physnet1\u0027}],"},{"line_number":418,"context_line":"                            \u0027tenant_id\u0027: \u0027tenant_one\u0027,"},{"line_number":419,"context_line":"                            \u0027vlan_transparent\u0027: \u0027True\u0027}}"},{"line_number":420,"context_line":"        network_req \u003d self.new_create_request(\u0027networks\u0027, data)"},{"line_number":421,"context_line":"        res \u003d network_req.get_response(self.api)"},{"line_number":422,"context_line":"        self.assertEqual(500, res.status_int)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_811b3372","line":419,"in_reply_to":"1aa78d24_eb0f76dd","updated":"2016-07-13 16:28:44.000000000","message":"I do not mind at all.  Any reduction in code duplication is always welcome.","commit_id":"6a3604e4d39193f43da13bad4900f34f531bd337"},{"author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"change_message_id":"977e70b70af5a4d082e2eae7c3d7be7a57eac1d8","unresolved":false,"context_lines":[{"line_number":396,"context_line":""},{"line_number":397,"context_line":"class TestMl2NetworksWithVlanTransparency("},{"line_number":398,"context_line":"    TestMl2NetworksWithVlanTransparencyBase):"},{"line_number":399,"context_line":"    # This will use the default mechanism_driver, which is openvswitch,"},{"line_number":400,"context_line":"    # and it does not support transparent VLANs"},{"line_number":401,"context_line":"    def test_create_network_vlan_transparent_fail(self):"},{"line_number":402,"context_line":"        network_req \u003d self.new_create_request(\u0027networks\u0027, self.data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"dada55a8_6e997e6e","line":399,"updated":"2016-07-25 06:16:06.000000000","message":"Is it possible to use the test mech driver: tests/unit/plugins/ml2/drivers/mechanism_test.py instead of the real drivers? The real drivers can change, and once they change their vlan transparent support, this test case needs to change.","commit_id":"71831ced00bcc55aff58e1bd61cd4c6237870917"},{"author":{"_account_id":10652,"name":"Erik Colnick","email":"erikcolnick@gmail.com","username":"ecolnick"},"change_message_id":"975be20a74958a747e30f6adb67758ee408b5d57","unresolved":false,"context_lines":[{"line_number":396,"context_line":""},{"line_number":397,"context_line":"class TestMl2NetworksWithVlanTransparency("},{"line_number":398,"context_line":"    TestMl2NetworksWithVlanTransparencyBase):"},{"line_number":399,"context_line":"    # This will use the default mechanism_driver, which is openvswitch,"},{"line_number":400,"context_line":"    # and it does not support transparent VLANs"},{"line_number":401,"context_line":"    def test_create_network_vlan_transparent_fail(self):"},{"line_number":402,"context_line":"        network_req \u003d self.new_create_request(\u0027networks\u0027, self.data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"dada55a8_84b39c4c","line":399,"in_reply_to":"dada55a8_6e997e6e","updated":"2016-07-26 15:15:12.000000000","message":"Done","commit_id":"71831ced00bcc55aff58e1bd61cd4c6237870917"}]}