)]}'
{"tempest/api/compute/v3/servers/test_attach_interfaces.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"0788a017e05904fdffd5514e9ffe55efb394bc7c","unresolved":false,"context_lines":[{"line_number":128,"context_line":"        # Add and Remove the fixed IP to server."},{"line_number":129,"context_line":"        server, ifs \u003d self._create_server_get_interfaces()"},{"line_number":130,"context_line":"        interface_count \u003d len(ifs)"},{"line_number":131,"context_line":"        self.assertTrue(interface_count \u003e 0)"},{"line_number":132,"context_line":"        self._check_interface(ifs[0])"},{"line_number":133,"context_line":"        network_id \u003d ifs[0][\u0027net_id\u0027]"},{"line_number":134,"context_line":"        resp, body \u003d self.client.add_fixed_ip(server[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVn%2F%2F%2BkM%3D","line":131,"updated":"2014-03-03 02:01:52.000000000","message":"Can we use assertGreater() here?\nI think it may bring us good information when we will get an assert error here like this.\n INTERFACE_COUNT not greater than 0","commit_id":"bb55dce535b32d9b180d2f0dcdb171f90c38cce5"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"8c18a7e71e30dbf83c37ed0915aa5c8b0e278af8","unresolved":false,"context_lines":[{"line_number":128,"context_line":"        # Add and Remove the fixed IP to server."},{"line_number":129,"context_line":"        server, ifs \u003d self._create_server_get_interfaces()"},{"line_number":130,"context_line":"        interface_count \u003d len(ifs)"},{"line_number":131,"context_line":"        self.assertTrue(interface_count \u003e 0)"},{"line_number":132,"context_line":"        self._check_interface(ifs[0])"},{"line_number":133,"context_line":"        network_id \u003d ifs[0][\u0027net_id\u0027]"},{"line_number":134,"context_line":"        resp, body \u003d self.client.add_fixed_ip(server[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVn%2F%2FQJM%3D","line":131,"in_reply_to":"AAAAVn%2F%2F%2BkM%3D","updated":"2014-03-07 08:53:03.000000000","message":"@Masayuki,\n\nSorry for my late comment. but one comment for https://review.openstack.org/#/c/78507/\n\nassertGreater() is a little problematic because of I8d41a568ac4c9fc7330077e6643b40eddc071f91","commit_id":"bb55dce535b32d9b180d2f0dcdb171f90c38cce5"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3949a5e89e41e8d96388fe100ec05799547f6e8e","unresolved":false,"context_lines":[{"line_number":128,"context_line":"        # Add and Remove the fixed IP to server."},{"line_number":129,"context_line":"        server, ifs \u003d self._create_server_get_interfaces()"},{"line_number":130,"context_line":"        interface_count \u003d len(ifs)"},{"line_number":131,"context_line":"        self.assertTrue(interface_count \u003e 0)"},{"line_number":132,"context_line":"        self._check_interface(ifs[0])"},{"line_number":133,"context_line":"        network_id \u003d ifs[0][\u0027net_id\u0027]"},{"line_number":134,"context_line":"        resp, body \u003d self.client.add_fixed_ip(server[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVn%2F%2F%2BHo%3D","line":131,"in_reply_to":"AAAAVn%2F%2F%2BkM%3D","updated":"2014-03-03 03:48:49.000000000","message":"Done.","commit_id":"bb55dce535b32d9b180d2f0dcdb171f90c38cce5"}],"tempest/api/compute/v3/servers/test_server_actions.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"df2013eddbcca0a12bbd23005d5574c0cb0551b9","unresolved":false,"context_lines":[{"line_number":437,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":438,"context_line":"    def test_remove_fixed_ip(self):"},{"line_number":439,"context_line":"        resp, body \u003d self.servers_client.remove_fixed_ip(self.server_id,"},{"line_number":440,"context_line":"                                                         self.ip)"},{"line_number":441,"context_line":"        self.assertEqual(resp.status, 202)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVX%2F%2F%2FQg%3D","line":440,"updated":"2014-02-27 00:19:14.000000000","message":"the same comment as https://review.openstack.org/#/c/76091/","commit_id":"aafc24c0e85088dca7d358853feb7dac9098344b"}]}