)]}'
{"rally/benchmark/scenarios/vm/utils.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"6ee911a19cf5461405d2c212f546735737054482","unresolved":false,"context_lines":[{"line_number":157,"context_line":"                                          script, is_file)"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"    @staticmethod"},{"line_number":160,"context_line":"    def _ping_ip_address(host):"},{"line_number":161,"context_line":"        ping \u003d \"ping\" if host.version \u003d\u003d 4 else \"ping6\""},{"line_number":162,"context_line":"        if sys.platform.startswith(\"linux\"):"},{"line_number":163,"context_line":"            cmd \u003d [ping, \"-c1\", \"-w1\", str(host)]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_653800f0","line":160,"updated":"2015-05-19 08:52:14.000000000","message":"Since `host\u0027 is expected to be a specific instance, please add docstring with param definition\n\n :param host: IPAddress","commit_id":"f08acf96beeadd9d33d2e1705db8d14cf4ad224d"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"4b9d3691a2f06b801b64936e6984d93af5435e22","unresolved":false,"context_lines":[{"line_number":157,"context_line":"                                          script, is_file)"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"    @staticmethod"},{"line_number":160,"context_line":"    def _ping_ip_address(host):"},{"line_number":161,"context_line":"        ping \u003d \"ping\" if host.version \u003d\u003d 4 else \"ping6\""},{"line_number":162,"context_line":"        if sys.platform.startswith(\"linux\"):"},{"line_number":163,"context_line":"            cmd \u003d [ping, \"-c1\", \"-w1\", str(host)]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_9b5178db","line":160,"in_reply_to":"7a016987_653800f0","updated":"2015-05-26 21:40:13.000000000","message":"Done","commit_id":"f08acf96beeadd9d33d2e1705db8d14cf4ad224d"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"6ee911a19cf5461405d2c212f546735737054482","unresolved":false,"context_lines":[{"line_number":170,"context_line":"        proc.wait()"},{"line_number":171,"context_line":"        LOG.debug(\"Host %s is ICMP %s\""},{"line_number":172,"context_line":"                  % (host, proc.returncode and \"down\" or \"up\"))"},{"line_number":173,"context_line":"        return \"ICMP UP\" if (proc.returncode \u003d\u003d 0) else \"ICMP DOWN\""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_05282c99","line":173,"updated":"2015-05-19 08:52:14.000000000","message":"Just recommendation: since \"ICMP UP\" is in use in at least two places, it seems reasonable to use constants","commit_id":"f08acf96beeadd9d33d2e1705db8d14cf4ad224d"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"4b9d3691a2f06b801b64936e6984d93af5435e22","unresolved":false,"context_lines":[{"line_number":170,"context_line":"        proc.wait()"},{"line_number":171,"context_line":"        LOG.debug(\"Host %s is ICMP %s\""},{"line_number":172,"context_line":"                  % (host, proc.returncode and \"down\" or \"up\"))"},{"line_number":173,"context_line":"        return \"ICMP UP\" if (proc.returncode \u003d\u003d 0) else \"ICMP DOWN\""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_7b5d9ceb","line":173,"in_reply_to":"7a016987_05282c99","updated":"2015-05-26 21:40:13.000000000","message":"Done","commit_id":"f08acf96beeadd9d33d2e1705db8d14cf4ad224d"}]}