Replace ssh connections in san.py with a connection pool

Bug #1074185 reported by Nirmal Ranganathan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Wishlist
Nirmal Ranganathan

Bug Description

Replace def _connect_to_ssh(self): to use a ssh connection pool instead of a new connection for each call.

Changed in cinder:
assignee: nobody → Nirmal Ranganathan (rnirmal)
status: New → In Progress
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/15255

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/15255
Committed: http://github.com/openstack/cinder/commit/b864a08b2aa7e7ba953dd4afce2c3042ba963ae7
Submitter: Jenkins
Branch: master

commit b864a08b2aa7e7ba953dd4afce2c3042ba963ae7
Author: Nirmal Ranganathan <email address hidden>
Date: Thu Nov 1 23:05:35 2012 -0500

    Adding a SSH Connection Pool.

    Adds a connection pool using eventlet.pools
    which works well with greenthreads. Adding new
    parameters ssh_min_pool_conn and ssh_max_pool_conn.

    Updating the _run_ssh method in san.py to use the
    connection pool and cleaning up the existing
    single connections, also added retries.

    Fixes bug 1074185

    Change-Id: I90dd89ffc025d09fc6ad060c4273508103b85456

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

Other bug subscribers

Remote bug watches

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