]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Removed sleep before 'YES' is sent to confim an operation
authorPradeep Sathasivam <psathasi@brocade.com>
Wed, 8 Apr 2015 07:26:23 +0000 (12:56 +0530)
committerPradeep Sathasivam <psathasi@brocade.com>
Thu, 9 Apr 2015 06:58:34 +0000 (06:58 +0000)
commite2980d164965f2e2cae31452774d11e2552fffe2
tree93ab2d1d97698a02de351f1db65fef0116926a1e
parent1c06834e8c0c7f4a55103cdcc08744e52f0d234d
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