)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27078,"name":"Manik Bindlish","email":"manikbindlish19@gmail.com","username":"manik"},"change_message_id":"e8a9e0ce7ba9e0cc64c7522fad43d0819ae426f1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"the saner defaults from the new class tests such"},{"line_number":12,"context_line":"\"test_reassign_port_between_servers\" is less prone to fail (see related"},{"line_number":13,"context_line":"bug)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Also, this patch intentionally does not attempt to replace every occurance"},{"line_number":16,"context_line":"of tempest.common.utils.linux.remote_client with its counter-part in"},{"line_number":17,"context_line":"tempest.list because that would require porting some of the methods that"},{"line_number":18,"context_line":"exists in the old class to the new one. The focus of this patch is to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3f79a3b5_deb855e9","line":15,"range":{"start_line":14,"start_character":0,"end_line":15,"end_character":74},"updated":"2018-10-11 04:55:23.000000000","message":"Line exceeding the maximum length","commit_id":"5edc43927968a58ed0c5160b18337bf6e5275111"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"77100016246aa97f724662ceebbc73b76670d4d8","unresolved":false,"context_lines":[{"line_number":12,"context_line":"\"test_reassign_port_between_servers\" is less prone to fail (see related"},{"line_number":13,"context_line":"bug)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Also, this patch intentionally does not attempt to replace every occurance"},{"line_number":16,"context_line":"of tempest.common.utils.linux.remote_client with its counter-part in"},{"line_number":17,"context_line":"tempest.list because that would require porting some of the methods that"},{"line_number":18,"context_line":"exists in the old class to the new one. The focus of this patch is to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3f79a3b5_9833d512","line":15,"range":{"start_line":15,"start_character":65,"end_line":15,"end_character":74},"updated":"2018-10-11 02:48:59.000000000","message":"occurrence","commit_id":"5edc43927968a58ed0c5160b18337bf6e5275111"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"beab0eed46fb11f126b9f1b31c0b3d2a4360fb13","unresolved":false,"context_lines":[{"line_number":11,"context_line":"the saner defaults from the new class tests such"},{"line_number":12,"context_line":"\"test_reassign_port_between_servers\" is less prone to fail (see related"},{"line_number":13,"context_line":"bug)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Also, this patch intentionally does not attempt to replace every occurance"},{"line_number":16,"context_line":"of tempest.common.utils.linux.remote_client with its counter-part in"},{"line_number":17,"context_line":"tempest.list because that would require porting some of the methods that"},{"line_number":18,"context_line":"exists in the old class to the new one. The focus of this patch is to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3f79a3b5_c43883ee","line":15,"range":{"start_line":14,"start_character":0,"end_line":15,"end_character":74},"in_reply_to":"3f79a3b5_deb855e9","updated":"2018-10-11 10:11:39.000000000","message":"++","commit_id":"5edc43927968a58ed0c5160b18337bf6e5275111"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"77100016246aa97f724662ceebbc73b76670d4d8","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Also, this patch intentionally does not attempt to replace every occurance"},{"line_number":16,"context_line":"of tempest.common.utils.linux.remote_client with its counter-part in"},{"line_number":17,"context_line":"tempest.list because that would require porting some of the methods that"},{"line_number":18,"context_line":"exists in the old class to the new one. The focus of this patch is to"},{"line_number":19,"context_line":"address the problem described in the related bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3f79a3b5_d82dcdae","line":17,"range":{"start_line":17,"start_character":8,"end_line":17,"end_character":12},"updated":"2018-10-11 02:48:59.000000000","message":"lib","commit_id":"5edc43927968a58ed0c5160b18337bf6e5275111"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"beab0eed46fb11f126b9f1b31c0b3d2a4360fb13","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Also, this patch intentionally does not attempt to replace every occurance"},{"line_number":16,"context_line":"of tempest.common.utils.linux.remote_client with its counter-part in"},{"line_number":17,"context_line":"tempest.list because that would require porting some of the methods that"},{"line_number":18,"context_line":"exists in the old class to the new one. The focus of this patch is to"},{"line_number":19,"context_line":"address the problem described in the related bug."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3f79a3b5_64216fb5","line":17,"range":{"start_line":17,"start_character":8,"end_line":17,"end_character":12},"in_reply_to":"3f79a3b5_d82dcdae","updated":"2018-10-11 10:11:39.000000000","message":"++","commit_id":"5edc43927968a58ed0c5160b18337bf6e5275111"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"aeb175fc4edee475ca611fbf74fece45775eba57","unresolved":false,"context_lines":[{"line_number":17,"context_line":"counter-part in tempest.lib because that would require porting"},{"line_number":18,"context_line":"some of the methods that exists in the old class to the new one."},{"line_number":19,"context_line":"The focus of this patch is to address the problem described in"},{"line_number":20,"context_line":"the related bug."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Related-Bug: #1796882"},{"line_number":23,"context_line":"Change-Id: I07bde710581270766f049c7147246971fed6cd91"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3f79a3b5_7d27a71f","line":20,"updated":"2018-10-12 10:05:10.000000000","message":"So what seems to be going on here is that, the default ssh_timeout for the new RemoteClient (the tempest.lib one) class is 300 [0] but, the old class (tempest.common) the default value is being set based on the DevStack BUILD_TIMEOUT option which by default is 196 [1][2][3].\n\nSo I think that difference 300 vs 196 is what is causing the tests to error with a SSH timeout exception.\n\n[0] https://github.com/openstack/tempest/blob/ce4b7d246e09b4dfcd7f9ecef3b2318b010396ac/tempest/lib/common/utils/linux/remote_client.py#L71\n\n[1] https://github.com/openstack-dev/devstack/blob/e9f3988a57498f8593ff51055be212402fec60ce/lib/tempest#L55\n\n[2] https://github.com/openstack-dev/devstack/blob/e9f3988a57498f8593ff51055be212402fec60ce/lib/tempest#L431\n\n[3] https://github.com/openstack/tempest/blob/ce4b7d246e09b4dfcd7f9ecef3b2318b010396ac/tempest/common/utils/linux/remote_client.py#L46","commit_id":"692422babfb76ff2cea216877185a86eecf946e2"}]}