)]}'
{"cinder/tests/unit/zonemanager/test_brcd_fc_san_lookup_service.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"309fc2dbc9f1cfa59f132a1caaf7425b31eb7e31","unresolved":false,"context_lines":[{"line_number":24,"context_line":"from oslo_config import cfg"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"from cinder import exception"},{"line_number":27,"context_line":"from cinder.ssh_utils import SSHPool"},{"line_number":28,"context_line":"from cinder import test"},{"line_number":29,"context_line":"from cinder.volume import configuration as conf"},{"line_number":30,"context_line":"import cinder.zonemanager.drivers.brocade.brcd_fc_san_lookup_service \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_cd16f2f0","line":27,"updated":"2015-08-28 15:17:04.000000000","message":"Please import only modules, not classes.\n\nfrom cinder import ssh_utils\n\nThen later:\n\nssh_utils.SSHPool","commit_id":"df2ce2c3fa8a5b15819d69ca6543c3a4477b2b49"}],"cinder/zonemanager/drivers/brocade/brcd_fc_san_lookup_service.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"309fc2dbc9f1cfa59f132a1caaf7425b31eb7e31","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from cinder import exception"},{"line_number":25,"context_line":"from cinder.i18n import _, _LE"},{"line_number":26,"context_line":"from cinder.ssh_utils import SSHPool"},{"line_number":27,"context_line":"from cinder import utils"},{"line_number":28,"context_line":"from cinder.zonemanager.drivers.brocade import brcd_fabric_opts as fabric_opts"},{"line_number":29,"context_line":"import cinder.zonemanager.drivers.brocade.fc_zone_constants as zone_constant"}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_cdc4b260","line":26,"updated":"2015-08-28 15:17:04.000000000","message":"Same.","commit_id":"df2ce2c3fa8a5b15819d69ca6543c3a4477b2b49"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"b615b62f38f3c0e64426aa115659ab82ebdba3a9","unresolved":false,"context_lines":[{"line_number":207,"context_line":"                switch_data, err \u003d processutils.ssh_execute(ssh, cmd)"},{"line_number":208,"context_line":"            except processutils.ProcessExecutionError as e:"},{"line_number":209,"context_line":"                msg \u003d (_(\"SSH Command failed with error \u0027%(err)s\u0027 \""},{"line_number":210,"context_line":"                         \"\u0027%(command)s\u0027\") % {\u0027err\u0027: six.text_type(e),"},{"line_number":211,"context_line":"                                             \u0027command\u0027: cmd})"},{"line_number":212,"context_line":"                LOG.error(msg)"},{"line_number":213,"context_line":"                raise exception.FCSanLookupServiceException(message\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_6d19e687","line":210,"updated":"2015-08-28 15:27:17.000000000","message":"Since we are changing this code I would like to update this error message.  Not a big fan of its current format:\n\n\"SSH Command failed with error: \u0027%(err)s\u0027  Command: \u0027%(command)s\u0027","commit_id":"df2ce2c3fa8a5b15819d69ca6543c3a4477b2b49"}]}