vdsm-patches June 2012

vdsm-patches@lists.fedorahosted.org

45 participants
405 discussions

Change in vdsm[master]: Close and remove the temporary file before raising a excepti...
by wenyi＠linux.vnet.ibm.com 07 Jun '12

07 Jun '12

Wenyi Gao has uploaded a new change for review. Change subject: Close and remove the temporary file before raising a exception ...................................................................... Close and remove the temporary file before raising a exception Change-Id: I93eafbcf78805669596258b3579ee49cc1b79360 Signed-off-by: Wenyi Gao <wenyi(a)linux.vnet.ibm.com> --- M vdsm/storage/multipath.py 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/85/4885/1 -- To view, visit http://gerrit.ovirt.org/4885 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I93eafbcf78805669596258b3579ee49cc1b79360 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenyi Gao <wenyi(a)linux.vnet.ibm.com>

5 9

Change in vdsm[master]: Change storageServer to parse some params as strings
by yzaslavs＠redhat.com 07 Jun '12

07 Jun '12

Yair Zaslavsky has uploaded a new change for review. Change subject: Change storageServer to parse some params as strings ...................................................................... Change storageServer to parse some params as strings engine is sending the connection information as a map of string keys and string values. VDSM tries to parse version,retrans and timeout as numbers, but due to the nature of the passed map it receives a string containing numerical values for these arguments. Since these arguments are later on added to a comma delimited list I suggest to parse them as strings. Change-Id: I4d1716a6f66a108d60f39fad14d92625a692d0e3 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M vdsm/storage/storageServer.py 1 file changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/62/5062/1 -- To view, visit http://gerrit.ovirt.org/5062 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4d1716a6f66a108d60f39fad14d92625a692d0e3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com>

4 12

Change in vdsm[master]: config qemu-sanlock to ignore readonly lease for default
by wudxw＠linux.vnet.ibm.com 06 Jun '12

06 Jun '12

Mark Wu has posted comments on this change. Change subject: config qemu-sanlock to ignore readonly lease for default ...................................................................... Patch Set 1: I would prefer that you didn't submit this You also need bump the required libvirt version to v0.9.12 in the spec file of vdsm package. But it's still not available on f17. -- To view, visit http://gerrit.ovirt.org/5057 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I116f3ce07fff8075902a862e0259198232efd2f4 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv <lvroyce(a)linux.vnet.ibm.com> Gerrit-Reviewer: Bing Bu Cao <mars(a)linux.vnet.ibm.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Mark Wu <wudxw(a)linux.vnet.ibm.com> Gerrit-Reviewer: Royce Lv <lvroyce(a)linux.vnet.ibm.com>

1 0

Change in vdsm[master]: config qemu-sanlock to ignore readonly lease for default
by lvroyce＠linux.vnet.ibm.com 06 Jun '12

06 Jun '12

Royce Lv has posted comments on this change. Change subject: config qemu-sanlock to ignore readonly lease for default ...................................................................... Patch Set 1: Verified tested agaist: libvirt:libvirt-0.9.12-1.fc16.x86_64 vdsm:vdsm-4.9.6-0.269.git3e44fe0.fc16 -- To view, visit http://gerrit.ovirt.org/5057 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I116f3ce07fff8075902a862e0259198232efd2f4 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv <lvroyce(a)linux.vnet.ibm.com> Gerrit-Reviewer: Bing Bu Cao <mars(a)linux.vnet.ibm.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Royce Lv <lvroyce(a)linux.vnet.ibm.com>

1 0

Change in vdsm[master]: config qemu-sanlock to ignore readonly lease for default
by mars＠linux.vnet.ibm.com 06 Jun '12

06 Jun '12

Bing Bu Cao has posted comments on this change. Change subject: config qemu-sanlock to ignore readonly lease for default ...................................................................... Patch Set 1: (1 inline comment) .................................................... File vdsm/vdsmd.init.in Line 377: set_if_default $qlconf ignore_readonly_and_shared_disks 1 As I know,the "ignore_readonly_and_shared_disks" was just merged into libvirt(0.9.12) one month ago.And now the libvirt (0.9.11) depended by vdsm will ignore this parameter. I think the patch only/will be ok after libvirt updated. https://bugzilla.redhat.com/show_bug.cgi?id=828633 -- To view, visit http://gerrit.ovirt.org/5057 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I116f3ce07fff8075902a862e0259198232efd2f4 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv <lvroyce(a)linux.vnet.ibm.com> Gerrit-Reviewer: Bing Bu Cao <mars(a)linux.vnet.ibm.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>

1 0

Re: [oVirt Jenkins] vdsm_unit_tests - Build # 176 - Failure!
by lhornyak＠redhat.com 06 Jun '12

06 Jun '12

Hi, I ran this test a couple of times but but did not fail. Possibly a random error in the test environment? ----- Original Message ----- > From: "Jenkins oVirt Server" <jenkins(a)ovirt.org> > To: abaron(a)redhat.com, danken(a)redhat.com, vdsm-patches(a)lists.fedorahosted.org, lhornyak(a)redhat.com > Sent: Friday, June 1, 2012 10:37:31 AM > Subject: [oVirt Jenkins] vdsm_unit_tests - Build # 176 - Failure! > > > BUILD FAILURE > Build URL: http://jenkins.ovirt.org/job/vdsm_unit_tests/176/ > Project: vdsm_unit_tests > Date of build: Fri, 01 Jun 2012 04:30:36 -0400 > Build duration: 6 min 54 sec > > CHANGE SET > Revision 2c6cf8d18ef2eff973862171633d941795971dec by lhornyak: > (CpuPinning) > change: edit vdsm/libvirtvm.py > Revision 488ca38c39ba24533774dd5224c0d8c392d28d82 by lhornyak: > (CpuPinning for vdsClient) > change: edit vdsm_cli/vdsClient.py > > > JUNIT RESULTS > > Name: alignmentScanTests Failed: 0 test(s), Passed: 0 test(s), > Skipped: 4 test(s), Total: 4 test(s) > > > Name: betterPopenTests Failed: 0 test(s), Passed: 9 test(s), Skipped: > 1 test(s), Total: 10 test(s) > > > Name: betterThreadingTests Failed: 0 test(s), Passed: 16 test(s), > Skipped: 0 test(s), Total: 16 test(s) > > > Name: capsTests Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 > test(s), Total: 1 test(s) > > > Name: fileUtilTests Failed: 0 test(s), Passed: 5 test(s), Skipped: 2 > test(s), Total: 7 test(s) > > > Name: gluster_cli_tests Failed: 0 test(s), Passed: 2 test(s), > Skipped: 0 test(s), Total: 2 test(s) > > > Name: guestIFTests Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 > test(s), Total: 1 test(s) > > > Name: hooksTests Failed: 0 test(s), Passed: 6 test(s), Skipped: 0 > test(s), Total: 6 test(s) > > > Name: lsblkTests Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 > test(s), Total: 1 test(s) > > > Name: main Failed: 0 test(s), Passed: 3 test(s), Skipped: 0 test(s), > Total: 3 test(s) > > > Name: md_utils_tests Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 > test(s), Total: 1 test(s) > > > Name: miscTests Failed: 1 test(s), Passed: 63 test(s), Skipped: 0 > test(s), Total: 64 test(s) > > Failed: miscTests.DdWatchCopy.testStop > > Name: mountTests Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 > test(s), Total: 1 test(s) > > > Name: netinfoTests Failed: 0 test(s), Passed: 3 test(s), Skipped: 0 > test(s), Total: 3 test(s) > > > Name: parted_utils_tests Failed: 0 test(s), Passed: 0 test(s), > Skipped: 1 test(s), Total: 1 test(s) > > > Name: persistentDictTests Failed: 0 test(s), Passed: 2 test(s), > Skipped: 0 test(s), Total: 2 test(s) > > > Name: processPoolTests Failed: 0 test(s), Passed: 10 test(s), > Skipped: 0 test(s), Total: 10 test(s) > > > Name: resourceManagerTests Failed: 0 test(s), Passed: 33 test(s), > Skipped: 0 test(s), Total: 33 test(s) > > >

2 2

Change in vdsm[master]: CpuPinning for vdsClient (#514657)
by lhornyak＠redhat.com 05 Jun '12

05 Jun '12

Laszlo Hornyak has abandoned this change. Change subject: CpuPinning for vdsClient (#514657) ...................................................................... Patch Set 1: Abandoned -- To view, visit http://gerrit.usersys.redhat.com/1312 To unsubscribe, visit http://gerrit.usersys.redhat.com/settings Gerrit-MessageType: abandon Gerrit-Change-Id: Ia1cf4c55d23bda450cad1543ac4d622f2949e03d Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>

1 0

Change in vdsm[master]: CpuPinning (#514657)
by lhornyak＠redhat.com 05 Jun '12

05 Jun '12

Laszlo Hornyak has abandoned this change. Change subject: CpuPinning (#514657) ...................................................................... Patch Set 2: Abandoned -- To view, visit http://gerrit.usersys.redhat.com/1311 To unsubscribe, visit http://gerrit.usersys.redhat.com/settings Gerrit-MessageType: abandon Gerrit-Change-Id: Ia7016848d365d0e59454372461d66d167b100e7e Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>

1 0

Change in vdsm[master]: CpuPinning (#514657)
by Dan Kenigsberg 05 Jun '12

05 Jun '12

Dan Kenigsberg has posted comments on this change. Change subject: CpuPinning (#514657) ...................................................................... Patch Set 2: Do not submit please abandon this. master branch is unused in this downstream gerrit. -- To view, visit http://gerrit.usersys.redhat.com/1311 To unsubscribe, visit http://gerrit.usersys.redhat.com/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia7016848d365d0e59454372461d66d167b100e7e Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>

1 0

Change in vdsm[master]: CpuPinning for vdsClient (#514657)
by Dan Kenigsberg 05 Jun '12

05 Jun '12

Dan Kenigsberg has posted comments on this change. Change subject: CpuPinning for vdsClient (#514657) ...................................................................... Patch Set 1: Do not submit please abandon this. master branch is unused in this downstream gerrit. -- To view, visit http://gerrit.usersys.redhat.com/1312 To unsubscribe, visit http://gerrit.usersys.redhat.com/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia1cf4c55d23bda450cad1543ac4d622f2949e03d Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

vdsm-patches June 2012