)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"64553d6d927d73ecc38e38dd653b32a34176eecf","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Elena Ezhova \u003ceezhova@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-04-09 17:12:49 +0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":" Improve test_load_balancer_basic"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch introduces multiple improvements of test_load_balancer_basic."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAW3%2F%2FrcU%3D","line":7,"updated":"2014-04-11 07:53:08.000000000","message":"I\u0027m curious why you put a white space at the head of the subject.\nVery nit though.","commit_id":"4a27b4623fe22be119ea7f4e10f37df6eb3b7186"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"abff9205d173118c7ff16ac87331942a0b0769b5","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Elena Ezhova \u003ceezhova@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-04-09 17:12:49 +0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":" Improve test_load_balancer_basic"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch introduces multiple improvements of test_load_balancer_basic."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAW3%2F%2FqYc%3D","line":7,"in_reply_to":"AAAAW3%2F%2FrcU%3D","updated":"2014-04-11 11:11:51.000000000","message":"Thanks, I\u0027ll remove it case I\u0027ll have to upload a new patchset.","commit_id":"4a27b4623fe22be119ea7f4e10f37df6eb3b7186"}],"tempest/common/utils/linux/remote_client.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"64553d6d927d73ecc38e38dd653b32a34176eecf","unresolved":false,"context_lines":[{"line_number":115,"context_line":""},{"line_number":116,"context_line":"    def get_pids(self, pr_name):"},{"line_number":117,"context_line":"        # Get pid(s) of a process/program"},{"line_number":118,"context_line":"        cmd \u003d \"ps -ef | grep %s | grep -v \u0027grep\u0027 | awk {\u0027print $1\u0027}\" % pr_name"},{"line_number":119,"context_line":"        return self.exec_command(cmd).split(\u0027\\n\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAW3%2F%2FrZ8%3D","line":118,"updated":"2014-04-11 07:53:08.000000000","message":"Can we use pgrep here?","commit_id":"4a27b4623fe22be119ea7f4e10f37df6eb3b7186"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"abff9205d173118c7ff16ac87331942a0b0769b5","unresolved":false,"context_lines":[{"line_number":115,"context_line":""},{"line_number":116,"context_line":"    def get_pids(self, pr_name):"},{"line_number":117,"context_line":"        # Get pid(s) of a process/program"},{"line_number":118,"context_line":"        cmd \u003d \"ps -ef | grep %s | grep -v \u0027grep\u0027 | awk {\u0027print $1\u0027}\" % pr_name"},{"line_number":119,"context_line":"        return self.exec_command(cmd).split(\u0027\\n\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAW3%2F%2FqYA%3D","line":118,"in_reply_to":"AAAAW3%2F%2FrZ8%3D","updated":"2014-04-11 11:11:51.000000000","message":"I\u0027m afraid there is no pgrep on cirros","commit_id":"4a27b4623fe22be119ea7f4e10f37df6eb3b7186"}],"tempest/scenario/test_load_balancer_basic.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"64553d6d927d73ecc38e38dd653b32a34176eecf","unresolved":false,"context_lines":[{"line_number":187,"context_line":"            pids \u003d ssh_client.get_pids(\u0027inetd\u0027)"},{"line_number":188,"context_line":"            if pids !\u003d [\u0027\u0027]:"},{"line_number":189,"context_line":"                # If there are any inetd processes, reload them"},{"line_number":190,"context_line":"                kill_cmd \u003d \"sudo kill -HUP %s\" % \u0027 \u0027.join(pids)"},{"line_number":191,"context_line":"                ssh_client.exec_command(kill_cmd)"},{"line_number":192,"context_line":"            else:"},{"line_number":193,"context_line":"                # In other case start inetd"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAW3%2F%2FrYE%3D","line":190,"updated":"2014-04-11 07:53:08.000000000","message":"Can we use pkill here? If yes, we can remove the get_pids().","commit_id":"4a27b4623fe22be119ea7f4e10f37df6eb3b7186"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"abff9205d173118c7ff16ac87331942a0b0769b5","unresolved":false,"context_lines":[{"line_number":187,"context_line":"            pids \u003d ssh_client.get_pids(\u0027inetd\u0027)"},{"line_number":188,"context_line":"            if pids !\u003d [\u0027\u0027]:"},{"line_number":189,"context_line":"                # If there are any inetd processes, reload them"},{"line_number":190,"context_line":"                kill_cmd \u003d \"sudo kill -HUP %s\" % \u0027 \u0027.join(pids)"},{"line_number":191,"context_line":"                ssh_client.exec_command(kill_cmd)"},{"line_number":192,"context_line":"            else:"},{"line_number":193,"context_line":"                # In other case start inetd"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAW3%2F%2FqX0%3D","line":190,"in_reply_to":"AAAAW3%2F%2FrYE%3D","updated":"2014-04-11 11:11:51.000000000","message":"And there is no pkill on cirros either.","commit_id":"4a27b4623fe22be119ea7f4e10f37df6eb3b7186"}]}