)]}'
{"tempest/lib/common/ssh.py":[{"author":{"_account_id":6671,"name":"Jerry Zhao","email":"xyzjerry@gmail.com","username":"zhaoxinyu"},"change_message_id":"2bb5f99ec501e99b19d910449c871932d8de948b","unresolved":false,"context_lines":[{"line_number":148,"context_line":"                if channel.recv_stderr_ready():"},{"line_number":149,"context_line":"                    err_chunk \u003d channel.recv_stderr(self.buf_size)"},{"line_number":150,"context_line":"                    err_data_chunks +\u003d err_chunk,"},{"line_number":151,"context_line":"                if not err_chunk and not out_chunk:"},{"line_number":152,"context_line":"                    break"},{"line_number":153,"context_line":"            out_data \u003d b\u0027\u0027.join(out_data_chunks)"},{"line_number":154,"context_line":"            err_data \u003d b\u0027\u0027.join(err_data_chunks)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_0de6a9af","line":151,"updated":"2016-04-19 23:06:17.000000000","message":"if the channel is not automatically closed after exec_command, do we need to close here with channel.close()? or you expect the brutal force of cleanup after the test case finishes.","commit_id":"eb9352fec85a229694304b7bfeb67588c5448e28"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"88a30921c4981abf7b32e880d30c0f1908feae98","unresolved":false,"context_lines":[{"line_number":148,"context_line":"                if channel.recv_stderr_ready():"},{"line_number":149,"context_line":"                    err_chunk \u003d channel.recv_stderr(self.buf_size)"},{"line_number":150,"context_line":"                    err_data_chunks +\u003d err_chunk,"},{"line_number":151,"context_line":"                if not err_chunk and not out_chunk:"},{"line_number":152,"context_line":"                    break"},{"line_number":153,"context_line":"            out_data \u003d b\u0027\u0027.join(out_data_chunks)"},{"line_number":154,"context_line":"            err_data \u003d b\u0027\u0027.join(err_data_chunks)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_cd181f21","line":151,"in_reply_to":"1a122d0e_0de6a9af","updated":"2016-04-20 08:22:55.000000000","message":"exec_command every time initiates new connection L118, L71. So I think that channel should be closed at the end of exec_command explicitly, but it is not related to current commit and should be fixed in separate patch.","commit_id":"eb9352fec85a229694304b7bfeb67588c5448e28"}]}