Hi,

Attached the patch for the latest update.
Added fixme line for the transport check.
Attached for review.
It's not tested yet. We are installing fedora 17 for it.

 
Thanks,
Roi


On Thu, Mar 1, 2012 at 6:29 PM, Roi Dayan <roi.dayan@gmail.com> wrote:
Hi,

The patch attached created on top of  change I3cedeb6c: Implement storage server connection aliasing 

The patch adds a new config option "iscsi_transports" in section "irs" that can accept transport list.
E.g. "iser,tcp"
When adding a new iscsi node first vdsm will try iser and then tcp.

The patch was not tested since the build of change I3cedeb6c fails.

/usr/bin/pyflakes vdsm vdsm_cli vds_bootstrap vdsm_reg vdsm_hooks
vdsm/storage/storageServer.py:25: 'Thread' imported but unused
vdsm/storage/storageServer.py:25: 'RLock' imported but unused
vdsm/storage/storageServer.py:25: 'Event' imported but unused
vdsm/storage/storageServer.py:34: 'asyncmethod' imported but unused
vdsm/storage/storageServer.py:34: 'AsyncCallStub' imported but unused
make[2]: *** [check-local] Error 1
make[2]: Leaving directory `/tmp/vdsm'
make[1]: *** [check-am] Error 2
make[1]: Leaving directory `/tmp/vdsm'
make: *** [check-recursive] Error 1

Thanks,
Roi

--

Roi 





--

Roi