March 2013 - vdsm-patches - Fedora Mailing-Lists

Change in vdsm[master]: [wip] Implement extendVolume in fileSD/Volume.

by Jenkins CI RO

oVirt Jenkins CI Server has posted comments on this change. Change subject: [wip] Implement extendVolume in fileSD/Volume. ...................................................................... Patch Set 2: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1447/ (2/2) -- To view, visit http://gerrit.ovirt.org/12713 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I15d73fd9e7727d9112db05ab892771b37d81152e Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 1 month

1
0
0 / 0

Change in vdsm[master]: [wip] Implement extendVolume in fileSD/Volume.

by Jenkins CI RO

oVirt Jenkins CI Server has posted comments on this change. Change subject: [wip] Implement extendVolume in fileSD/Volume. ...................................................................... Patch Set 2: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1484/ (1/2) -- To view, visit http://gerrit.ovirt.org/12713 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I15d73fd9e7727d9112db05ab892771b37d81152e Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 1 month

1
0
0 / 0

Change in vdsm[master]: [wip] Implement extendVolume in fileSD/Volume.

by Federico Simoncelli

Federico Simoncelli has posted comments on this change. Change subject: [wip] Implement extendVolume in fileSD/Volume. ...................................................................... Patch Set 1: Sadly to be fully working you have to expose this command in a never-ending list of files: BindingXMLRPC.py, API.py and hsm.py. You should also expose the command with vdsClient (which is also good if you want to test it in order to tick the "Verified" flag here). -- To view, visit http://gerrit.ovirt.org/12713 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I15d73fd9e7727d9112db05ab892771b37d81152e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 1 month

1
0
0 / 0

Change in vdsm[master]: [wip] Implement extendVolume in fileSD/Volume. Fix missing c...

by Jenkins CI RO

oVirt Jenkins CI Server has posted comments on this change. Change subject: [wip] Implement extendVolume in fileSD/Volume. Fix missing colon. ...................................................................... Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1483/ : FAILURE http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1446/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12717 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic632eea0fe6abe0af8406b2b21bf8878c11e8002 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 1 month

1
0
0 / 0

Change in vdsm[master]: [wip] Implement extendVolume in fileSD/Volume. Fix missing c...

by Jenkins CI RO

oVirt Jenkins CI Server has posted comments on this change. Change subject: [wip] Implement extendVolume in fileSD/Volume. Fix missing colon. ...................................................................... Patch Set 1: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1483/ (2/2) -- To view, visit http://gerrit.ovirt.org/12717 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic632eea0fe6abe0af8406b2b21bf8878c11e8002 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 1 month

1
0
0 / 0

Change in vdsm[master]: [wip] Implement extendVolume in fileSD/Volume. Fix missing c...

by Jenkins CI RO

oVirt Jenkins CI Server has posted comments on this change. Change subject: [wip] Implement extendVolume in fileSD/Volume. Fix missing colon. ...................................................................... Patch Set 1: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1446/ (1/2) -- To view, visit http://gerrit.ovirt.org/12717 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic632eea0fe6abe0af8406b2b21bf8878c11e8002 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 1 month

1
0
0 / 0

Change in vdsm[master]: [wip] Implement extendVolume in fileSD/Volume.

by Jenkins CI RO

oVirt Jenkins CI Server has posted comments on this change. Change subject: [wip] Implement extendVolume in fileSD/Volume. ...................................................................... Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1482/ : FAILURE http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1445/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12713 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I15d73fd9e7727d9112db05ab892771b37d81152e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 1 month

1
0
0 / 0

Change in vdsm[master]: [wip] Implement extendVolume in fileSD/Volume.

by Jenkins CI RO

oVirt Jenkins CI Server has posted comments on this change. Change subject: [wip] Implement extendVolume in fileSD/Volume. ...................................................................... Patch Set 1: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1482/ (2/2) -- To view, visit http://gerrit.ovirt.org/12713 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I15d73fd9e7727d9112db05ab892771b37d81152e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 1 month

1
0
0 / 0

Change in vdsm[master]: [wip] Implement extendVolume in fileSD/Volume.

by Jenkins CI RO

oVirt Jenkins CI Server has posted comments on this change. Change subject: [wip] Implement extendVolume in fileSD/Volume. ...................................................................... Patch Set 1: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1445/ (1/2) -- To view, visit http://gerrit.ovirt.org/12713 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I15d73fd9e7727d9112db05ab892771b37d81152e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Richard Marston <mail(a)richardmarston.net> Gerrit-Reviewer: oVirt Jenkins CI Server

11 years, 1 month

1
0
0 / 0

Change in vdsm[master]: Don't crash on libvirt network re-definition.

by asegurap＠redhat.com

Antoni Segura Puimedon has uploaded a new change for review. Change subject: Don't crash on libvirt network re-definition. ...................................................................... Don't crash on libvirt network re-definition. This patch is aimed to solve the libvirt syncing bug described in bugurl. In short, the vdsmd restart scenario is solved by the previous patch (30c63c5). This one solves the case when the interface backing a bridge/network has disappeared (and thus is not reported by netinfo to the engine) and then the engine sends a new setupnetwork that attempts to recreate the partially defined network. Up until now, libvirt would report an exception. Now we will not raise that exception unless the network that is being recreated is different than the one libvirt knows about. Bug-Url: https://bugzilla.redhat.com/861701 Change-Id: I76d95e20b7aa99280e604abdb1663c6c5c7dd32e Signed-off-by: Antoni S. Puimedon <asegurap(a)redhat.com> --- M vdsm/configNetwork.py 1 file changed, 16 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/01/10901/1 diff --git a/vdsm/configNetwork.py b/vdsm/configNetwork.py index e784ac1..c046dca 100755 --- a/vdsm/configNetwork.py +++ b/vdsm/configNetwork.py @@ -27,6 +27,7 @@ import logging import threading from xml.sax.saxutils import escape +from xml.dom.minidom import parseString import glob import shutil @@ -204,7 +205,21 @@ (escape(netName), escape(iface))) if not skipBackup: self._networkBackup(network) - self._createNetwork(netXml) + try: + self._createNetwork(netXml) + except libvirt.libvirtError: + conn = libvirtconnection.get() + if netName not in conn.listNetworks(): + raise + net = conn.networkLookupByName(netName) + if bridged: + if net.bridgeName() != network: + raise + else: + if parseString(net.XMLDesc(0)).\ + getElementsByTagName('interface')[0].\ + getAttribute('dev') != iface: + raise def _removeNetwork(self, network): netName = netinfo.LIBVIRT_NET_PREFIX + network -- To view, visit http://gerrit.ovirt.org/10901 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I76d95e20b7aa99280e604abdb1663c6c5c7dd32e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Antoni Segura Puimedon <asegurap(a)redhat.com>

11 years, 1 month

4
38
0 / 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

vdsm-patches March 2013