Change in vdsm[master]: call sync before umount in mkFloppyFs
Zhou Zheng Sheng has uploaded a new change for review. Change subject: call sync before umount in mkFloppyFs .. call sync before umount in mkFloppyFs mkFloppyFs creates a floppy image, mount it, write some content, and umount the image. On some systems, if we umount immediately after writing content, umount will fail, because the data is not fully synced to disk, and the device is still busy. This problem causes unit test test_mkFloppyFs in mkimageTests.py fail. To avoid the busy device problem, call /usr/bin/sync before umount. Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Signed-off-by: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com --- M configure.ac M vdsm/constants.py.in M vdsm/mkimage.py M vdsm/storage/misc.py 4 files changed, 7 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/95/6995/1 -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
Zhou Zheng Sheng has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 1: This problem causes unit test test_mkFloppyFs in tests/mkimageTests.py fail. My uname -a is as follow, Linux zhshzhouf17 3.5.0-2.fc17.x86_64 #1 SMP Mon Jul 30 14:48:59 UTC 2012 x86_64 x86_64 x86_64 GNU/Linux On my system if I umount immediately after wrting file contents, umount will fail. To reproduce the problem, you can use the script below. The script mimics what mkFloppyFs does. #!/usr/bin/bash mkdir tmpMount /usr/sbin/mkfs.msdos -C floppy.img 1440 /usr/bin/mount -o loop floppy.img tmpMount echo '123'tmpMount/tst1.txt echo '123'tmpMount/tst2.txt echo '123'tmpMount/tst3.txt echo '123'tmpMount/tst4.txt echo '123'tmpMount/tst5.txt echo '123'tmpMount/tst6.txt echo '123'tmpMount/tst7.txt echo '123'tmpMount/tst8.txt echo '123'tmpMount/tst9.txt #sleep 2 /usr/bin/umount -f tmpMount rm -rf tmpMount floppy.img When I run the script, umount fails. On some other machine of VM, the script may not fail. If I uncomment the sleep command in the script, umount works again. This means some data need to be sync to disk, so the device is busy, and umount can not detach the devices. On my system, if I sync before umount, umount will work. Some of my colleagues run into this problem as well. The patch run through autobuild.sh -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Added glusterVolumeStatus verb
Bala.FA has posted comments on this change.
Change subject: Added glusterVolumeStatus verb
..
Patch Set 2: (6 inline comments)
File vdsm_cli/vdsClientGluster.py
Line 198: if len(args) == 2:
I got with meaningful argument names
File vdsm/gluster/cli.py
Line 89: status['name'] = tree.find('volStatus/volName').text
Done
Line 95: value = {}
Done
Line 103: value['status'] = 'OFFLINE'
Done
Line 128: bricks = []
Done
Line 152: bricks = []
Done
--
To view, visit http://gerrit.ovirt.org/6374
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8769c2da299a598ed7500c4780a3329972f4a8ac
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Ayal Baron aba...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: Vijay Bellur vbel...@gmail.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: API: Provide a working getAllTasks API
Bala.FA has posted comments on this change.
Change subject: API: Provide a working getAllTasks API
..
Patch Set 3: Looks good to me, but someone else must approve
(1 inline comment)
File vdsm_cli/vdsClient.py
Line 1176: keys = [x.strip() for x in args[0].split(',')]
In that case, I would like to have list of keys as less of params eg. keys =
args[1:]
--
To view, visit http://gerrit.ovirt.org/3613
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I43371d358dfdfcfcee14aca40ed06d63be7c0a44
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke a...@us.ibm.com
Gerrit-Reviewer: Adam Litke a...@us.ibm.com
Gerrit-Reviewer: Ayal Baron aba...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Doron Fediuck dfedi...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com
Gerrit-Reviewer: Shireesh Anjal san...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
Mark Wu has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 1: Is it necessary to add a new constant and a function to fix a problem in a test case? How about just call sync in the test case? -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: ShaoHe Feng shao...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: glusterHostsList returns host info as list of dictionaries
Bala.FA has posted comments on this change. Change subject: glusterHostsList returns host info as list of dictionaries .. Patch Set 1: (1 inline comment) Commit Message Line 7: glusterHostsList returns host info as list of dictionaries Done -- To view, visit http://gerrit.ovirt.org/6614 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia05d59020bd4019dab2020988dc621d026b805b6 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Bala.FA barum...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
Mark Wu has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 1: (1 inline comment) File vdsm/mkimage.py Line 86: storage.misc.syncFileSystem() add a simple comment to explain why it needs a sync here? -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: ShaoHe Feng shao...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
Zhou Zheng Sheng has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 1: (1 inline comment) Mark Wu, the problem is in the function mkFloppyFs in vdsm/mkimage.py, not in the test case. The test case just calls mkFloppyFs and fail, so we can know there is a problem. As we write more test cases, we may reveal the same problem in other vdsm source files, so we may want to call sync in other files, that's why I add a function. File vdsm/mkimage.py Line 86: storage.misc.syncFileSystem() Yes. Otherwise other programmers will get confused. -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: ShaoHe Feng shao...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
Mark Wu has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 1: I see there's an existing lazy option for the umount() function, so it's easier to use lazy umount than sync. -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: ShaoHe Feng shao...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
Zhou Zheng Sheng has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 1: I have tried lazy umount as well, it does not work. The test case will mount the image immediately after the previous umount, if I use lazy umount, the check will fail, because the data is not synced to the image yet. So I have to sync. By the way, I tried fsync as well. The result is, if I fsync every file and close them, umount still throws exception. I have to fsync the files and umount lazyli to get it work. With fsync and a lazy umount, the data is written to the disk before umount, so when the test case mount again immediately, it checks the contents succesfully. -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: ShaoHe Feng shao...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Move option 'save_image_format' to qemu.conf
Mark Wu has posted comments on this change. Change subject: Move option 'save_image_format' to qemu.conf .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/5928 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I55d3e33783071cacbc71a31c5c46083e99fe65d1 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Move option 'save_image_format' to qemu.conf
Mark Wu has posted comments on this change. Change subject: Move option 'save_image_format' to qemu.conf .. Patch Set 2: Jenkins failed in the test of ResourceManagerTests.testStressTest because it can't create a new thread due to the ulimit of process number. It's not related to this patch -- To view, visit http://gerrit.ovirt.org/5928 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I55d3e33783071cacbc71a31c5c46083e99fe65d1 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
Zhou Zheng Sheng has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 1: Thanks mark and hejie. After collecting more advice, I will make a new patch set later. -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: ShaoHe Feng shao...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Support for QEMU cmdline hook
Deepak C Shetty has posted comments on this change.
Change subject: Support for QEMU cmdline hook
..
Patch Set 2: (2 inline comments)
File vdsm_hooks/qemucmdline/before_vm_start.py
Line 43: For eg: qemu_cmdline='{-cdrom:path/to/iso, ...}'
Done
File vdsm_hooks/qemucmdline/README
Line 9: The os environ 'qemu_cmdline' points to a json dict having one
Done
--
To view, visit http://gerrit.ovirt.org/6969
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I48477926d271bb9f6ae74f79de5d8c4d37b87caa
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Deepak C Shetty deepa...@linux.vnet.ibm.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Deepak C Shetty deepa...@linux.vnet.ibm.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: minimalistic (incomplete) test for mkimage.py with fail for ...
Dan Kenigsberg has posted comments on this change. Change subject: minimalistic (incomplete) test for mkimage.py with fail for non-64base data .. Patch Set 1: Fails; Do not submit -- To view, visit http://gerrit.ovirt.org/6275 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1a7656359beb5d116ad1919e237fe084a1208d71 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Josef Pacula josef.pac...@gmail.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Michal Skrivanek michal.skriva...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
Zhou Zheng Sheng has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 2: Verified patch set 2: add some comments Run through autobuild.sh and fix the problem for me. -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: ShaoHe Feng shao...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: glusterHostsList returns host info as list of dictionaries
Bala.FA has posted comments on this change. Change subject: glusterHostsList returns host info as list of dictionaries .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/6614 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia05d59020bd4019dab2020988dc621d026b805b6 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Bala.FA barum...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Move createLibvirtNetwork and removeLibvirtNetwork to the Co...
Dan Kenigsberg has submitted this change and it was merged. Change subject: Move createLibvirtNetwork and removeLibvirtNetwork to the ConfigWriter .. Move createLibvirtNetwork and removeLibvirtNetwork to the ConfigWriter Change-Id: I672ce5fbe0c5db826d53664285ea6212f49fa50b Signed-off-by: Igor Lvovsky ilvov...@redhat.com --- M vdsm/clientIF.py M vdsm/configNetwork.py M vdsm/netinfo.py 3 files changed, 53 insertions(+), 49 deletions(-) Approvals: oVirt Jenkins CI Server: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6893 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I672ce5fbe0c5db826d53664285ea6212f49fa50b Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Add atomic/persist backup capabilities for logical networks
Dan Kenigsberg has submitted this change and it was merged. Change subject: Add atomic/persist backup capabilities for logical networks .. Add atomic/persist backup capabilities for logical networks Change-Id: If9be3e7e7efc8b8ca3781e953f52bda59b8d37cf Signed-off-by: Igor Lvovsky ilvov...@redhat.com --- M vdsm/configNetwork.py 1 file changed, 66 insertions(+), 18 deletions(-) Approvals: oVirt Jenkins CI Server: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6913 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If9be3e7e7efc8b8ca3781e953f52bda59b8d37cf Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: configNetwork: Don't swallow exception during removeLibvirtN...
Dan Kenigsberg has submitted this change and it was merged. Change subject: configNetwork: Don't swallow exception during removeLibvirtNetwork .. configNetwork: Don't swallow exception during removeLibvirtNetwork In this patch: - No need to swallow libvirt exceptions during removeLibvirtNetwork - No need to run forcibly removeLibvirtNetwork(network) before createLibvirtNetwork(network) Change-Id: I4953348d79bc2ee2bc9dfddc93588e97d3c7a208 Signed-off-by: Igor Lvovsky ilvov...@redhat.com --- M vdsm/configNetwork.py 1 file changed, 8 insertions(+), 13 deletions(-) Approvals: oVirt Jenkins CI Server: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6914 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4953348d79bc2ee2bc9dfddc93588e97d3c7a208 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#832765 - configNetwork: add restoreAtomicNetworkBackup
Dan Kenigsberg has submitted this change and it was merged. Change subject: BZ#832765 - configNetwork: add restoreAtomicNetworkBackup .. BZ#832765 - configNetwork: add restoreAtomicNetworkBackup Change-Id: I132aa07c6fa2d0bd56ab5a6a79134aed4d0bfb85 Signed-off-by: Igor Lvovsky ilvov...@redhat.com --- M vdsm/configNetwork.py 1 file changed, 25 insertions(+), 6 deletions(-) Approvals: oVirt Jenkins CI Server: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6915 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I132aa07c6fa2d0bd56ab5a6a79134aed4d0bfb85 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Add restoreBackups and clearBackups
Igor Lvovsky has posted comments on this change. Change subject: Add restoreBackups and clearBackups .. Patch Set 3: (1 inline comment) File vdsm/configNetwork.py Line 350: shutil.rmtree(netinfo.NET_CONF_BACK_DIR, ignore_errors=True) what do you mean? Do you prefer to leave directory and remove only files? why? -- To view, visit http://gerrit.ovirt.org/6976 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia0c74b29a77b935e82e817a93ab10faefb3709fa Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: API: Provide a working getAllTasks API
Adam Litke has posted comments on this change. Change subject: API: Provide a working getAllTasks API .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/3613 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I43371d358dfdfcfcee14aca40ed06d63be7c0a44 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: Adam Litke a...@us.ibm.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Doron Fediuck dfedi...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: Shireesh Anjal san...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Remove orphaned API: getStorageConnections
Adam Litke has posted comments on this change. Change subject: Remove orphaned API: getStorageConnections .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/6107 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If48da46ba371171a575281a7578cdd006f4e33ae Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: Adam Litke a...@us.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: Shu Ming shum...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Initialize the pthread RLock attribute
Dan Kenigsberg has posted comments on this change. Change subject: Initialize the pthread RLock attribute .. Patch Set 3: (1 inline comment) File vdsm/pthread.py Line 74 I still do not understand why you've made all these changes... why adding _libpthread.pthread_mutexattr_init(attr) to the old code would not have been enough. -- To view, visit http://gerrit.ovirt.org/6988 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3d71606113c8bbb17620a753f534532510d92c6a Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#842771 Fix for exception TypeError: cannot marshal None
Gal Hammer has posted comments on this change. Change subject: BZ#842771 Fix for exception TypeError: cannot marshal None .. Patch Set 1: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6909 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Gal Hammer gham...@redhat.com Gerrit-Reviewer: Barak Azulay bazu...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Gal Hammer gham...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Added glusterVolumeStatus verb
Bala.FA has posted comments on this change. Change subject: Added glusterVolumeStatus verb .. Patch Set 2: (1 inline comment) File vdsm_cli/vdsClientGluster.py Line 198: if len(args) == 2: Fixing meaningful name of rest of methods will be done in another patch. Let me know its OK to you. -- To view, visit http://gerrit.ovirt.org/6374 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8769c2da299a598ed7500c4780a3329972f4a8ac Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Bala.FA barum...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com Gerrit-Reviewer: Vijay Bellur vbel...@gmail.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Added xml option in gluster command execution.
Hello Ayal Baron, Timothy Asir, Saggi Mizrahi, Federico Simoncelli, Dan Kenigsberg, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/6999 to review the following change. Change subject: Added xml option in gluster command execution. .. Added xml option in gluster command execution. This helps to run gluster command which outputs in xml or standard format. Change-Id: I385f992005d446b417be84eb7ff484e4edf6e5b6 Signed-off-by: Bala.FA barum...@redhat.com --- M vdsm/gluster/cli.py M vdsm/gluster/exception.py 2 files changed, 28 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/99/6999/1 -- To view, visit http://gerrit.ovirt.org/6999 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I385f992005d446b417be84eb7ff484e4edf6e5b6 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Bala.FA barum...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Updateing ChangeLog just to force an update.
Robert Middleswarth has uploaded a new change for review. Change subject: Updateing ChangeLog just to force an update. .. Updateing ChangeLog just to force an update. Change-Id: I860431fb30168a36de1a220fb52da0176e2fd60e Signed-off-by: Robert Middleswarth rob...@middleswarth.net --- M ChangeLog 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/93/6993/1 -- To view, visit http://gerrit.ovirt.org/6993 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I860431fb30168a36de1a220fb52da0176e2fd60e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Robert Middleswarth rob...@middleswarth.net ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Another minor change to tigger a build
Robert Middleswarth has uploaded a new change for review. Change subject: Another minor change to tigger a build .. Another minor change to tigger a build Change-Id: Ic355534713fbd0ebd4cad8fb9ea79668c32d58f2 Signed-off-by: Robert Middleswarth rob...@middleswarth.net --- M ChangeLog 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/94/6994/1 -- To view, visit http://gerrit.ovirt.org/6994 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic355534713fbd0ebd4cad8fb9ea79668c32d58f2 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Robert Middleswarth rob...@middleswarth.net ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Another minor change to tigger a build
Robert Middleswarth has abandoned this change. Change subject: Another minor change to tigger a build .. Patch Set 1: Abandoned This was here just to test the system. -- To view, visit http://gerrit.ovirt.org/6994 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: abandon Gerrit-Change-Id: Ic355534713fbd0ebd4cad8fb9ea79668c32d58f2 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Robert Middleswarth rob...@middleswarth.net Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Updateing ChangeLog just to force an update.
Robert Middleswarth has abandoned this change. Change subject: Updateing ChangeLog just to force an update. .. Patch Set 1: Abandoned Testing Jenkins -- To view, visit http://gerrit.ovirt.org/6993 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: abandon Gerrit-Change-Id: I860431fb30168a36de1a220fb52da0176e2fd60e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Robert Middleswarth rob...@middleswarth.net Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#844656 Respawn the domain monitor when needed
Omri Hochman has posted comments on this change. Change subject: BZ#844656 Respawn the domain monitor when needed .. Patch Set 6: Verified -- To view, visit http://gerrit.ovirt.org/6787 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I47e4fb6883d72578ee8083e46e4f660b656a5ffc Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Omri Hochman hoo...@gmail.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#844656 Release the _syncroot lock during _findDomain
Omri Hochman has posted comments on this change. Change subject: BZ#844656 Release the _syncroot lock during _findDomain .. Patch Set 7: Verified -- To view, visit http://gerrit.ovirt.org/6822 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8088d5fe716a3a08c3e5cef2d2d9a654ee96f60a Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Eduardo ewars...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Omri Hochman hoo...@gmail.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Rename Global class to API
Adam Litke has uploaded a new change for review. Change subject: schema: Rename Global class to API .. schema: Rename Global class to API The API class is the root of the API. It will have functions to connect and disconnect from the server. It will also have all of the commands that used to be defined as 'Global' since it doesn't make sense to sub-class them. Change-Id: I64afff9e5b5f6c977a3c097b7f903f1b9527df8f Signed-off-by: Adam Litke a...@us.ibm.com --- M vdsm_api/vdsmapi-schema.json 1 file changed, 43 insertions(+), 43 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/10/7010/1 -- To view, visit http://gerrit.ovirt.org/7010 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I64afff9e5b5f6c977a3c097b7f903f1b9527df8f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Remove remaining instance of IntStr
Adam Litke has uploaded a new change for review. Change subject: schema: Remove remaining instance of IntStr .. schema: Remove remaining instance of IntStr Oops, we missed one IntStr type. Remove it. Change-Id: I5a906652eed51f2d575d8d31fb598360edcd977f Signed-off-by: Adam Litke a...@us.ibm.com --- M vdsm_api/vdsmapi-schema.json 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/09/7009/1 -- To view, visit http://gerrit.ovirt.org/7009 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I5a906652eed51f2d575d8d31fb598360edcd977f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Remove invalid names
Adam Litke has uploaded a new change for review. Change subject: schema: Remove invalid names .. schema: Remove invalid names The identifier type causes conflicts with the GObject type system. Type is not a great name anyway so choose better names where it occurs. The name StorageDomainClass conflicts with the GObject class structure for the StorageDomain class. Rename it to StorageDomainImageClass since it identifies the class of images stored on the storage domain. Change-Id: Idb43f5960f60a2064a5a41eeafcb7a7ef1899c77 Signed-off-by: Adam Litke a...@us.ibm.com --- M vdsm_api/vdsmapi-schema.json 1 file changed, 79 insertions(+), 77 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/13/7013/1 -- To view, visit http://gerrit.ovirt.org/7013 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Idb43f5960f60a2064a5a41eeafcb7a7ef1899c77 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Add class constructors to the schema
Adam Litke has uploaded a new change for review. Change subject: schema: Add class constructors to the schema .. schema: Add class constructors to the schema All classes will accept parameters when they are instantiated. Record the parameters in the schema. Change-Id: Ifee309b2459da125d351b218accb0f4eff0d78a3 Signed-off-by: Adam Litke a...@us.ibm.com --- M vdsm_api/vdsmapi-schema.json 1 file changed, 61 insertions(+), 35 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/11/7011/1 -- To view, visit http://gerrit.ovirt.org/7011 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ifee309b2459da125d351b218accb0f4eff0d78a3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Fix some typos
Adam Litke has uploaded a new change for review. Change subject: schema: Fix some typos .. schema: Fix some typos Just fix a few outstanding typos in the schema. This should be pretty uncontroversial. Change-Id: I1bf77059f36c3a5252031495007c9f094edf7bcf Signed-off-by: Adam Litke a...@us.ibm.com --- M vdsm_api/vdsmapi-schema.json 1 file changed, 6 insertions(+), 6 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/12/7012/1 -- To view, visit http://gerrit.ovirt.org/7012 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I1bf77059f36c3a5252031495007c9f094edf7bcf Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Fixup command return values
Adam Litke has uploaded a new change for review. Change subject: schema: Fixup command return values .. schema: Fixup command return values Commands return a single unnamed typed item. Fix the schema to reflect this. Some vdsm commands return multiple keys in the result dictionary and those keys need to be combined into a new type representing the complete return value. Change-Id: Id5e10d4cfeb9734ee0c8d36a7c9c24022e7b3e61 Signed-off-by: Adam Litke a...@us.ibm.com --- M vdsm_api/vdsmapi-schema.json 1 file changed, 167 insertions(+), 149 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/17/7017/1 -- To view, visit http://gerrit.ovirt.org/7017 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id5e10d4cfeb9734ee0c8d36a7c9c24022e7b3e61 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Fix up VmDefinition types
Adam Litke has uploaded a new change for review. Change subject: schema: Fix up VmDefinition types .. schema: Fix up VmDefinition types There is no such thing as a VmShort definition so we do not need the union type for VmDefinition. Also, VmDefinition is the same as VmStatus so just define the definition as an alias of VmFullStatus. Change-Id: Iecb73a3ca21908ca3bcb51a3d2c51319291296d9 Signed-off-by: Adam Litke a...@us.ibm.com --- M vdsm_api/vdsmapi-schema.json 1 file changed, 12 insertions(+), 12 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/15/7015/1 -- To view, visit http://gerrit.ovirt.org/7015 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iecb73a3ca21908ca3bcb51a3d2c51319291296d9 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Add StorageDomainStatsMap map type
Adam Litke has uploaded a new change for review. Change subject: schema: Add StorageDomainStatsMap map type .. schema: Add StorageDomainStatsMap map type API.getStorageRepoStats refers to an undefined mapping type. Add it. Change-Id: I3a0b0fe634b858a4fd8be935be0f02fae8c132b0 Signed-off-by: Adam Litke a...@us.ibm.com --- M vdsm_api/vdsmapi-schema.json 1 file changed, 10 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/14/7014/1 -- To view, visit http://gerrit.ovirt.org/7014 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3a0b0fe634b858a4fd8be935be0f02fae8c132b0 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Introduce a Test object
Adam Litke has uploaded a new change for review. Change subject: schema: Introduce a Test object .. schema: Introduce a Test object Add a Test Class that can be used for testing complex argument and return value handling. Currently I am envisioning using this for unit tests where we don't want to connect to a real vdsm server. Change-Id: I66c7f17f243c92a0cf6260b4f872477413fe6d49 Signed-off-by: Adam Litke a...@us.ibm.com --- M vdsm_api/vdsmapi-schema.json 1 file changed, 51 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/18/7018/1 -- To view, visit http://gerrit.ovirt.org/7018 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I66c7f17f243c92a0cf6260b4f872477413fe6d49 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: [RFC] Support for GLUSTERFS_DOMAIN
Deepak C Shetty has posted comments on this change. Change subject: [RFC] Support for GLUSTERFS_DOMAIN .. Patch Set 1: (8 inline comments) I have posted v2. Pls provide your review comments. File vdsm/libvirtvm.py Line 88: self._vm._dom.blockInfo(vmDrive.path, 0) Done Line 991: # Determine if its network type Done Line 1000: def blockDev(self): Done Line 1003: self._blockDev = utils.isBlockDevice(self.path['path']) Done Line 1521: if drv.path == diskParams['path']: Done Line 1875: mergeStatus['path'] = mergeDrive.path Done Line 1993: capacity, alloc, physical = self._dom.blockInfo(d.path, 0) Done. For all the .path changes, I avoided them, by defining new key in drive dict by name 'vmVolInfo'. Posted v2 with the same. File vdsm/storage/volume.py Line 294: clsName, shareVolumeRollback, [dstPath])) Do you mean this change is unrelated to the patch. Yes, but i did it for pep8 compliance -- To view, visit http://gerrit.ovirt.org/6856 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9ac37da88625f20d148beaf53bb6371c15b33ad7 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Support for QEMU cmdline hook
Dan Kenigsberg has posted comments on this change. Change subject: Support for QEMU cmdline hook .. Patch Set 4: Looks good to me, approved annoying whitespace removed. -- To view, visit http://gerrit.ovirt.org/6969 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I48477926d271bb9f6ae74f79de5d8c4d37b87caa Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#846014 - Fix virtio hotplug disk fail to a VM with IDE di...
Dan Kenigsberg has submitted this change and it was merged. Change subject: BZ#846014 - Fix virtio hotplug disk fail to a VM with IDE disks. .. BZ#846014 - Fix virtio hotplug disk fail to a VM with IDE disks. Change-Id: Ia8f9a317dd07f3fb8f69ecf4c647e91e5e9064dc Signed-off-by: Eduardo ewars...@redhat.com --- M vdsm/vm.py 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Ayal Baron: Looks good to me, approved oVirt Jenkins CI Server: Verified Dan Kenigsberg: -- To view, visit http://gerrit.ovirt.org/6970 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8f9a317dd07f3fb8f69ecf4c647e91e5e9064dc Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Eduardo ewars...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Daniel Paikov pai...@gmail.com Gerrit-Reviewer: Eduardo ewars...@redhat.com Gerrit-Reviewer: Gal Hammer gham...@redhat.com Gerrit-Reviewer: Haim Ateya hat...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Add restoreBackups and clearBackups
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add restoreBackups and clearBackups .. Patch Set 4: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests/945/ -- To view, visit http://gerrit.ovirt.org/6976 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia0c74b29a77b935e82e817a93ab10faefb3709fa Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#832765 - Load backups from backup files to memory
oVirt Jenkins CI Server has posted comments on this change. Change subject: BZ#832765 - Load backups from backup files to memory .. Patch Set 7: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests/946/ -- To view, visit http://gerrit.ovirt.org/6967 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia2003e1b0b198fd14d69cc0ed59b2e6d0d8abe36 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#832765 - Rewrite vdsm-restore-net-config as python module
oVirt Jenkins CI Server has posted comments on this change. Change subject: BZ#832765 - Rewrite vdsm-restore-net-config as python module .. Patch Set 3: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests/947/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6991 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I68d07e3754305dfc7e02d67c59b7c801b9d80bc5 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#832765 - Load backups from backup files to memory
oVirt Jenkins CI Server has posted comments on this change. Change subject: BZ#832765 - Load backups from backup files to memory .. Patch Set 7: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests/946/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6967 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia2003e1b0b198fd14d69cc0ed59b2e6d0d8abe36 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#832765 - Rewrite vdsm-restore-net-config as python module
oVirt Jenkins CI Server has posted comments on this change. Change subject: BZ#832765 - Rewrite vdsm-restore-net-config as python module .. Patch Set 3: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests/947/ -- To view, visit http://gerrit.ovirt.org/6991 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I68d07e3754305dfc7e02d67c59b7c801b9d80bc5 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Livnat Peer lp...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/1/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/1/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/2/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/2/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/3/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/3/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/4/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/4/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/5/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/5/ : FAILURE -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Remove remaining instance of IntStr
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Remove remaining instance of IntStr .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/6/ -- To view, visit http://gerrit.ovirt.org/7009 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5a906652eed51f2d575d8d31fb598360edcd977f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Remove remaining instance of IntStr
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Remove remaining instance of IntStr .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/6/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7009 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5a906652eed51f2d575d8d31fb598360edcd977f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Rename Global class to API
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Rename Global class to API .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/7/ -- To view, visit http://gerrit.ovirt.org/7010 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I64afff9e5b5f6c977a3c097b7f903f1b9527df8f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Rename Global class to API
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Rename Global class to API .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/7/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7010 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I64afff9e5b5f6c977a3c097b7f903f1b9527df8f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Add class constructors to the schema
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Add class constructors to the schema .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/8/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7011 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifee309b2459da125d351b218accb0f4eff0d78a3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Fix some typos
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Fix some typos .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/9/ -- To view, visit http://gerrit.ovirt.org/7012 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1bf77059f36c3a5252031495007c9f094edf7bcf Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Fix some typos
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Fix some typos .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/9/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7012 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1bf77059f36c3a5252031495007c9f094edf7bcf Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Remove invalid names
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Remove invalid names .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/10/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7013 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idb43f5960f60a2064a5a41eeafcb7a7ef1899c77 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Add StorageDomainStatsMap map type
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Add StorageDomainStatsMap map type .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/11/ -- To view, visit http://gerrit.ovirt.org/7014 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3a0b0fe634b858a4fd8be935be0f02fae8c132b0 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Add StorageDomainStatsMap map type
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Add StorageDomainStatsMap map type .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/11/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7014 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3a0b0fe634b858a4fd8be935be0f02fae8c132b0 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Fix up VmDefinition types
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Fix up VmDefinition types .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/12/ -- To view, visit http://gerrit.ovirt.org/7015 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iecb73a3ca21908ca3bcb51a3d2c51319291296d9 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Fix up VmDefinition types
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Fix up VmDefinition types .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/12/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7015 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iecb73a3ca21908ca3bcb51a3d2c51319291296d9 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: ConnectionError should be an object not a tuple
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: ConnectionError should be an object not a tuple .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/13/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7016 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iac8c639f42f2ecf080aa41878a6050d9bc385ddc Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: ConnectionError should be an object not a tuple
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: ConnectionError should be an object not a tuple .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/13/ -- To view, visit http://gerrit.ovirt.org/7016 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iac8c639f42f2ecf080aa41878a6050d9bc385ddc Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Fixup command return values
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Fixup command return values .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/14/ -- To view, visit http://gerrit.ovirt.org/7017 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id5e10d4cfeb9734ee0c8d36a7c9c24022e7b3e61 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Fixup command return values
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Fixup command return values .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/14/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7017 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id5e10d4cfeb9734ee0c8d36a7c9c24022e7b3e61 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Introduce a Test object
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Introduce a Test object .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/15/ -- To view, visit http://gerrit.ovirt.org/7018 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I66c7f17f243c92a0cf6260b4f872477413fe6d49 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: schema: Introduce a Test object
oVirt Jenkins CI Server has posted comments on this change. Change subject: schema: Introduce a Test object .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/15/ : FAILURE -- To view, visit http://gerrit.ovirt.org/7018 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I66c7f17f243c92a0cf6260b4f872477413fe6d49 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke a...@us.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/17/ : FAILURE -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/17/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/16/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/16/ : FAILURE -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: [RFC] Support for GLUSTERFS_DOMAIN
oVirt Jenkins CI Server has posted comments on this change. Change subject: [RFC] Support for GLUSTERFS_DOMAIN .. Patch Set 2: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/27/ -- To view, visit http://gerrit.ovirt.org/6856 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9ac37da88625f20d148beaf53bb6371c15b33ad7 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: [RFC] Support for GLUSTERFS_DOMAIN
oVirt Jenkins CI Server has posted comments on this change. Change subject: [RFC] Support for GLUSTERFS_DOMAIN .. Patch Set 2: Fails Build Failed http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/27/ : ABORTED -- To view, visit http://gerrit.ovirt.org/6856 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9ac37da88625f20d148beaf53bb6371c15b33ad7 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/18/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/28/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/28/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/29/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/29/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
oVirt Jenkins CI Server has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 2: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/30/ -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: ShaoHe Feng shao...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: call sync before umount in mkFloppyFs
oVirt Jenkins CI Server has posted comments on this change. Change subject: call sync before umount in mkFloppyFs .. Patch Set 2: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/30/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I708e70aa1e898b32bb3c1009d411389d7adbe549 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wu...@linux.vnet.ibm.com Gerrit-Reviewer: ShaoHe Feng shao...@linux.vnet.ibm.com Gerrit-Reviewer: Zhou Zheng Sheng zhshz...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Support for QEMU cmdline hook
oVirt Jenkins CI Server has posted comments on this change. Change subject: Support for QEMU cmdline hook .. Patch Set 3: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/34/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6969 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I48477926d271bb9f6ae74f79de5d8c4d37b87caa Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Support for QEMU cmdline hook
oVirt Jenkins CI Server has posted comments on this change. Change subject: Support for QEMU cmdline hook .. Patch Set 3: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/34/ -- To view, visit http://gerrit.ovirt.org/6969 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I48477926d271bb9f6ae74f79de5d8c4d37b87caa Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Deepak C Shetty deepa...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#844656 Release the _syncroot lock during _findDomain
oVirt Jenkins CI Server has posted comments on this change. Change subject: BZ#844656 Release the _syncroot lock during _findDomain .. Patch Set 7: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/41/ -- To view, visit http://gerrit.ovirt.org/6822 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8088d5fe716a3a08c3e5cef2d2d9a654ee96f60a Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Eduardo ewars...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Omri Hochman hoo...@gmail.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Specify the ACPI flag when shutting down VMs
oVirt Jenkins CI Server has posted comments on this change. Change subject: Specify the ACPI flag when shutting down VMs .. Patch Set 1: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/18/ -- To view, visit http://gerrit.ovirt.org/6996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ca628ce6f4a5c7f6a937a14b12a5078b32e89dd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Bing Bu Cao m...@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: BZ#844656 Release the _syncroot lock during _findDomain
oVirt Jenkins CI Server has posted comments on this change. Change subject: BZ#844656 Release the _syncroot lock during _findDomain .. Patch Set 7: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/41/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/6822 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8088d5fe716a3a08c3e5cef2d2d9a654ee96f60a Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Eduardo ewars...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Omri Hochman hoo...@gmail.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Support the quiesce flag during a live snapshot
oVirt Jenkins CI Server has posted comments on this change. Change subject: Support the quiesce flag during a live snapshot .. Patch Set 5: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/47/ -- To view, visit http://gerrit.ovirt.org/2223 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I61c7c89fb42afa627b3e45cd74af03e032452a7d Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Support the quiesce flag during a live snapshot
oVirt Jenkins CI Server has posted comments on this change. Change subject: Support the quiesce flag during a live snapshot .. Patch Set 5: Build Successful http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/47/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/2223 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I61c7c89fb42afa627b3e45cd74af03e032452a7d Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Ayal Baron aba...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com Gerrit-Reviewer: Royce Lv lvro...@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Initialize the pthread RLock attribute
oVirt Jenkins CI Server has posted comments on this change. Change subject: Initialize the pthread RLock attribute .. Patch Set 3: No score Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/48/ -- To view, visit http://gerrit.ovirt.org/6988 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3d71606113c8bbb17620a753f534532510d92c6a Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com Gerrit-Reviewer: Omri Hochman hoo...@gmail.com Gerrit-Reviewer: Saggi Mizrahi smizr...@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
