Cinder

cisco cli _ssh_execute does not scale for concurrent deployments

Bug #1447776 reported by Yucong Feng
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
Yucong Feng
Cinder 7.0.0 "liberty"

Bug Description

In /zonemanager/drivers/cisco/cisco_fc_zone_client_cli.py _ssh_execute()
there is a random sleep of 0.2 to 5.0 seconds between the time the command is sent and when it sends ZoneConstant. YES, to confirm the operation.

There is significant delays with this sleep seen in concurrent deploys of 10 or more VM.

Brocade has this recently fixed in their CLI, bug 1436594

Jay Bryant (jsbryant)
Changed in cinder:
status: New → Confirmed
importance: Undecided → Medium
Yucong Feng (yfeng)
Changed in cinder:
assignee: nobody → Yucong Feng (yfeng)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/177335

Changed in cinder:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/177335
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=ad77e835e8d9143f8da29fa6f13b64837972fc99
Submitter: Jenkins
Branch: master

commit ad77e835e8d9143f8da29fa6f13b64837972fc99
Author: Yucong Feng <email address hidden>
Date: Fri Apr 24 10:49:52 2015 -0500

    Removing sleep between when a command is sent and 'YES' is sent

    Removing sleep between when a command is sent and 'YES' is sent to
    confirm the operation. We believe it is added because the Cisco CLI
    is mirrored based on Brocade. Removing would increase performance when
    launching concurrent vms.

    Closes-Bug: #1447776

    Change-Id: I3102922456fa9dde3a63ec623565a3427e9b357d

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → liberty-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-1 → 7.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.