]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Refactor SSHPool.get() to use Pool.get()
authorxiaoxi_chen <xiaoxi.chen@intel.com>
Fri, 12 Jul 2013 08:51:50 +0000 (16:51 +0800)
committerxiaoxi_chen <xiaoxi.chen@intel.com>
Fri, 12 Jul 2013 15:41:44 +0000 (23:41 +0800)
commitcccf7b92c189bcadf6898019730d84bfd4fe70b6
treebe7fa84229dea277e4c6e150b95cc7394260720f
parent76d8d6c1480ef7b74545107252f47bad858b6ecd
Refactor SSHPool.get() to use Pool.get()

In previous code of SSHPool.get(), we pasted the code
from Pool.get() and check if a connection is active before return
it.

However, it's much simpler and cleaner to just call the Pool.get()
and then check the connection before return. With this,we can free
ourselves from manually keeping up with code of Pool.get() in
upstream package eventlet

As a side effect,this patch fixed bug #1194393 which caused by a
previous bug in eventlet codes before revision 1072

fixed bug #1194393
Change-Id: Ic2bf2fa1ad82cf8669b6c491c955dcab39eb1510
cinder/utils.py