)]}'
{"tempest/api/network/base.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"9649620d2b590ec662fe5f8643ff7d60bef895fc","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    def setUpClass(cls):"},{"line_number":50,"context_line":"        super(BaseNetworkTest, cls).setUpClass()"},{"line_number":51,"context_line":"        os \u003d clients.Manager()"},{"line_number":52,"context_line":"        os \u003d clients.Manager(interface\u003dcls._interface)"},{"line_number":53,"context_line":"        cls.os \u003d os"},{"line_number":54,"context_line":"        cls.network_cfg \u003d os.config.network"},{"line_number":55,"context_line":"        if not cls.config.service_available.neutron:"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAARn%2F%2F91M%3D","line":52,"updated":"2013-08-19 15:56:48.000000000","message":"This looks weird why do you create a Manager object and then create it again with an interface set?","commit_id":"1066c8b98348bfb7372bc516908edb623cc0bc5b"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"997db80e1460d07edbbfaab7c83798008b578961","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    @classmethod"},{"line_number":49,"context_line":"    def setUpClass(cls):"},{"line_number":50,"context_line":"        super(BaseNetworkTest, cls).setUpClass()"},{"line_number":51,"context_line":"        os \u003d clients.Manager(interface\u003dcls._interface)"},{"line_number":52,"context_line":"        cls.os \u003d os"},{"line_number":53,"context_line":"        cls.network_cfg \u003d os.config.network"},{"line_number":54,"context_line":"        if not cls.config.service_available.neutron:"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAARn%2F%2F6XY%3D","line":51,"updated":"2013-08-20 10:18:25.000000000","message":"can\u0027t we just set cls.os here in one line?","commit_id":"29b5c350f201399a4b45d0b4fa01972c339f9675"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"4a787e463646b07be898adffe19411abe6abfec0","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    @classmethod"},{"line_number":49,"context_line":"    def setUpClass(cls):"},{"line_number":50,"context_line":"        super(BaseNetworkTest, cls).setUpClass()"},{"line_number":51,"context_line":"        os \u003d clients.Manager(interface\u003dcls._interface)"},{"line_number":52,"context_line":"        cls.os \u003d os"},{"line_number":53,"context_line":"        cls.network_cfg \u003d os.config.network"},{"line_number":54,"context_line":"        if not cls.config.service_available.neutron:"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAARn%2F%2F5p8%3D","line":51,"updated":"2013-08-20 13:41:27.000000000","message":"I think you could just set cls.os and drop line 52","commit_id":"4e4d825fe1abcf1debccb8b96441f457581808b8"}],"tempest/api/network/test_networks.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"6a64445e43d6a569b62c957ada3b58f863aba262","unresolved":false,"context_lines":[{"line_number":116,"context_line":"    def test_list_networks(self):"},{"line_number":117,"context_line":"        # Verify the network exists in the list of all networks"},{"line_number":118,"context_line":"        resp, body \u003d self.client.list_networks()"},{"line_number":119,"context_line":"        networks \u003d body[\u0027networks\u0027]"},{"line_number":120,"context_line":"        found \u003d None"},{"line_number":121,"context_line":"        for n in networks:"},{"line_number":122,"context_line":"            if (n[\u0027id\u0027] \u003d\u003d self.network[\u0027id\u0027]):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAARH%2F%2FzAA%3D","side":"PARENT","line":119,"updated":"2013-08-17 12:26:15.000000000","message":"The xml version must be able to work without removing this part.","commit_id":"02f5bbe01ba335a61f081f2eb216aa23956c5d1d"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"6a64445e43d6a569b62c957ada3b58f863aba262","unresolved":false,"context_lines":[{"line_number":137,"context_line":"    def test_list_subnets(self):"},{"line_number":138,"context_line":"        # Verify the subnet exists in the list of all subnets"},{"line_number":139,"context_line":"        resp, body \u003d self.client.list_subnets()"},{"line_number":140,"context_line":"        subnets \u003d body[\u0027subnets\u0027]"},{"line_number":141,"context_line":"        found \u003d None"},{"line_number":142,"context_line":"        for n in subnets:"},{"line_number":143,"context_line":"            if (n[\u0027id\u0027] \u003d\u003d self.subnet[\u0027id\u0027]):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAARH%2F%2Fy%2F8%3D","side":"PARENT","line":140,"updated":"2013-08-17 12:26:15.000000000","message":"The xml version must be able to work without removing this part.","commit_id":"02f5bbe01ba335a61f081f2eb216aa23956c5d1d"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"6a64445e43d6a569b62c957ada3b58f863aba262","unresolved":false,"context_lines":[{"line_number":174,"context_line":"        # Verify the port exists in the list of all ports"},{"line_number":175,"context_line":"        resp, body \u003d self.client.list_ports()"},{"line_number":176,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":177,"context_line":"        ports_list \u003d body[\u0027ports\u0027]"},{"line_number":178,"context_line":"        found \u003d None"},{"line_number":179,"context_line":"        for n in ports_list:"},{"line_number":180,"context_line":"            if (n[\u0027id\u0027] \u003d\u003d self.port[\u0027id\u0027]):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAARH%2F%2Fy%2F4%3D","side":"PARENT","line":177,"updated":"2013-08-17 12:26:15.000000000","message":"The xml version must be able to work without removing this part.","commit_id":"02f5bbe01ba335a61f081f2eb216aa23956c5d1d"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"6a64445e43d6a569b62c957ada3b58f863aba262","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright 2012 OpenStack, LLC"},{"line_number":4,"context_line":"# All Rights Reserved."}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAARH%2F%2Fy%2F0%3D","line":1,"updated":"2013-08-17 12:26:15.000000000","message":"Unintended withe space removal.","commit_id":"292053b49a9fc77da5e60fcf5dc1c279af482236"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"9649620d2b590ec662fe5f8643ff7d60bef895fc","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":135,"context_line":"        networks \u003d body[\u0027networks\u0027]"},{"line_number":136,"context_line":"        self.assertIn(self.network[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], networks))"},{"line_number":137,"context_line":"        self.assertIn(self.network[\u0027name\u0027], map(lambda x: x[\u0027name\u0027], networks))"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":140,"context_line":"    def test_show_subnet(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAARn%2F%2F9zY%3D","line":137,"updated":"2013-08-19 15:56:48.000000000","message":"and here","commit_id":"1066c8b98348bfb7372bc516908edb623cc0bc5b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"9649620d2b590ec662fe5f8643ff7d60bef895fc","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        self.assertIn(self.subnet[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], subnets))"},{"line_number":155,"context_line":"        self.assertIn(self.subnet[\u0027name\u0027], map(lambda x: x[\u0027name\u0027], subnets))"},{"line_number":156,"context_line":"        self.assertIn(self.subnet[\u0027network_id\u0027],"},{"line_number":157,"context_line":"                      map(lambda x: x[\u0027network_id\u0027], subnets))"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":160,"context_line":"    def test_create_update_delete_port(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAARn%2F%2F9zw%3D","line":157,"updated":"2013-08-19 15:56:48.000000000","message":"Same as below...","commit_id":"1066c8b98348bfb7372bc516908edb623cc0bc5b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"9649620d2b590ec662fe5f8643ff7d60bef895fc","unresolved":false,"context_lines":[{"line_number":173,"context_line":"        updated_port \u003d body[\u0027port\u0027]"},{"line_number":174,"context_line":"        self.assertEqual(updated_port[\u0027name\u0027], new_port)"},{"line_number":175,"context_line":"        # Verification of port delete"},{"line_number":176,"context_line":"        resp, body \u003d self.client.delete_port(port_id)"},{"line_number":177,"context_line":"        self.assertEqual(\u0027204\u0027, resp[\u0027status\u0027])"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"    @attr(type\u003d\u0027gate\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAARn%2F%2F9zg%3D","line":176,"updated":"2013-08-19 15:56:48.000000000","message":"Same here","commit_id":"1066c8b98348bfb7372bc516908edb623cc0bc5b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"9649620d2b590ec662fe5f8643ff7d60bef895fc","unresolved":false,"context_lines":[{"line_number":193,"context_line":"        self.assertIn(self.port[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], ports))"},{"line_number":194,"context_line":"        self.assertIn(self.port[\u0027name\u0027], map(lambda x: x[\u0027name\u0027], ports))"},{"line_number":195,"context_line":"        self.assertIn(self.port[\u0027network_id\u0027],"},{"line_number":196,"context_line":"                      map(lambda x: x[\u0027network_id\u0027], ports))"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":199,"context_line":"    def test_show_non_existent_network(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAARn%2F%2F9z0%3D","line":196,"updated":"2013-08-19 15:56:48.000000000","message":"While I like this clean up what does it have to do with XML support?","commit_id":"1066c8b98348bfb7372bc516908edb623cc0bc5b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"9649620d2b590ec662fe5f8643ff7d60bef895fc","unresolved":false,"context_lines":[{"line_number":210,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":211,"context_line":"    def test_show_non_existent_port(self):"},{"line_number":212,"context_line":"        non_exist_id \u003d rand_name(\u0027port\u0027)"},{"line_number":213,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.show_port,"},{"line_number":214,"context_line":"                          non_exist_id)"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    @attr(type\u003d\u0027gate\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAARn%2F%2F9y8%3D","line":213,"updated":"2013-08-19 15:56:48.000000000","message":"why are you adding a new test? I thought this commit was for adding xml support?","commit_id":"1066c8b98348bfb7372bc516908edb623cc0bc5b"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"class NetworksTestJSON(base.BaseNetworkTest):"},{"line_number":27,"context_line":"    _interface \u003d \u0027json\u0027"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    \"\"\""},{"line_number":30,"context_line":"    Tests the following operations in the Neutron API using the REST client for"},{"line_number":31,"context_line":"    Neutron:"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6wM%3D","line":28,"updated":"2013-08-20 08:58:22.000000000","message":"This 2 line is required.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        network update"},{"line_number":45,"context_line":"        subnet update"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"     v2.0 of the Neutron API is assumed. It is also assumed that the following"},{"line_number":48,"context_line":"     options are defined in the [network] section of etc/tempest.conf:"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        tenant_network_cidr with a block of cidr\u0027s from which smaller blocks"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6wA%3D","line":47,"updated":"2013-08-20 08:58:22.000000000","message":"Unrelated change.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    @classmethod"},{"line_number":58,"context_line":"    def setUpClass(cls):"},{"line_number":59,"context_line":"        super(NetworksTestJSON, cls).setUpClass()"},{"line_number":60,"context_line":"        cls.network \u003d cls.create_network()"},{"line_number":61,"context_line":"        cls.name \u003d cls.network[\u0027name\u0027]"},{"line_number":62,"context_line":"        cls.subnet \u003d cls.create_subnet(cls.network)"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6v0%3D","line":59,"updated":"2013-08-20 08:58:22.000000000","message":"this line is required.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        cls.name \u003d cls.network[\u0027name\u0027]"},{"line_number":62,"context_line":"        cls.subnet \u003d cls.create_subnet(cls.network)"},{"line_number":63,"context_line":"        cls.cidr \u003d cls.subnet[\u0027cidr\u0027]"},{"line_number":64,"context_line":"        p_name \u003d rand_name(\u0027port-\u0027)"},{"line_number":65,"context_line":"        cls.port \u003d cls.create_port(cls.network, p_name)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def _delete_networks(self, created_networks):"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6s0%3D","line":64,"updated":"2013-08-20 08:58:22.000000000","message":"Can it postponed to the second change ?","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        network \u003d body[\u0027network\u0027]"},{"line_number":86,"context_line":"        net_id \u003d network[\u0027id\u0027]"},{"line_number":87,"context_line":"        # Verification of network update"},{"line_number":88,"context_line":"        new_net \u003d \"New_Network\""},{"line_number":89,"context_line":"        resp, body \u003d self.client.update_network(net_id, new_net)"},{"line_number":90,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":91,"context_line":"        updated_net \u003d body[\u0027network\u0027]"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6vs%3D","line":88,"updated":"2013-08-20 08:58:22.000000000","message":"Unrelated change.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        resp, body \u003d self.client.update_network(net_id, new_net)"},{"line_number":90,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":91,"context_line":"        updated_net \u003d body[\u0027network\u0027]"},{"line_number":92,"context_line":"        self.assertEqual(updated_net[\u0027name\u0027], new_net)"},{"line_number":93,"context_line":"        # Find a cidr that is not in use yet and create a subnet with it"},{"line_number":94,"context_line":"        cidr \u003d netaddr.IPNetwork(self.network_cfg.tenant_network_cidr)"},{"line_number":95,"context_line":"        mask_bits \u003d self.network_cfg.tenant_network_mask_bits"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6vo%3D","line":92,"updated":"2013-08-20 08:58:22.000000000","message":"Unrelated change.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        subnet \u003d body[\u0027subnet\u0027]"},{"line_number":107,"context_line":"        subnet_id \u003d subnet[\u0027id\u0027]"},{"line_number":108,"context_line":"        # Verification of subnet update"},{"line_number":109,"context_line":"        new_subnet \u003d \"New_Subnet\""},{"line_number":110,"context_line":"        resp, body \u003d self.client.update_subnet(subnet_id, new_subnet)"},{"line_number":111,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":112,"context_line":"        updated_subnet \u003d body[\u0027subnet\u0027]"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6vc%3D","line":109,"updated":"2013-08-20 08:58:22.000000000","message":"Not related change.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":130,"context_line":"    def test_list_networks(self):"},{"line_number":131,"context_line":"        # Verify the network exists in the list of all networks"},{"line_number":132,"context_line":"        resp, body \u003d self.client.list_networks()"},{"line_number":133,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":134,"context_line":"        networks \u003d body[\u0027networks\u0027]"},{"line_number":135,"context_line":"        found \u003d None"},{"line_number":136,"context_line":"        for n in networks:"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6vQ%3D","line":133,"updated":"2013-08-20 08:58:22.000000000","message":"Unrelated change.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":152,"context_line":"    def test_list_subnets(self):"},{"line_number":153,"context_line":"        # Verify the subnet exists in the list of all subnets"},{"line_number":154,"context_line":"        resp, body \u003d self.client.list_subnets()"},{"line_number":155,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":156,"context_line":"        subnets \u003d body[\u0027subnets\u0027]"},{"line_number":157,"context_line":"        found \u003d None"},{"line_number":158,"context_line":"        for n in subnets:"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6vM%3D","line":155,"updated":"2013-08-20 08:58:22.000000000","message":"Not related change.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":165,"context_line":"    def test_create_update_delete_port(self):"},{"line_number":166,"context_line":"        # Verify that successful port creation, update \u0026 deletion"},{"line_number":167,"context_line":"        rand_port \u003d rand_name(\u0027port-\u0027)"},{"line_number":168,"context_line":"        resp, body \u003d self.client.create_port(self.network[\u0027id\u0027],"},{"line_number":169,"context_line":"                                             name\u003drand_port,"},{"line_number":170,"context_line":"                                             admin_state_up\u003d\u0027True\u0027)"},{"line_number":171,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6u8%3D","line":168,"updated":"2013-08-20 08:58:22.000000000","message":"Can it be moved to the next change ?","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":194,"context_line":"        # Verify the port exists in the list of all ports"},{"line_number":195,"context_line":"        resp, body \u003d self.client.list_ports()"},{"line_number":196,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":197,"context_line":"        ports \u003d body[\u0027ports\u0027]"},{"line_number":198,"context_line":"        found \u003d None"},{"line_number":199,"context_line":"        for n in ports:"},{"line_number":200,"context_line":"            if (n[\u0027id\u0027] \u003d\u003d self.port[\u0027id\u0027]):"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6s8%3D","line":197,"updated":"2013-08-20 08:58:22.000000000","message":"Unrelated change.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"5dc7f3e7a28bd53100b2423598929c7d41972570","unresolved":false,"context_lines":[{"line_number":230,"context_line":"            self.assertIsNotNone(n[\u0027id\u0027])"},{"line_number":231,"context_line":"            self.assertIn(n[\u0027id\u0027], networks_list)"},{"line_number":232,"context_line":""},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"class NetworksTestXML(NetworksTestJSON):"},{"line_number":235,"context_line":"    _interface \u003d \u0027xml\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAARn%2F%2F6us%3D","line":233,"updated":"2013-08-20 08:58:22.000000000","message":"Required change.","commit_id":"771f5f47d2b42322dc2698b5db6bcf77aa2ce605"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"997db80e1460d07edbbfaab7c83798008b578961","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        resp, body \u003d self.client.create_port(self.network[\u0027id\u0027])"},{"line_number":167,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":168,"context_line":"        port \u003d body[\u0027port\u0027]"},{"line_number":169,"context_line":"        port_id \u003d port[\u0027id\u0027]"},{"line_number":170,"context_line":"        # Verification of port update"},{"line_number":171,"context_line":"        new_port \u003d \"New_Port\""},{"line_number":172,"context_line":"        resp, body \u003d self.client.update_port(port_id, new_port)"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAARn%2F%2F6W4%3D","line":169,"updated":"2013-08-20 10:18:25.000000000","message":"I didn\u0027t like this much, isn\u0027t it just complicating it comparing to the original code?","commit_id":"29b5c350f201399a4b45d0b4fa01972c339f9675"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"4a787e463646b07be898adffe19411abe6abfec0","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        resp, body \u003d self.client.create_port(self.network[\u0027id\u0027])"},{"line_number":167,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":168,"context_line":"        port \u003d body[\u0027port\u0027]"},{"line_number":169,"context_line":"        port_id \u003d port[\u0027id\u0027]"},{"line_number":170,"context_line":"        # Verification of port update"},{"line_number":171,"context_line":"        new_port \u003d \"New_Port\""},{"line_number":172,"context_line":"        resp, body \u003d self.client.update_port(port_id, new_port)"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAARn%2F%2F5pU%3D","line":169,"updated":"2013-08-20 13:41:27.000000000","message":"I think this additions and changes at lines 172 and 177 are unneeded","commit_id":"4e4d825fe1abcf1debccb8b96441f457581808b8"}],"tempest/services/network/json/network_client.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"9649620d2b590ec662fe5f8643ff7d60bef895fc","unresolved":false,"context_lines":[{"line_number":115,"context_line":"            }"},{"line_number":116,"context_line":"        }"},{"line_number":117,"context_line":"        for key, val in kwargs.items():"},{"line_number":118,"context_line":"            post_body[\u0027port\u0027][key] \u003d val"},{"line_number":119,"context_line":"        body \u003d json.dumps(post_body)"},{"line_number":120,"context_line":"        uri \u003d \u0027%s/ports\u0027 % (self.uri_prefix)"},{"line_number":121,"context_line":"        resp, body \u003d self.post(uri, headers\u003dself.headers, body\u003dbody)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAARn%2F%2F91I%3D","line":118,"updated":"2013-08-19 15:56:48.000000000","message":"This is out of scope for adding xml support to the network tests.","commit_id":"1066c8b98348bfb7372bc516908edb623cc0bc5b"}]}