)]}'
{"ironic/drivers/modules/ssh.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"95a7144c2f59d625b084dda7a1da4cc8756cc868","unresolved":false,"context_lines":[{"line_number":491,"context_line":"        cmd_to_exec \u003d \"%s %s\" % (driver_info[\u0027cmd_set\u0027][\u0027base_cmd\u0027],"},{"line_number":492,"context_line":"                                 driver_info[\u0027cmd_set\u0027][\u0027list_all\u0027])"},{"line_number":493,"context_line":"        full_node_list \u003d _ssh_execute(ssh_obj, cmd_to_exec)"},{"line_number":494,"context_line":"        LOG.debug(\"Retrieved Node List: %s\", repr(full_node_list))"},{"line_number":495,"context_line":"        # for each node check Mac Addresses"},{"line_number":496,"context_line":"        for node in full_node_list:"},{"line_number":497,"context_line":"            if not node:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_ed9f034e","line":494,"updated":"2016-11-09 10:09:19.000000000","message":"nit: this should use %r instad of an explicit repr() call","commit_id":"8b6894483b4d86189c48fbe414272761dbe5c933"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"aed1e0cfaef08e2c880bc79278a242fbf0036300","unresolved":false,"context_lines":[{"line_number":491,"context_line":"        cmd_to_exec \u003d \"%s %s\" % (driver_info[\u0027cmd_set\u0027][\u0027base_cmd\u0027],"},{"line_number":492,"context_line":"                                 driver_info[\u0027cmd_set\u0027][\u0027list_all\u0027])"},{"line_number":493,"context_line":"        full_node_list \u003d _ssh_execute(ssh_obj, cmd_to_exec)"},{"line_number":494,"context_line":"        LOG.debug(\"Retrieved Node List: %s\", repr(full_node_list))"},{"line_number":495,"context_line":"        # for each node check Mac Addresses"},{"line_number":496,"context_line":"        for node in full_node_list:"},{"line_number":497,"context_line":"            if not node:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_aa55e450","line":494,"in_reply_to":"9a629dbe_ed9f034e","updated":"2016-11-09 14:19:08.000000000","message":"Yes, but since ssh driver is deprecated and this repr() was already there, we shouldn\u0027t modify it.","commit_id":"8b6894483b4d86189c48fbe414272761dbe5c933"}]}
