The version that is currently up is not the latest batch, I will send a final tested
version today.
In any case I don't think I agree with using a list as part of the internal API.
This creates an ambiguity of the resulting connection.
----- Original Message -----
From: "Roi Dayan" <roi.dayan(a)gmail.com>
To: vdsm-patches(a)lists.fedorahosted.org
Cc: "Roi Dayan" <roid(a)mellanox.com>, "Itzik Brown"
<ItzikB(a)mellanox.com>
Sent: Thursday, March 1, 2012 11:29:29 AM
Subject: iSER patch for VDSM
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
_______________________________________________
vdsm-patches mailing list
vdsm-patches(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/vdsm-patches