Change in vdsm[master]: fix test error, NET_CONF_BACK_DIR is not in configNetwork mo...

2012-07-11 Thread shaohef
ShaoHe Feng has uploaded a new change for review. Change subject: fix test error, NET_CONF_BACK_DIR is not in configNetwork module .. fix test error, NET_CONF_BACK_DIR is not in configNetwork module NET_CONF_BACK_DIR is not in

Change in vdsm[master]: configNetTests: fix another netinfo-rebase issue

2012-07-11 Thread ilvovsky
Igor Lvovsky has posted comments on this change. Change subject: configNetTests: fix another netinfo-rebase issue .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6121 To unsubscribe, visit

Change in vdsm[master]: Orthogonal storage repository conversion

2012-07-11 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: Orthogonal storage repository conversion .. Patch Set 32: (1 inline comment) Commit Message Line 10: not

Change in vdsm[master]: Use only one NETPREFIX constant

2012-07-11 Thread ilvovsky
Igor Lvovsky has posted comments on this change. Change subject: Use only one NETPREFIX constant .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6115 To unsubscribe,

Change in vdsm[master]: Use domain proxies instead of actual domain references

2012-07-11 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: Use domain proxies instead of actual domain references .. Patch Set 32: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: Use domain proxies instead of actual domain references

2012-07-11 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: Use domain proxies instead of actual domain references .. Patch Set 32: No score -- To view, visit http://gerrit.ovirt.org/3044 To unsubscribe, visit

Change in vdsm[master]: fix test error, NET_CONF_BACK_DIR is not in configNetwork mo...

2012-07-11 Thread ilvovsky
Igor Lvovsky has posted comments on this change. Change subject: fix test error, NET_CONF_BACK_DIR is not in configNetwork module .. Patch Set 1: I would prefer that you didn't submit this -- To view, visit

Change in vdsm[master]: Related to BZ#833099 - Removing unused exceptions.

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Related to BZ#833099 - Removing unused exceptions. .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6085 To unsubscribe, visit

Change in vdsm[master]: Related to BZ#833099 - spmRole is a StoragePool attribute.

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Related to BZ#833099 - spmRole is a StoragePool attribute. .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6084 To

Change in vdsm[master]: Add the hostId parameter to reconstructMaster

2012-07-11 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: Add the hostId parameter to reconstructMaster .. Patch Set 12: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5068 To

Change in vdsm[master]: Remove the unused nocache option

2012-07-11 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: Remove the unused nocache option .. Patch Set 17: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/3747 To unsubscribe,

Change in vdsm[master]: Uniform the block and file volume methods

2012-07-11 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: Uniform the block and file volume methods .. Patch Set 21: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/3683 To

Change in vdsm[master]: Unify the volume creation code in volume.create

2012-07-11 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: Unify the volume creation code in volume.create .. Patch Set 22: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/3627 To

Change in vdsm[master]: Use domain proxies instead of actual domain references

2012-07-11 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: Use domain proxies instead of actual domain references .. Patch Set 33: Verified; Looks good to me, approved -- To view, visit

Change in vdsm[master]: Orthogonal storage repository conversion

2012-07-11 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: Orthogonal storage repository conversion .. Patch Set 33: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/3045 To

Change in vdsm[master]: Add the formatConverter for Storage Domain V3

2012-07-11 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: Add the formatConverter for Storage Domain V3 .. Patch Set 17: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/3841 To

Change in vdsm[master]: bz#831938 raise exception when error occurs in connectStorag...

2012-07-11 Thread lvroyce
Royce Lv has uploaded a new change for review. Change subject: bz#831938 raise exception when error occurs in connectStorageServer .. bz#831938 raise exception when error occurs in connectStorageServer connectStorageServer

Change in vdsm[master]: cleanup: use iproute instead of vconfig for vlan management

2012-07-11 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: cleanup: use iproute instead of vconfig for vlan management .. cleanup: use iproute instead of vconfig for vlan management **EXPERIMENTAL** iproute utility is

Change in vdsm[master]: send POSIXFS instead of SHAREDFS

2012-07-11 Thread lhornyak
Laszlo Hornyak has posted comments on this change. Change subject: send POSIXFS instead of SHAREDFS .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/6103 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[master]: Use domain proxies instead of actual domain references

2012-07-11 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Use domain proxies instead of actual domain references .. Patch Set 33: I would prefer that you didn't submit this Ayal, Federico, Is this one so urgent?

Change in vdsm[master]: configNetTests: fix another netinfo-rebase issue

2012-07-11 Thread ilvovsky
Igor Lvovsky has posted comments on this change. Change subject: configNetTests: fix another netinfo-rebase issue .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/6121 To unsubscribe, visit

Change in vdsm[master]: configNetTests: fix another netinfo-rebase issue

2012-07-11 Thread ilvovsky
Igor Lvovsky has submitted this change and it was merged. Change subject: configNetTests: fix another netinfo-rebase issue .. configNetTests: fix another netinfo-rebase issue configNetTests have been stubbing the variable

Change in vdsm[master]: configNetTests: do not call a non-existing TestCaseBase.__de...

2012-07-11 Thread ilvovsky
Igor Lvovsky has posted comments on this change. Change subject: configNetTests: do not call a non-existing TestCaseBase.__del__ .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6104 To

Change in vdsm[master]: configNetTests: do not call a non-existing TestCaseBase.__de...

2012-07-11 Thread ilvovsky
Igor Lvovsky has submitted this change and it was merged. Change subject: configNetTests: do not call a non-existing TestCaseBase.__del__ .. configNetTests: do not call a non-existing TestCaseBase.__del__ This gives an ominous

Change in vdsm[master]: vdsmd: Don't keep the temporary file generated by configurin...

2012-07-11 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vdsmd: Don't keep the temporary file generated by configuring libvirt .. Patch Set 1: I do not think it is highly important, but I won't nack a bump (fedora

Change in vdsm[master]: remove symlink when connect local storage failed

2012-07-11 Thread lvroyce
Royce Lv has uploaded a new change for review. Change subject: remove symlink when connect local storage failed .. remove symlink when connect local storage failed if connectStorageServer failed for some reason, symlink remains

Change in vdsm[master]: clean .pyc files generated in unit tests

2012-07-11 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: clean .pyc files generated in unit tests .. Patch Set 3: @He Jie: Thanks very much! If we do not generate .pyc files at all during unit test, we will not

Change in vdsm[master]: Update error code of migrateToURI2() on abortJob()

2012-07-11 Thread wudxw
Mark Wu has uploaded a new change for review. Change subject: Update error code of migrateToURI2() on abortJob() .. Update error code of migrateToURI2() on abortJob() Before migrateToURI2() just returned a generl error

Change in vdsm[master]: pthread.py: avoid stdout noise on process shutdown

2012-07-11 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: pthread.py: avoid stdout noise on process shutdown .. Patch Set 2: Verified; Looks good to me, but someone else must approve Reproduced and solved by this patch.

Change in vdsm[master]: Handle bridgeless networks too in getNetworksAndVlansForBond...

2012-07-11 Thread ilvovsky
Igor Lvovsky has uploaded a new change for review. Change subject: Handle bridgeless networks too in getNetworksAndVlansForBonding .. Handle bridgeless networks too in getNetworksAndVlansForBonding Change-Id:

Change in vdsm[master]: Fix bridgeless networks report

2012-07-11 Thread ilvovsky
Igor Lvovsky has uploaded a new change for review. Change subject: Fix bridgeless networks report .. Fix bridgeless networks report Change-Id: I2523c10bf1692bac7429c7be056297a9b857541b Signed-off-by: Igor Lvovsky

Change in vdsm[master]: MOM Integration

2012-07-11 Thread wudxw
Mark Wu has posted comments on this change. Change subject: MOM Integration .. Patch Set 12: Other questions except the file name(MOM.py) has been answered. -- To view, visit http://gerrit.ovirt.org/2367 To unsubscribe, visit

Change in vdsm[master]: Fix bridgeless networks report

2012-07-11 Thread ilvovsky
Igor Lvovsky has posted comments on this change. Change subject: Fix bridgeless networks report .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/6147 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[master]: Handle bridgeless networks too in getNetworksAndVlansForBond...

2012-07-11 Thread ilvovsky
Igor Lvovsky has posted comments on this change. Change subject: Handle bridgeless networks too in getNetworksAndVlansForBonding .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/6143 To unsubscribe, visit

Change in vdsm[master]: Fix bridgeless networks report

2012-07-11 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix bridgeless networks report .. Patch Set 1: Looks good to me, approved oops, sorry... -- To view, visit http://gerrit.ovirt.org/6147 To unsubscribe, visit

Change in vdsm[master]: Fix bridgeless networks report

2012-07-11 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Fix bridgeless networks report .. Fix bridgeless networks report Change-Id: I2523c10bf1692bac7429c7be056297a9b857541b Signed-off-by: Igor Lvovsky

Change in vdsm[master]: pthread.py: avoid stdout noise on process shutdown

2012-07-11 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: pthread.py: avoid stdout noise on process shutdown .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5659 To unsubscribe,

Change in vdsm[master]: pthread.py: avoid stdout noise on process shutdown

2012-07-11 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: pthread.py: avoid stdout noise on process shutdown .. pthread.py: avoid stdout noise on process shutdown For some reason, on process shutdown,

Change in vdsm[master]: BZ#833099 - Induce MSD reconstruct when getSpmStatus fail.

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: BZ#833099 - Induce MSD reconstruct when getSpmStatus fail. .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6083 To

Change in vdsm[master]: Related to BZ#833099 - Removing unused exceptions.

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Related to BZ#833099 - Removing unused exceptions. .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6085 To unsubscribe, visit

Change in vdsm[master]: Related to BZ#833099 - spmRole is a StoragePool attribute.

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Related to BZ#833099 - spmRole is a StoragePool attribute. .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6084 To

Change in vdsm[master]: BZ#784931 - Fixing race condition in deactivateSD().

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: BZ#784931 - Fixing race condition in deactivateSD(). .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5677 To unsubscribe,

Change in vdsm[master]: Fix 2nd order exception msg in lvm._initpvs().

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Fix 2nd order exception msg in lvm._initpvs(). .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5319 To unsubscribe, visit

Change in vdsm[master]: bootstrap: m2crypto is no longer used

2012-07-11 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: bootstrap: m2crypto is no longer used .. Patch Set 2: Kaul: upstream Vdsm does not really try to support prehistoric vdsm that used m2crypto. And even if

Change in vdsm[master]: Remove unnecesary preparePaths.

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Remove unnecesary preparePaths. .. Patch Set 1: Do not submit I agree with Federico, this code is needed unless you find a different way to make sure that no

Change in vdsm[master]: bootstrap: m2crypto is no longer used

2012-07-11 Thread ykaul
Yaniv Kaul has posted comments on this change. Change subject: bootstrap: m2crypto is no longer used .. Patch Set 2: From the code it looked like the bootstrap is used to install any VDSM version, new or old. So even if the

Change in vdsm[master]: Introducing the template activation leak.

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Introducing the template activation leak. .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/863 To unsubscribe, visit

Change in vdsm[master]: Related to BZ#784931 - Removing the unused useCache paramete...

2012-07-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Related to BZ#784931 - Removing the unused useCache parameter. .. Patch Set 1: Looks good to me, approved (1 inline comment)

Change in vdsm[master]: BZ#837443: removeVlan() drop/remove interface

2012-07-11 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: BZ#837443: removeVlan() drop/remove interface .. Patch Set 3: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: BZ#837443: removeVlan() drop/remove interface

2012-07-11 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: BZ#837443: removeVlan() drop/remove interface .. Patch Set 3: No score -- To view, visit http://gerrit.ovirt.org/6090 To unsubscribe, visit

Change in vdsm[master]: BZ#837443: remove bridge before add VDSM bridge

2012-07-11 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: BZ#837443: remove bridge before add VDSM bridge .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/6016 To unsubscribe, visit

Change in vdsm[master]: BZ#837443: removeVlan() drop/remove interface

2012-07-11 Thread ilvovsky
Igor Lvovsky has posted comments on this change. Change subject: BZ#837443: removeVlan() drop/remove interface .. Patch Set 5: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6090 To

Change in vdsm[master]: BZ#837443: remove bridge before add VDSM bridge

2012-07-11 Thread ilvovsky
Igor Lvovsky has posted comments on this change. Change subject: BZ#837443: remove bridge before add VDSM bridge .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6016 To

Change in vdsm[master]: Change source of CPU sockets/cores to /sys and add cpuThre...

2012-07-11 Thread apahim
Amador Pahim has posted comments on this change. Change subject: Change source of CPU sockets/cores to /sys and add cpuThreads. .. Patch Set 4: (8 inline comments) Commit

Change in vdsm[master]: Use more descriptive exceptions for blocksize errors.

2012-07-11 Thread lyarwood
Lee Yarwood has abandoned this change. Change subject: Use more descriptive exceptions for blocksize errors. .. Patch Set 1: Abandoned -- To view, visit http://gerrit.ovirt.org/3688 To unsubscribe, visit

Change in vdsm[master]: BZ#839312 - Use cp to copy sparse files to ensure they remai...

2012-07-11 Thread lyarwood
Lee Yarwood has uploaded a new change for review. Change subject: BZ#839312 - Use cp to copy sparse files to ensure they remain sparse. .. BZ#839312 - Use cp to copy sparse files to ensure they remain sparse. Previously all

Change in vdsm[master]: Change source of CPU sockets/cores to /sys and add cpuThre...

2012-07-11 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: Change source of CPU sockets/cores to /sys and add cpuThreads. .. Patch Set 5: I would prefer that you didn't submit this Hi, actually, the new topology

Change in vdsm[master]: clean .pyc files generated in unit tests

2012-07-11 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: clean .pyc files generated in unit tests .. Patch Set 3: I would prefer that you didn't submit this why wouldn't diff --git a/tests/run_tests_local.sh.in

Change in vdsm[master]: BZ#837443: removeVlan() drop/remove interface

2012-07-11 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: BZ#837443: removeVlan() drop/remove interface .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6090 To unsubscribe, visit

Change in vdsm[master]: build: use consistent python path

2012-07-11 Thread Alon Bar-Lev
Alon Bar-Lev has abandoned this change. Change subject: build: use consistent python path .. Patch Set 2: Abandoned Did this 2cnd stage only to see if can be done without modifying the code of vdsm. -- To view, visit

Change in vdsm[master]: fix test error, NET_CONF_BACK_DIR is not in configNetwork mo...

2012-07-11 Thread shaohef
ShaoHe Feng has posted comments on this change. Change subject: fix test error, NET_CONF_BACK_DIR is not in configNetwork module .. Patch Set 1: Igor, yes, it is same with Dan's patch -- To view, visit

Change in vdsm[master]: fix test error, NET_CONF_BACK_DIR is not in configNetwork mo...

2012-07-11 Thread shaohef
ShaoHe Feng has abandoned this change. Change subject: fix test error, NET_CONF_BACK_DIR is not in configNetwork module .. Patch Set 1: Abandoned it is same with Dan's patch -- To view, visit http://gerrit.ovirt.org/6131 To

Change in vdsm[master]: iscsi: fix update discoverydb with wrong key

2012-07-11 Thread Xu He Jie
Xu He Jie has posted comments on this change. Change subject: iscsi: fix update discoverydb with wrong key .. Patch Set 1: Mark, yes, it dones't used by iscsiadm. But as the comment in the code, I guess we prefer to keep this

Change in vdsm[master]: BZ#837443: remove bridge before add VDSM bridge

2012-07-11 Thread wudxw
Mark Wu has posted comments on this change. Change subject: BZ#837443: remove bridge before add VDSM bridge .. Patch Set 4: Looks good to me, but someone else must approve (1 inline comment)

Change in vdsm[master]: Suppress generating .py{c, o} files when 'make check'

2012-07-11 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: Suppress generating .py{c,o} files when 'make check' .. Patch Set 4: The purpose of this patch is to avoid false positives in 'make check'. We sometimes

Change in vdsm[master]: Suppress generating .py{c, o} files when 'make check'

2012-07-11 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: Suppress generating .py{c,o} files when 'make check' .. Patch Set 4: Sorry. I just look up the dictionary, by saying false positives I actually mean false

Change in vdsm[master]: Suppress generating .py{c, o} files when 'make check'

2012-07-11 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: Suppress generating .py{c,o} files when 'make check' .. Patch Set 5: This message is the same as the one for patch set 4. Just looked up the dictionary and