)]}'
{"heat/engine/resources/openstack/nova/server.py":[{"author":{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},"change_message_id":"afebf51d97e4a621ac1afe0bcaf5a63a45298c11","unresolved":false,"context_lines":[{"line_number":939,"context_line":"                    self.NETWORK_ID, self.NETWORK_UUID)"},{"line_number":940,"context_line":"            else:"},{"line_number":941,"context_line":"                net_id \u003d self.client_plugin("},{"line_number":942,"context_line":"                    \u0027nova\u0027).get_nova_network_id(net.get(self.NETWORK_ID))"},{"line_number":943,"context_line":"        return net_id"},{"line_number":944,"context_line":""},{"line_number":945,"context_line":"    def update_networks_matching_iface_port(self, nets, interfaces):"}],"source_content_type":"text/x-python","patch_set":3,"id":"baa041b7_25a498a4","line":942,"updated":"2015-04-02 09:11:05.000000000","message":"This could just be self.client_plugin() or self.nova()\n\nThat actually highlights that we\u0027ve got inconsistencies all over the place around those interfaces, so I won\u0027t block on it.","commit_id":"d213a2414e26b8ce26838d16548b4cfabef20da1"}],"heat/tests/test_server.py":[{"author":{"_account_id":10487,"name":"Kanagaraj Manickam","email":"mkr1481@gmail.com","username":"kanagarajm"},"change_message_id":"50a04ecaee8233da6491e093e095006190006c95","unresolved":false,"context_lines":[{"line_number":2513,"context_line":"               \u0027fixed_ip\u0027: \u00271.2.3.4\u0027}"},{"line_number":2514,"context_line":"        self.patchobject(neutron.NeutronClientPlugin, \u0027resolve_network\u0027,"},{"line_number":2515,"context_line":"                         return_value\u003d\u0027f3ef5d2f-d7ba-4b27-af66-58ca0b81e032\u0027)"},{"line_number":2516,"context_line":"        net_id \u003d server._get_network_id(net)"},{"line_number":2517,"context_line":"        self.assertEqual(\u0027f3ef5d2f-d7ba-4b27-af66-58ca0b81e032\u0027, net_id)"},{"line_number":2518,"context_line":""},{"line_number":2519,"context_line":"    def test_get_network_matches_no_matching(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa041b7_208fff24","line":2516,"updated":"2015-04-01 17:44:40.000000000","message":"-1: could you kindly check whether you have added test cases for nova network client? as _get_network_id handles both clients. if so, kindly let me know , i will revert -1. thanks","commit_id":"25e657db44d98bc26b13ad76ec9dd5764c32d2b9"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"6b1756c45205df7eed5bff4ad5a763b3c7a9af61","unresolved":false,"context_lines":[{"line_number":2513,"context_line":"               \u0027fixed_ip\u0027: \u00271.2.3.4\u0027}"},{"line_number":2514,"context_line":"        self.patchobject(neutron.NeutronClientPlugin, \u0027resolve_network\u0027,"},{"line_number":2515,"context_line":"                         return_value\u003d\u0027f3ef5d2f-d7ba-4b27-af66-58ca0b81e032\u0027)"},{"line_number":2516,"context_line":"        net_id \u003d server._get_network_id(net)"},{"line_number":2517,"context_line":"        self.assertEqual(\u0027f3ef5d2f-d7ba-4b27-af66-58ca0b81e032\u0027, net_id)"},{"line_number":2518,"context_line":""},{"line_number":2519,"context_line":"    def test_get_network_matches_no_matching(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa041b7_6f2608bc","line":2516,"in_reply_to":"baa041b7_208fff24","updated":"2015-04-02 02:02:47.000000000","message":"Done. Thanks for noticing it.","commit_id":"25e657db44d98bc26b13ad76ec9dd5764c32d2b9"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"108ac1a2438d52a2487bf6acf0fbe8657833777b","unresolved":false,"context_lines":[{"line_number":2521,"context_line":"        self.patchobject(nova.NovaClientPlugin, \u0027get_nova_network_id\u0027,"},{"line_number":2522,"context_line":"                         return_value\u003d\u0027f3ef5d2f-d7ba-4b27-af66-58ca0b81e032\u0027)"},{"line_number":2523,"context_line":"        net_id \u003d server._get_network_id(net)"},{"line_number":2524,"context_line":"        self.assertEqual(\u0027f3ef5d2f-d7ba-4b27-af66-58ca0b81e032\u0027, net_id)"},{"line_number":2525,"context_line":""},{"line_number":2526,"context_line":"    def test_get_network_matches_no_matching(self):"},{"line_number":2527,"context_line":"        return_server \u003d self.fc.servers.list()[3]"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa041b7_3fa4a755","line":2524,"updated":"2015-04-02 08:09:53.000000000","message":"Add test for getting network id by name from nova client.","commit_id":"10aa4d600639c321abf310182891391ba872849a"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"d0b920bffdf7fbd7297586ddc123d118dc8f1de1","unresolved":false,"context_lines":[{"line_number":2521,"context_line":"        self.patchobject(nova.NovaClientPlugin, \u0027get_nova_network_id\u0027,"},{"line_number":2522,"context_line":"                         return_value\u003d\u0027f3ef5d2f-d7ba-4b27-af66-58ca0b81e032\u0027)"},{"line_number":2523,"context_line":"        net_id \u003d server._get_network_id(net)"},{"line_number":2524,"context_line":"        self.assertEqual(\u0027f3ef5d2f-d7ba-4b27-af66-58ca0b81e032\u0027, net_id)"},{"line_number":2525,"context_line":""},{"line_number":2526,"context_line":"    def test_get_network_matches_no_matching(self):"},{"line_number":2527,"context_line":"        return_server \u003d self.fc.servers.list()[3]"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa041b7_227dae1c","line":2524,"in_reply_to":"baa041b7_3fa4a755","updated":"2015-04-02 08:40:37.000000000","message":"It does not make much difference, but will add it.","commit_id":"10aa4d600639c321abf310182891391ba872849a"}]}