summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhilipp Hahn <hahn@univention.de>2018-11-21 07:55:57 +0100
committerPhilipp Hahn <hahn@univention.de>2020-08-06 08:50:37 +0200
commitee5c856af7563619305a0c68ab32c06e6e6c1b2d (patch)
treec041309150f2865fb367e30dfa519fd4b036d7bc
parent0ca8dc6340a85d0cb021d860a166ad4b92251d23 (diff)
downloadlibvirt-python-ee5c856af7563619305a0c68ab32c06e6e6c1b2d.tar.gz
Remove legacy libvirtError arguments
The fields have been deprecated in C with git:f60dc0bc09f09c6817d6706a9edb1579a3e2b2b8 They are only passed to the libvirtError constructor, but not stored for later or used anywhere else. sed -ri '/raise libvirtError/s/, \w+=self(\._dom)?//' *.py Signed-off-by: Philipp Hahn <hahn@univention.de>
-rwxr-xr-xgenerator.py38
-rw-r--r--libvirt-override-virConnect.py52
-rw-r--r--libvirt-override-virDomain.py14
-rw-r--r--libvirt-override-virDomainCheckpoint.py2
-rw-r--r--libvirt-override-virDomainSnapshot.py2
-rw-r--r--libvirt-override-virNetwork.py2
-rw-r--r--libvirt-override-virStoragePool.py2
-rw-r--r--libvirt-override.py2
8 files changed, 57 insertions, 57 deletions
diff --git a/generator.py b/generator.py
index f69466c..0229df2 100755
--- a/generator.py
+++ b/generator.py
@@ -1720,31 +1720,31 @@ def buildWrappers(module):
else:
if classname == "virConnect":
classes.write(
- " if ret is None:raise libvirtError('%s() failed', conn=self)\n" %
+ " if ret is None:raise libvirtError('%s() failed')\n" %
(name))
elif classname == "virDomain":
classes.write(
- " if ret is None:raise libvirtError('%s() failed', dom=self)\n" %
+ " if ret is None:raise libvirtError('%s() failed')\n" %
(name))
elif classname == "virNetwork":
classes.write(
- " if ret is None:raise libvirtError('%s() failed', net=self)\n" %
+ " if ret is None:raise libvirtError('%s() failed')\n" %
(name))
elif classname == "virInterface":
classes.write(
- " if ret is None:raise libvirtError('%s() failed', net=self)\n" %
+ " if ret is None:raise libvirtError('%s() failed')\n" %
(name))
elif classname == "virStoragePool":
classes.write(
- " if ret is None:raise libvirtError('%s() failed', pool=self)\n" %
+ " if ret is None:raise libvirtError('%s() failed')\n" %
(name))
elif classname == "virStorageVol":
classes.write(
- " if ret is None:raise libvirtError('%s() failed', vol=self)\n" %
+ " if ret is None:raise libvirtError('%s() failed')\n" %
(name))
elif classname in ["virDomainCheckpoint", "virDomainSnapshot"]:
classes.write(
- " if ret is None:raise libvirtError('%s() failed', dom=self._dom)\n" %
+ " if ret is None:raise libvirtError('%s() failed')\n" %
(name))
else:
classes.write(
@@ -1776,27 +1776,27 @@ def buildWrappers(module):
test = functions_int_default_test
if classname == "virConnect":
classes.write((" if " + test +
- ": raise libvirtError('%s() failed', conn=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virDomain":
classes.write((" if " + test +
- ": raise libvirtError('%s() failed', dom=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virNetwork":
classes.write((" if " + test +
- ": raise libvirtError('%s() failed', net=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virInterface":
classes.write((" if " + test +
- ": raise libvirtError('%s() failed', net=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virStoragePool":
classes.write((" if " + test +
- ": raise libvirtError('%s() failed', pool=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virStorageVol":
classes.write((" if " + test +
- ": raise libvirtError('%s() failed', vol=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
else:
classes.write((" if " + test +
@@ -1809,27 +1809,27 @@ def buildWrappers(module):
if name not in functions_noexcept:
if classname == "virConnect":
classes.write((" if %s is None" +
- ": raise libvirtError('%s() failed', conn=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virDomain":
classes.write((" if %s is None" +
- ": raise libvirtError('%s() failed', dom=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virNetwork":
classes.write((" if %s is None" +
- ": raise libvirtError('%s() failed', net=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virInterface":
classes.write((" if %s is None" +
- ": raise libvirtError('%s() failed', net=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virStoragePool":
classes.write((" if %s is None" +
- ": raise libvirtError('%s() failed', pool=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
elif classname == "virStorageVol":
classes.write((" if %s is None" +
- ": raise libvirtError('%s() failed', vol=self)\n") %
+ ": raise libvirtError('%s() failed')\n") %
("ret", name))
else:
classes.write((" if %s is None" +
diff --git a/libvirt-override-virConnect.py b/libvirt-override-virConnect.py
index cf2b269..ed5f45b 100644
--- a/libvirt-override-virConnect.py
+++ b/libvirt-override-virConnect.py
@@ -26,7 +26,7 @@
del self.domainEventCallbacks
ret = libvirtmod.virConnectDomainEventDeregister(self._o, self)
if ret == -1:
- raise libvirtError('virConnectDomainEventDeregister() failed', conn=self)
+ raise libvirtError('virConnectDomainEventDeregister() failed')
except AttributeError:
pass
@@ -39,7 +39,7 @@
self.domainEventCallbacks = {cb: opaque}
ret = libvirtmod.virConnectDomainEventRegister(self._o, self)
if ret == -1:
- raise libvirtError('virConnectDomainEventRegister() failed', conn=self)
+ raise libvirtError('virConnectDomainEventRegister() failed')
def _dispatchDomainEventCallbacks(self, dom, event, detail):
"""Dispatches events to python user domain event callbacks
@@ -275,7 +275,7 @@
try:
ret = libvirtmod.virConnectDomainEventDeregisterAny(self._o, callbackID)
if ret == -1:
- raise libvirtError('virConnectDomainEventDeregisterAny() failed', conn=self)
+ raise libvirtError('virConnectDomainEventDeregisterAny() failed')
del self.domainEventCallbackID[callbackID]
except AttributeError:
pass
@@ -295,7 +295,7 @@
try:
ret = libvirtmod.virConnectNetworkEventDeregisterAny(self._o, callbackID)
if ret == -1:
- raise libvirtError('virConnectNetworkEventDeregisterAny() failed', conn=self)
+ raise libvirtError('virConnectNetworkEventDeregisterAny() failed')
del self.networkEventCallbackID[callbackID]
except AttributeError:
pass
@@ -311,7 +311,7 @@
else:
ret = libvirtmod.virConnectNetworkEventRegisterAny(self._o, net._o, eventID, cbData)
if ret == -1:
- raise libvirtError('virConnectNetworkEventRegisterAny() failed', conn=self)
+ raise libvirtError('virConnectNetworkEventRegisterAny() failed')
self.networkEventCallbackID[ret] = opaque
return ret
@@ -326,7 +326,7 @@
else:
ret = libvirtmod.virConnectDomainEventRegisterAny(self._o, dom._o, eventID, cbData)
if ret == -1:
- raise libvirtError('virConnectDomainEventRegisterAny() failed', conn=self)
+ raise libvirtError('virConnectDomainEventRegisterAny() failed')
self.domainEventCallbackID[ret] = opaque
return ret
@@ -356,7 +356,7 @@
try:
ret = libvirtmod.virConnectStoragePoolEventDeregisterAny(self._o, callbackID)
if ret == -1:
- raise libvirtError('virConnectStoragePoolEventDeregisterAny() failed', conn=self)
+ raise libvirtError('virConnectStoragePoolEventDeregisterAny() failed')
del self.storagePoolEventCallbackID[callbackID]
except AttributeError:
pass
@@ -372,7 +372,7 @@
else:
ret = libvirtmod.virConnectStoragePoolEventRegisterAny(self._o, pool._o, eventID, cbData)
if ret == -1:
- raise libvirtError('virConnectStoragePoolEventRegisterAny() failed', conn=self)
+ raise libvirtError('virConnectStoragePoolEventRegisterAny() failed')
self.storagePoolEventCallbackID[ret] = opaque
return ret
@@ -402,7 +402,7 @@
try:
ret = libvirtmod.virConnectNodeDeviceEventDeregisterAny(self._o, callbackID)
if ret == -1:
- raise libvirtError('virConnectNodeDeviceEventDeregisterAny() failed', conn=self)
+ raise libvirtError('virConnectNodeDeviceEventDeregisterAny() failed')
del self.nodeDeviceEventCallbackID[callbackID]
except AttributeError:
pass
@@ -418,7 +418,7 @@
else:
ret = libvirtmod.virConnectNodeDeviceEventRegisterAny(self._o, dev._o, eventID, cbData)
if ret == -1:
- raise libvirtError('virConnectNodeDeviceEventRegisterAny() failed', conn=self)
+ raise libvirtError('virConnectNodeDeviceEventRegisterAny() failed')
self.nodeDeviceEventCallbackID[ret] = opaque
return ret
@@ -446,7 +446,7 @@
try:
ret = libvirtmod.virConnectSecretEventDeregisterAny(self._o, callbackID)
if ret == -1:
- raise libvirtError('virConnectSecretEventDeregisterAny() failed', conn=self)
+ raise libvirtError('virConnectSecretEventDeregisterAny() failed')
del self.secretEventCallbackID[callbackID]
except AttributeError:
pass
@@ -462,7 +462,7 @@
else:
ret = libvirtmod.virConnectSecretEventRegisterAny(self._o, secret._o, eventID, cbData)
if ret == -1:
- raise libvirtError('virConnectSecretEventRegisterAny() failed', conn=self)
+ raise libvirtError('virConnectSecretEventRegisterAny() failed')
self.secretEventCallbackID[ret] = opaque
return ret
@@ -470,7 +470,7 @@
"""List all domains and returns a list of domain objects"""
ret = libvirtmod.virConnectListAllDomains(self._o, flags)
if ret is None:
- raise libvirtError("virConnectListAllDomains() failed", conn=self)
+ raise libvirtError("virConnectListAllDomains() failed")
retlist = list()
for domptr in ret:
@@ -482,7 +482,7 @@
"""Returns a list of storage pool objects"""
ret = libvirtmod.virConnectListAllStoragePools(self._o, flags)
if ret is None:
- raise libvirtError("virConnectListAllStoragePools() failed", conn=self)
+ raise libvirtError("virConnectListAllStoragePools() failed")
retlist = list()
for poolptr in ret:
@@ -494,7 +494,7 @@
"""Returns a list of network objects"""
ret = libvirtmod.virConnectListAllNetworks(self._o, flags)
if ret is None:
- raise libvirtError("virConnectListAllNetworks() failed", conn=self)
+ raise libvirtError("virConnectListAllNetworks() failed")
retlist = list()
for netptr in ret:
@@ -506,7 +506,7 @@
"""Returns a list of interface objects"""
ret = libvirtmod.virConnectListAllInterfaces(self._o, flags)
if ret is None:
- raise libvirtError("virConnectListAllInterfaces() failed", conn=self)
+ raise libvirtError("virConnectListAllInterfaces() failed")
retlist = list()
for ifaceptr in ret:
@@ -518,7 +518,7 @@
"""Returns a list of host node device objects"""
ret = libvirtmod.virConnectListAllNodeDevices(self._o, flags)
if ret is None:
- raise libvirtError("virConnectListAllNodeDevices() failed", conn=self)
+ raise libvirtError("virConnectListAllNodeDevices() failed")
retlist = list()
for devptr in ret:
@@ -530,7 +530,7 @@
"""Returns a list of network filter objects"""
ret = libvirtmod.virConnectListAllNWFilters(self._o, flags)
if ret is None:
- raise libvirtError("virConnectListAllNWFilters() failed", conn=self)
+ raise libvirtError("virConnectListAllNWFilters() failed")
retlist = list()
for filter_ptr in ret:
@@ -542,7 +542,7 @@
"""Returns a list of network filter binding objects"""
ret = libvirtmod.virConnectListAllNWFilterBindings(self._o, flags)
if ret is None:
- raise libvirtError("virConnectListAllNWFilterBindings() failed", conn=self)
+ raise libvirtError("virConnectListAllNWFilterBindings() failed")
retlist = list()
for filter_ptr in ret:
@@ -554,7 +554,7 @@
"""Returns a list of secret objects"""
ret = libvirtmod.virConnectListAllSecrets(self._o, flags)
if ret is None:
- raise libvirtError("virConnectListAllSecrets() failed", conn=self)
+ raise libvirtError("virConnectListAllSecrets() failed")
retlist = list()
for secret_ptr in ret:
@@ -574,7 +574,7 @@
"""Removes a close event callback"""
ret = libvirtmod.virConnectUnregisterCloseCallback(self._o)
if ret == -1:
- raise libvirtError('virConnectUnregisterCloseCallback() failed', conn=self)
+ raise libvirtError('virConnectUnregisterCloseCallback() failed')
def registerCloseCallback(self, cb, opaque):
"""Adds a close event callback, providing a notification
@@ -582,7 +582,7 @@
cbData = {"cb": cb, "conn": self, "opaque": opaque}
ret = libvirtmod.virConnectRegisterCloseCallback(self._o, cbData)
if ret == -1:
- raise libvirtError('virConnectRegisterCloseCallback() failed', conn=self)
+ raise libvirtError('virConnectRegisterCloseCallback() failed')
return ret
def createXMLWithFiles(self, xmlDesc, files, flags=0):
@@ -612,7 +612,7 @@
block attempts at migration, save-to-file, or snapshots. """
ret = libvirtmod.virDomainCreateXMLWithFiles(self._o, xmlDesc, files, flags)
if ret is None:
- raise libvirtError('virDomainCreateXMLWithFiles() failed', conn=self)
+ raise libvirtError('virDomainCreateXMLWithFiles() failed')
__tmp = virDomain(self, _obj=ret)
return __tmp
@@ -660,7 +660,7 @@
VIR_CONNECT_GET_ALL_DOMAINS_STATS_OTHER for all other states. """
ret = libvirtmod.virConnectGetAllDomainStats(self._o, stats, flags)
if ret is None:
- raise libvirtError("virConnectGetAllDomainStats() failed", conn=self)
+ raise libvirtError("virConnectGetAllDomainStats() failed")
retlist = list()
for elem in ret:
@@ -701,13 +701,13 @@
domlist = list()
for dom in doms:
if not isinstance(dom, virDomain):
- raise libvirtError("domain list contains non-domain elements", conn=self)
+ raise libvirtError("domain list contains non-domain elements")
domlist.append(dom._o)
ret = libvirtmod.virDomainListGetStats(self._o, domlist, stats, flags)
if ret is None:
- raise libvirtError("virDomainListGetStats() failed", conn=self)
+ raise libvirtError("virDomainListGetStats() failed")
retlist = list()
for elem in ret:
diff --git a/libvirt-override-virDomain.py b/libvirt-override-virDomain.py
index 7541c01..4e42053 100644
--- a/libvirt-override-virDomain.py
+++ b/libvirt-override-virDomain.py
@@ -2,7 +2,7 @@
"""List all snapshots and returns a list of snapshot objects"""
ret = libvirtmod.virDomainListAllSnapshots(self._o, flags)
if ret is None:
- raise libvirtError("virDomainListAllSnapshots() failed", conn=self)
+ raise libvirtError("virDomainListAllSnapshots() failed")
retlist = list()
for snapptr in ret:
@@ -14,7 +14,7 @@
"""List all checkpoints and returns a list of checkpoint objects"""
ret = libvirtmod.virDomainListAllCheckpoints(self._o, flags)
if ret is None:
- raise libvirtError("virDomainListAllCheckpoints() failed", conn=self)
+ raise libvirtError("virDomainListAllCheckpoints() failed")
retlist = list()
for chkptr in ret:
@@ -57,28 +57,28 @@
file for this domain is discarded, and the domain boots from scratch. """
ret = libvirtmod.virDomainCreateWithFiles(self._o, files, flags)
if ret == -1:
- raise libvirtError('virDomainCreateWithFiles() failed', dom=self)
+ raise libvirtError('virDomainCreateWithFiles() failed')
return ret
def fsFreeze(self, mountpoints=None, flags=0):
"""Freeze specified filesystems within the guest """
ret = libvirtmod.virDomainFSFreeze(self._o, mountpoints, flags)
if ret == -1:
- raise libvirtError('virDomainFSFreeze() failed', dom=self)
+ raise libvirtError('virDomainFSFreeze() failed')
return ret
def fsThaw(self, mountpoints=None, flags=0):
"""Thaw specified filesystems within the guest """
ret = libvirtmod.virDomainFSThaw(self._o, mountpoints, flags)
if ret == -1:
- raise libvirtError('virDomainFSThaw() failed', dom=self)
+ raise libvirtError('virDomainFSThaw() failed')
return ret
def getTime(self, flags=0):
"""Extract information about guest time """
ret = libvirtmod.virDomainGetTime(self._o, flags)
if ret == None:
- raise libvirtError('virDomainGetTime() failed', dom=self)
+ raise libvirtError('virDomainGetTime() failed')
return ret
def setTime(self, time=None, flags=0):
@@ -86,5 +86,5 @@
'seconds' field for seconds and 'nseconds' field for nanoseconds """
ret = libvirtmod.virDomainSetTime(self._o, time, flags)
if ret == -1:
- raise libvirtError('virDomainSetTime() failed', dom=self)
+ raise libvirtError('virDomainSetTime() failed')
return ret
diff --git a/libvirt-override-virDomainCheckpoint.py b/libvirt-override-virDomainCheckpoint.py
index 14e750e..d4ccd88 100644
--- a/libvirt-override-virDomainCheckpoint.py
+++ b/libvirt-override-virDomainCheckpoint.py
@@ -10,7 +10,7 @@
"""List all child checkpoints and returns a list of checkpoint objects"""
ret = libvirtmod.virDomainCheckpointListAllChildren(self._o, flags)
if ret is None:
- raise libvirtError("virDomainCheckpointListAllChildren() failed", conn=self)
+ raise libvirtError("virDomainCheckpointListAllChildren() failed")
retlist = list()
for chkptr in ret:
diff --git a/libvirt-override-virDomainSnapshot.py b/libvirt-override-virDomainSnapshot.py
index 87eac86..fd4b311 100644
--- a/libvirt-override-virDomainSnapshot.py
+++ b/libvirt-override-virDomainSnapshot.py
@@ -10,7 +10,7 @@
"""List all child snapshots and returns a list of snapshot objects"""
ret = libvirtmod.virDomainSnapshotListAllChildren(self._o, flags)
if ret is None:
- raise libvirtError("virDomainSnapshotListAllChildren() failed", conn=self)
+ raise libvirtError("virDomainSnapshotListAllChildren() failed")
retlist = list()
for snapptr in ret:
diff --git a/libvirt-override-virNetwork.py b/libvirt-override-virNetwork.py
index 6e86590..df38914 100644
--- a/libvirt-override-virNetwork.py
+++ b/libvirt-override-virNetwork.py
@@ -2,7 +2,7 @@
"""List all ports on the network and returns a list of network port objects"""
ret = libvirtmod.virNetworkListAllPorts(self._o, flags)
if ret is None:
- raise libvirtError("virNetworkListAllPorts() failed", conn=self)
+ raise libvirtError("virNetworkListAllPorts() failed")
retlist = list()
for domptr in ret:
diff --git a/libvirt-override-virStoragePool.py b/libvirt-override-virStoragePool.py
index 94ced00..8bd5745 100644
--- a/libvirt-override-virStoragePool.py
+++ b/libvirt-override-virStoragePool.py
@@ -2,7 +2,7 @@
"""List all storage volumes and returns a list of storage volume objects"""
ret = libvirtmod.virStoragePoolListAllVolumes(self._o, flags)
if ret is None:
- raise libvirtError("virStoragePoolListAllVolumes() failed", conn=self)
+ raise libvirtError("virStoragePoolListAllVolumes() failed")
retlist = list()
for volptr in ret:
diff --git a/libvirt-override.py b/libvirt-override.py
index ff422a6..192831e 100644
--- a/libvirt-override.py
+++ b/libvirt-override.py
@@ -19,7 +19,7 @@ except ImportError:
# The root of all libvirt errors.
class libvirtError(Exception):
- def __init__(self, defmsg, conn=None, dom=None, net=None, pool=None, vol=None):
+ def __init__(self, defmsg):
# Never call virConnGetLastError().
# virGetLastError() is now thread local