From e2980d164965f2e2cae31452774d11e2552fffe2 Mon Sep 17 00:00:00 2001 From: Pradeep Sathasivam Date: Wed, 8 Apr 2015 12:56:23 +0530 Subject: [PATCH] Removed sleep before 'YES' is sent to confim an operation Removed sleep between the time a command is sent and 'YES' is sent to confim the operation. We have discussed this internally and have concluded that the sleep was mistakenly added in the initial coding from an example of the SSH code. We have done full functionality testing of the zoning operations on Brocade switches with the sleep removed and have found no side effects from the removal Additionally, no other Brocade management applications which perform zoning on our switches require sleep intervals between execution of commands. We are confident that the sleep is not necessary and can be removed Change-Id: I2ac6e50ff3d231b4d404f1a0be29f907e8d20458 Closes-Bug: #1436594 --- cinder/zonemanager/drivers/brocade/brcd_fc_zone_client_cli.py | 1 - 1 file changed, 1 deletion(-) diff --git a/cinder/zonemanager/drivers/brocade/brcd_fc_zone_client_cli.py b/cinder/zonemanager/drivers/brocade/brcd_fc_zone_client_cli.py index 3eaa2728b..52eeef1cd 100644 --- a/cinder/zonemanager/drivers/brocade/brcd_fc_zone_client_cli.py +++ b/cinder/zonemanager/drivers/brocade/brcd_fc_zone_client_cli.py @@ -441,7 +441,6 @@ class BrcdFCZoneClientCLI(object): attempts -= 1 try: stdin, stdout, stderr = ssh.exec_command(command) - greenthread.sleep(random.randint(20, 500) / 100.0) stdin.write("%s\n" % ZoneConstant.YES) channel = stdout.channel exit_status = channel.recv_exit_status() -- 2.45.2