)]}'
{"tempest/common/utils/linux/remote_client.py":[{"author":{"_account_id":1420,"name":"Adam Gandelman","email":"adamg@ubuntu.com","username":"gandelman-a"},"change_message_id":"e624e766cd33bd2b480fcd28263a42da48304306","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        boot_time \u003d time.time() - int(boot_secs)"},{"line_number":82,"context_line":"        return time.localtime(boot_time)"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    def write_to_console(self, message, console_device\u003d\u0027/dev/console\u0027):"},{"line_number":85,"context_line":"        message \u003d re.sub(\"([$\\\\`])\", \"\\\\\\\\\\\\\\\\\\\\1\", message)"},{"line_number":86,"context_line":"        # usually to /dev/ttyS0"},{"line_number":87,"context_line":"        cmd \u003d \u0027sudo sh -c \"echo \\\\\"%s\\\\\" \u003e%s\"\u0027 % (message, console_device)"}],"source_content_type":"text/x-python","patch_set":5,"id":"da86d52c_f15c614d","line":84,"updated":"2015-02-05 08:50:35.000000000","message":"The only place this is called has been modified to specify console_device\u003d\u0027/dev/ttyS0\u0027.  Why not make that the default here?","commit_id":"68129be3ce24da317175d00fb3e1c6167f0a814b"},{"author":{"_account_id":1420,"name":"Adam Gandelman","email":"adamg@ubuntu.com","username":"gandelman-a"},"change_message_id":"3c83226134169bc9e325853aeafd81a742290933","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        boot_time \u003d time.time() - int(boot_secs)"},{"line_number":82,"context_line":"        return time.localtime(boot_time)"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    def write_to_console(self, message, console_device\u003d\u0027/dev/console\u0027):"},{"line_number":85,"context_line":"        message \u003d re.sub(\"([$\\\\`])\", \"\\\\\\\\\\\\\\\\\\\\1\", message)"},{"line_number":86,"context_line":"        # usually to /dev/ttyS0"},{"line_number":87,"context_line":"        cmd \u003d \u0027sudo sh -c \"echo \\\\\"%s\\\\\" \u003e%s\"\u0027 % (message, console_device)"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba7be1f8_9bf4295c","line":84,"updated":"2015-03-03 18:19:52.000000000","message":"The only place this is called has been modified to specify console_device\u003d\u0027/dev/ttyS0\u0027. Why not make that the default here?","commit_id":"4495d5af65d26fffe6a2383e575d54e158832264"}],"tempest/thirdparty/boto/test_ec2_instance_run.py":[{"author":{"_account_id":1420,"name":"Adam Gandelman","email":"adamg@ubuntu.com","username":"gandelman-a"},"change_message_id":"b549cf88b5505a0991dbba8de77c254548c8dedc","unresolved":false,"context_lines":[{"line_number":262,"context_line":"                                         CONF.compute.ssh_user,"},{"line_number":263,"context_line":"                                         pkey\u003dself.keypair.material)"},{"line_number":264,"context_line":"        text \u003d data_utils.rand_name(\"Pattern text for console output -\")"},{"line_number":265,"context_line":"        resp \u003d ssh.write_to_console(text, console_device\u003d\u0027/dev/ttyS0\u0027)"},{"line_number":266,"context_line":"        self.assertFalse(resp)"},{"line_number":267,"context_line":""},{"line_number":268,"context_line":"        def _output():"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_4127b864","line":265,"updated":"2015-01-21 23:52:42.000000000","message":"This changes the default.    I assume you probably want this exposed as a new config option (defaulting to /dev/console) and that value passed here.","commit_id":"85dc174999c1db523f58f7447d800ed3c40b2919"},{"author":{"_account_id":5280,"name":"Nicolas Simonds","email":"nic@metacloud.com","username":"nicolas.simonds"},"change_message_id":"e92febe92359f3434687c01dfa3ea91d16392ae2","unresolved":false,"context_lines":[{"line_number":262,"context_line":"                                         CONF.compute.ssh_user,"},{"line_number":263,"context_line":"                                         pkey\u003dself.keypair.material)"},{"line_number":264,"context_line":"        text \u003d data_utils.rand_name(\"Pattern text for console output -\")"},{"line_number":265,"context_line":"        resp \u003d ssh.write_to_console(text, console_device\u003d\u0027/dev/ttyS0\u0027)"},{"line_number":266,"context_line":"        self.assertFalse(resp)"},{"line_number":267,"context_line":""},{"line_number":268,"context_line":"        def _output():"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_101894d2","line":265,"in_reply_to":"1a930d6b_4127b864","updated":"2015-01-22 04:11:42.000000000","message":"I don\u0027t think so; in this case, the test is explicitly attempting to write to the console.log file for the instance, which prior to this patch is assumed to be /dev/console.  As per the related-bug, this is a quirky little CirrOS-ism that is, in fact, quite harmful to other Linuxes, and for the purposes on this test, the desired console device should be written to explicitly.\n\ntl;dr; this test is currently making an invalid assumption, and this test fixes that rather than allowing users to toggle bad behavior on/off.","commit_id":"85dc174999c1db523f58f7447d800ed3c40b2919"}]}