)]}'
{"tempest/scenario/test_network_basic_ops.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7a60969954e3a02a0f0118a0b3869e3ffd389eaa","unresolved":false,"context_lines":[{"line_number":213,"context_line":"            name \u003d data_utils.rand_name(\u0027server-smoke-%d-\u0027 % i)"},{"line_number":214,"context_line":"            self._create_server(name, network)"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    def _log_console_output(self):"},{"line_number":217,"context_line":"        for server, key in self.servers.items():"},{"line_number":218,"context_line":"            LOG.debug(\u0027Console output for %s\u0027, server.id)"},{"line_number":219,"context_line":"            LOG.debug(server.get_console_output())"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2Fk4U%3D","line":216,"updated":"2014-01-10 21:06:20.000000000","message":"I think we should move this to the manager.py base test class and take servers as an argument instead. \n\nThere are other tests that perform ssh into the guest (some of the volume tests for example) So this can probably be useful for debugging those tests in the future too. You don\u0027t need to enable it for those tests, but I feel making it easier to enable for them is probably something to do upfront.","commit_id":"4fb7ce6bcfd5d7a3f4571cfe594456ec85e5f32e"},{"author":{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},"change_message_id":"6df1044593a4c63966921413b7f607f2588d7329","unresolved":false,"context_lines":[{"line_number":213,"context_line":"            name \u003d data_utils.rand_name(\u0027server-smoke-%d-\u0027 % i)"},{"line_number":214,"context_line":"            self._create_server(name, network)"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    def _log_console_output(self):"},{"line_number":217,"context_line":"        for server, key in self.servers.items():"},{"line_number":218,"context_line":"            LOG.debug(\u0027Console output for %s\u0027, server.id)"},{"line_number":219,"context_line":"            LOG.debug(server.get_console_output())"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2Fk2I%3D","line":216,"in_reply_to":"AAAAUH%2F%2Fk4U%3D","updated":"2014-01-10 21:37:31.000000000","message":"LOG.debug(server.get_console_output()) is just one line.\nHow to manage servers is different per test cases, so I\u0027m not sure adding helper is helpful for the other test cases.","commit_id":"4fb7ce6bcfd5d7a3f4571cfe594456ec85e5f32e"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7a60969954e3a02a0f0118a0b3869e3ffd389eaa","unresolved":false,"context_lines":[{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    def _log_console_output(self):"},{"line_number":217,"context_line":"        for server, key in self.servers.items():"},{"line_number":218,"context_line":"            LOG.debug(\u0027Console output for %s\u0027, server.id)"},{"line_number":219,"context_line":"            LOG.debug(server.get_console_output())"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    def _check_tenant_network_connectivity(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2Fk3Y%3D","line":218,"updated":"2014-01-10 21:06:20.000000000","message":"Do you want to log the key also?","commit_id":"4fb7ce6bcfd5d7a3f4571cfe594456ec85e5f32e"},{"author":{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},"change_message_id":"6df1044593a4c63966921413b7f607f2588d7329","unresolved":false,"context_lines":[{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    def _log_console_output(self):"},{"line_number":217,"context_line":"        for server, key in self.servers.items():"},{"line_number":218,"context_line":"            LOG.debug(\u0027Console output for %s\u0027, server.id)"},{"line_number":219,"context_line":"            LOG.debug(server.get_console_output())"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    def _check_tenant_network_connectivity(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2Fktw%3D","line":218,"in_reply_to":"AAAAUH%2F%2Fk3Y%3D","updated":"2014-01-10 21:37:31.000000000","message":"key is specified by the test code. so IMO no need to log it","commit_id":"4fb7ce6bcfd5d7a3f4571cfe594456ec85e5f32e"}]}
