)]}'
{"tempest/thirdparty/boto/test_ec2_instance_run.py":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"5e1d545101917accc3a700bf69220d9ea5fb9a1b","unresolved":false,"context_lines":[{"line_number":288,"context_line":"        except Exception:"},{"line_number":289,"context_line":"            if not CONF.compute_feature_enabled.console_output:"},{"line_number":290,"context_line":"                LOG.debug(\u0027Console output not supported, cannot log\u0027)"},{"line_number":291,"context_line":"                return"},{"line_number":292,"context_line":"            console_output \u003d instance.get_console_output().output"},{"line_number":293,"context_line":"            LOG.debug(\u0027Console output for %s\\nbody\u003d\\n%s\u0027,"},{"line_number":294,"context_line":"                      instance.id, console_output)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_61750d31","line":291,"updated":"2015-05-26 05:04:13.000000000","message":"your actually changing the behavior of this test. With a simple return the test will pass even though it may fail to ssh in general. So it should be \"raise\".","commit_id":"0417096ab320cee6044022301cfbf03dbe4f6e18"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"8803683961e3aa9de65e2ad5fcb03f9ddaa6ab58","unresolved":false,"context_lines":[{"line_number":288,"context_line":"        except Exception:"},{"line_number":289,"context_line":"            if not CONF.compute_feature_enabled.console_output:"},{"line_number":290,"context_line":"                LOG.debug(\u0027Console output not supported, cannot log\u0027)"},{"line_number":291,"context_line":"                return"},{"line_number":292,"context_line":"            console_output \u003d instance.get_console_output().output"},{"line_number":293,"context_line":"            LOG.debug(\u0027Console output for %s\\nbody\u003d\\n%s\u0027,"},{"line_number":294,"context_line":"                      instance.id, console_output)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_a64bcd2a","line":291,"in_reply_to":"7a016987_61750d31","updated":"2015-05-27 19:24:27.000000000","message":"Oops, thanks","commit_id":"0417096ab320cee6044022301cfbf03dbe4f6e18"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"5e1d545101917accc3a700bf69220d9ea5fb9a1b","unresolved":false,"context_lines":[{"line_number":291,"context_line":"                return"},{"line_number":292,"context_line":"            console_output \u003d instance.get_console_output().output"},{"line_number":293,"context_line":"            LOG.debug(\u0027Console output for %s\\nbody\u003d\\n%s\u0027,"},{"line_number":294,"context_line":"                      instance.id, console_output)"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        self.assertFalse(resp)"},{"line_number":297,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_01089990","line":294,"updated":"2015-05-26 05:04:13.000000000","message":"Since your are already mentioning it in the commit message this code block is a duplicate from scenario manager. Can we move this functionality into RemoteClient to make it generally available?","commit_id":"0417096ab320cee6044022301cfbf03dbe4f6e18"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"8803683961e3aa9de65e2ad5fcb03f9ddaa6ab58","unresolved":false,"context_lines":[{"line_number":291,"context_line":"                return"},{"line_number":292,"context_line":"            console_output \u003d instance.get_console_output().output"},{"line_number":293,"context_line":"            LOG.debug(\u0027Console output for %s\\nbody\u003d\\n%s\u0027,"},{"line_number":294,"context_line":"                      instance.id, console_output)"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        self.assertFalse(resp)"},{"line_number":297,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_8621f1fe","line":294,"in_reply_to":"7a016987_01089990","updated":"2015-05-27 19:24:27.000000000","message":"I think not, because we need access to the instance object to get the console output, which also varies how to get it depending on whether it\u0027s an ec2 instance object or not. If you have a suggestion, let me know.","commit_id":"0417096ab320cee6044022301cfbf03dbe4f6e18"}]}