)]}'
{"tempest_lib/common/ssh.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"2d10d8de902ddf9f015a9c7615d50bd74fde0a81","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        exit_status \u003d channel.recv_exit_status()"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"        # If the executing host is linux-based, poll the channel"},{"line_number":120,"context_line":"        if hasattr(select, \u0027poll\u0027):"},{"line_number":121,"context_line":"            out_data_chunks \u003d []"},{"line_number":122,"context_line":"            err_data_chunks \u003d []"},{"line_number":123,"context_line":"            poll \u003d select.poll()"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_baac3836","line":120,"updated":"2015-08-18 22:43:10.000000000","message":"Does the module just not have this attr on non-linux envs?","commit_id":"477973eaddf70cc7ff11d925d0a5198bb0a4ebfc"},{"author":{"_account_id":97,"name":"Daryl Walleck","email":"daryl.walleck@rackspace.com","username":"dwalleck"},"change_message_id":"a131e7b881c0493b33f687cf0cf420c58c2f5617","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        exit_status \u003d channel.recv_exit_status()"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"        # If the executing host is linux-based, poll the channel"},{"line_number":120,"context_line":"        if hasattr(select, \u0027poll\u0027):"},{"line_number":121,"context_line":"            out_data_chunks \u003d []"},{"line_number":122,"context_line":"            err_data_chunks \u003d []"},{"line_number":123,"context_line":"            poll \u003d select.poll()"}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_7a49facf","line":120,"in_reply_to":"fa1b9901_baac3836","updated":"2015-08-27 04:48:08.000000000","message":"That\u0027s correct. It\u0027s very strange behavior because this module has different methods depending on the platform Python is running on. I went with this check because it\u0027s most closely tied to the issue, but it also would be possible to run platform.system() to get the underlying OS and use that to make the decision","commit_id":"477973eaddf70cc7ff11d925d0a5198bb0a4ebfc"}],"tempest_lib/tests/test_ssh.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"90d8a9072a08732c400efe1da1abe2bddc4234aa","unresolved":false,"context_lines":[{"line_number":225,"context_line":""},{"line_number":226,"context_line":"        chan_mock.makefile.assert_called_once_with(\u0027rb\u0027, 1024)"},{"line_number":227,"context_line":"        chan_mock.makefile_stderr.assert_called_once_with(\u0027rb\u0027, 1024)"},{"line_number":228,"context_line":"        std_out_mock.read.assert_called_once_with()"},{"line_number":229,"context_line":"        std_err_mock.read.assert_called_once_with()"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_9fa88079","line":228,"updated":"2015-09-08 19:56:16.000000000","message":"Do we also want to assert that select is never attempted to be called? Something like:\n\n  self.assertEqual(len(select_mock.mock_calls), 0, \u0027Select mock was called but we shouldn\u0027t call it if mock isn\u0027t available\u0027)","commit_id":"a0433588a5d30a2172b7eece3b6791f0ccf238ec"}]}