)]}'
{"tempest/api/compute/v3/servers/test_server_actions.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a26fa69a5984e11c506a5ce4f65e7b5ca0b9a5de","unresolved":false,"context_lines":[{"line_number":417,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":418,"context_line":"    def test_get_vnc_console(self):"},{"line_number":419,"context_line":"        # Get the VNC console of type \u0027novnc\u0027"},{"line_number":420,"context_line":"        console_type \u003d \u0027novnc\u0027"},{"line_number":421,"context_line":"        resp, body \u003d self.servers_client.get_vnc_console(self.server_id,"},{"line_number":422,"context_line":"                                                         console_type)"},{"line_number":423,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVn%2F%2F%2Bu8%3D","line":420,"updated":"2014-03-03 01:18:55.000000000","message":"I think there is a little redundant here. How about like this?\n\n console_types \u003d [\u0027novnc\u0027, \u0027xvpvnc\u0027]\n for console_type in console_types:\n  resp, body \u003d self.servers_client.get_vnc_console(self.server_id,\n                                                   console_type)\n  self.assertEqual(200, resp.status)\n     :","commit_id":"12ac875f6d4b829a25468b0aad06224c9a0e0e0b"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"cddd07b7c3da7d390f3e43970bd81f7ec19e6964","unresolved":false,"context_lines":[{"line_number":417,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":418,"context_line":"    def test_get_vnc_console(self):"},{"line_number":419,"context_line":"        # Get the VNC console of type \u0027novnc\u0027"},{"line_number":420,"context_line":"        console_type \u003d \u0027novnc\u0027"},{"line_number":421,"context_line":"        resp, body \u003d self.servers_client.get_vnc_console(self.server_id,"},{"line_number":422,"context_line":"                                                         console_type)"},{"line_number":423,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVn%2F%2F%2Boc%3D","line":420,"in_reply_to":"AAAAVn%2F%2F%2Bu8%3D","updated":"2014-03-03 01:45:57.000000000","message":"Yes, I will correct it.","commit_id":"12ac875f6d4b829a25468b0aad06224c9a0e0e0b"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a26fa69a5984e11c506a5ce4f65e7b5ca0b9a5de","unresolved":false,"context_lines":[{"line_number":421,"context_line":"        resp, body \u003d self.servers_client.get_vnc_console(self.server_id,"},{"line_number":422,"context_line":"                                                         console_type)"},{"line_number":423,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":424,"context_line":"        self.assertEqual(body[\u0027type\u0027], console_type)"},{"line_number":425,"context_line":"        self.assertNotEqual(body[\u0027url\u0027], \u0027\u0027)"},{"line_number":426,"context_line":"        # Get the VNC console of type \u0027xvpvnc\u0027"},{"line_number":427,"context_line":"        console_type \u003d \u0027xvpvnc\u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVn%2F%2F%2Bvk%3D","line":424,"updated":"2014-03-03 01:18:55.000000000","message":"Oops, I\u0027ve noticed wrong order of assertEquals args.\nRight order of arguments is \"expected, actual\".\n\nPlease see bug 1277104 in details.\n\nThere are some dittos below.","commit_id":"12ac875f6d4b829a25468b0aad06224c9a0e0e0b"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"cddd07b7c3da7d390f3e43970bd81f7ec19e6964","unresolved":false,"context_lines":[{"line_number":421,"context_line":"        resp, body \u003d self.servers_client.get_vnc_console(self.server_id,"},{"line_number":422,"context_line":"                                                         console_type)"},{"line_number":423,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":424,"context_line":"        self.assertEqual(body[\u0027type\u0027], console_type)"},{"line_number":425,"context_line":"        self.assertNotEqual(body[\u0027url\u0027], \u0027\u0027)"},{"line_number":426,"context_line":"        # Get the VNC console of type \u0027xvpvnc\u0027"},{"line_number":427,"context_line":"        console_type \u003d \u0027xvpvnc\u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVn%2F%2F%2BoU%3D","line":424,"in_reply_to":"AAAAVn%2F%2F%2Bvk%3D","updated":"2014-03-03 01:45:57.000000000","message":"Yes, my bad. I will correct it.","commit_id":"12ac875f6d4b829a25468b0aad06224c9a0e0e0b"}],"tempest/config.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"d099b028da1cf3b7759eae3c08a04252ba086d1b","unresolved":false,"context_lines":[{"line_number":264,"context_line":"                default\u003dFalse,"},{"line_number":265,"context_line":"                help\u003d\"Does the test environment block migration support \""},{"line_number":266,"context_line":"                     \"cinder iSCSI volumes\"),"},{"line_number":267,"context_line":"    cfg.BoolOpt(\u0027vnc_console_enabled\u0027,"},{"line_number":268,"context_line":"                default\u003dFalse,"},{"line_number":269,"context_line":"                help\u003d\u0027Enable VNC console. This configuration value should\u0027"},{"line_number":270,"context_line":"                     \u0027be same as [nova.vnc]-\u003evnc_enabled in nova.conf\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVX%2F%2F0RQ%3D","line":267,"updated":"2014-02-28 04:10:33.000000000","message":"I think the word of \u0027_enabled\u0027 is redundant. So just \u0027vnc_console\u0027 is better here.","commit_id":"e0ddca96e8ffc251c52b6f54c21207be9e433a7c"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"d099b028da1cf3b7759eae3c08a04252ba086d1b","unresolved":false,"context_lines":[{"line_number":266,"context_line":"                     \"cinder iSCSI volumes\"),"},{"line_number":267,"context_line":"    cfg.BoolOpt(\u0027vnc_console_enabled\u0027,"},{"line_number":268,"context_line":"                default\u003dFalse,"},{"line_number":269,"context_line":"                help\u003d\u0027Enable VNC console. This configuration value should\u0027"},{"line_number":270,"context_line":"                     \u0027be same as [nova.vnc]-\u003evnc_enabled in nova.conf\u0027)"},{"line_number":271,"context_line":"]"},{"line_number":272,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVX%2F%2F0Xk%3D","line":269,"updated":"2014-02-28 04:10:33.000000000","message":"You need to add an extra space after \u0027should\u0027","commit_id":"e0ddca96e8ffc251c52b6f54c21207be9e433a7c"}]}