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" <[email protected]> > To: [email protected] > Cc: "Roi Dayan" <[email protected]>, "Itzik Brown" <[email protected]> > 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 > [email protected] > https://fedorahosted.org/mailman/listinfo/vdsm-patches > _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
