10:41 a.m.
* Implemented Suspend and Resume APIs.
* Reused virsh functions for taking the Event Strings names from ENUM.
* Merged all Domain Lifecycle events signals into one signal.
Katerina Koukiou (5):
Implement Suspend method for Domain interface.
Implement Resume method for Domain interface.
utils: Introduce functions and macros for translating ENUMs to strings
events: Introduce virtDBusEventsDomainEventToString function
Merge all Domain lifecycle signals into one signal called Domain.
data/org.libvirt.Connect.xml | 54 +++-------------------------------------
data/org.libvirt.Domain.xml | 8 ++++++
src/domain.c | 42 +++++++++++++++++++++++++++++++
src/events.c | 59 +++++++++++++++++---------------------------
src/events.h | 3 +++
src/util.c | 27 ++++++++++++++++++++
src/util.h | 29 ++++++++++++++++++++++
test/test_connect.py | 10 +++-----
test/test_domain.py | 40 +++++++++++++++++++++++++-----
9 files changed, 172 insertions(+), 100 deletions(-)
--
2.15.0
10:41 a.m.
New subject: [libvirt] [dbus PATCH v2 1/5] Implement Suspend method for Domain interface.
Signed-off-by: Katerina Koukiou <kkoukiou(a)redhat.com>
---
data/org.libvirt.Domain.xml | 4 ++++
src/domain.c | 21 +++++++++++++++++++++
test/test_domain.py | 15 +++++++++++++++
3 files changed, 40 insertions(+)
diff --git a/data/org.libvirt.Domain.xml b/data/org.libvirt.Domain.xml
index 39212ff..f31d078 100644
--- a/data/org.libvirt.Domain.xml
+++ b/data/org.libvirt.Domain.xml
@@ -84,6 +84,10 @@
value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainUndefineFla...
<arg name="flags" type="u" direction="in"/>
</method>
+ <method name="Suspend">
+ <annotation name="org.gtk.GDBus.DocString"
+ value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainSuspend&quo...
+ </method>
<signal name="DeviceAdded">
<annotation name="org.gtk.GDBus.DocString"
value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#virConnectDomainEven...
diff --git a/src/domain.c b/src/domain.c
index 4ff9f60..cb13b6b 100644
--- a/src/domain.c
+++ b/src/domain.c
@@ -450,6 +450,26 @@ virtDBusDomainUndefine(GVariant *inArgs G_GNUC_UNUSED,
virtDBusUtilSetLastVirtError(error);
}
+static void
+virtDBusDomainSuspend(GVariant *inArgs G_GNUC_UNUSED,
+ GUnixFDList *inFDs G_GNUC_UNUSED,
+ const gchar *objectPath,
+ gpointer userData,
+ GVariant **outArgs G_GNUC_UNUSED,
+ GUnixFDList **outFDs G_GNUC_UNUSED,
+ GError **error)
+{
+ virtDBusConnect *connect = userData;
+ g_autoptr(virDomain) domain = NULL;
+
+ domain = virtDBusDomainGetVirDomain(connect, objectPath, error);
+ if (!domain)
+ return;
+
+ if (virDomainSuspend(domain) < 0)
+ virtDBusUtilSetLastVirtError(error);
+}
+
static virtDBusGDBusPropertyTable virtDBusDomainPropertyTable[] = {
{ "Name", virtDBusDomainGetName, NULL },
{ "UUID", virtDBusDomainGetUUID, NULL },
@@ -472,6 +492,7 @@ static virtDBusGDBusMethodTable virtDBusDomainMethodTable[] = {
{ "Reset", virtDBusDomainReset },
{ "Create", virtDBusDomainCreate },
{ "Undefine", virtDBusDomainUndefine },
+ { "Suspend", virtDBusDomainSuspend },
{ 0 }
};
diff --git a/test/test_domain.py b/test/test_domain.py
index e617b1b..bb1b498 100755
--- a/test/test_domain.py
+++ b/test/test_domain.py
@@ -67,6 +67,21 @@ class TestDomain(libvirttest.BaseTestClass):
self.main_loop()
+ def test_suspend(self):
+ def domain_suspended(name, path):
+ assert name == 'test'
+ assert isinstance(path, dbus.ObjectPath)
+ self.loop.quit()
+
+ self.connect.connect_to_signal('DomainSuspended', domain_suspended)
+
+ obj, domain = self.domain()
+ domain.Suspend()
+
+ state = obj.Get('org.libvirt.Domain', 'State',
dbus_interface=dbus.PROPERTIES_IFACE)
+ assert state == 'paused'
+
+ self.main_loop()
if __name__ == '__main__':
libvirttest.run()
--
2.15.0
10:41 a.m.
New subject: [libvirt] [dbus PATCH v2 2/5] Implement Resume method for Domain interface.
Signed-off-by: Katerina Koukiou <kkoukiou(a)redhat.com>
---
data/org.libvirt.Domain.xml | 4 ++++
src/domain.c | 21 +++++++++++++++++++++
test/test_domain.py | 17 +++++++++++++++++
3 files changed, 42 insertions(+)
diff --git a/data/org.libvirt.Domain.xml b/data/org.libvirt.Domain.xml
index f31d078..7679018 100644
--- a/data/org.libvirt.Domain.xml
+++ b/data/org.libvirt.Domain.xml
@@ -88,6 +88,10 @@
<annotation name="org.gtk.GDBus.DocString"
value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainSuspend&quo...
</method>
+ <method name="Resume">
+ <annotation name="org.gtk.GDBus.DocString"
+ value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainResume"...
+ </method>
<signal name="DeviceAdded">
<annotation name="org.gtk.GDBus.DocString"
value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#virConnectDomainEven...
diff --git a/src/domain.c b/src/domain.c
index cb13b6b..a530987 100644
--- a/src/domain.c
+++ b/src/domain.c
@@ -470,6 +470,26 @@ virtDBusDomainSuspend(GVariant *inArgs G_GNUC_UNUSED,
virtDBusUtilSetLastVirtError(error);
}
+static void
+virtDBusDomainResume(GVariant *inArgs G_GNUC_UNUSED,
+ GUnixFDList *inFDs G_GNUC_UNUSED,
+ const gchar *objectPath,
+ gpointer userData,
+ GVariant **outArgs G_GNUC_UNUSED,
+ GUnixFDList **outFDs G_GNUC_UNUSED,
+ GError **error)
+{
+ virtDBusConnect *connect = userData;
+ g_autoptr(virDomain) domain = NULL;
+
+ domain = virtDBusDomainGetVirDomain(connect, objectPath, error);
+ if (!domain)
+ return;
+
+ if (virDomainResume(domain) < 0)
+ virtDBusUtilSetLastVirtError(error);
+}
+
static virtDBusGDBusPropertyTable virtDBusDomainPropertyTable[] = {
{ "Name", virtDBusDomainGetName, NULL },
{ "UUID", virtDBusDomainGetUUID, NULL },
@@ -493,6 +513,7 @@ static virtDBusGDBusMethodTable virtDBusDomainMethodTable[] = {
{ "Create", virtDBusDomainCreate },
{ "Undefine", virtDBusDomainUndefine },
{ "Suspend", virtDBusDomainSuspend },
+ { "Resume", virtDBusDomainResume },
{ 0 }
};
diff --git a/test/test_domain.py b/test/test_domain.py
index bb1b498..1433ff5 100755
--- a/test/test_domain.py
+++ b/test/test_domain.py
@@ -83,5 +83,22 @@ class TestDomain(libvirttest.BaseTestClass):
self.main_loop()
+ def test_resume(self):
+ def domain_resumed(name, path):
+ assert name == 'test'
+ assert isinstance(path, dbus.ObjectPath)
+ self.loop.quit()
+
+ self.connect.connect_to_signal('DomainResumed', domain_resumed)
+
+ obj, domain = self.domain()
+ domain.Suspend()
+ domain.Resume()
+
+ state = obj.Get('org.libvirt.Domain', 'State',
dbus_interface=dbus.PROPERTIES_IFACE)
+ assert state == 'running'
+
+ self.main_loop()
+
if __name__ == '__main__':
libvirttest.run()
--
2.15.0
10:41 a.m.
New subject: [libvirt] [dbus PATCH v2 3/5] utils: Introduce functions and macros for translating ENUMs to strings
The functions were copied from src/util/virutil.* files from
libvirt project
Signed-off-by: Katerina Koukiou <kkoukiou(a)redhat.com>
---
src/util.c | 27 +++++++++++++++++++++++++++
src/util.h | 28 ++++++++++++++++++++++++++++
2 files changed, 55 insertions(+)
diff --git a/src/util.c b/src/util.c
index d6c27f3..3179dd7 100644
--- a/src/util.c
+++ b/src/util.c
@@ -124,3 +124,30 @@ virtDBusUtilVirDomainListFree(virDomainPtr *domains)
g_free(domains);
}
+
+const gchar *
+virtDBusUtilEnumToString(const gchar *const*types,
+ guint ntypes,
+ gint type)
+{
+ if (type < 0 || (unsigned)type >= ntypes)
+ return NULL;
+
+ return types[type];
+}
+
+gint
+virtDBusUtilEnumFromString(const gchar *const*types,
+ guint ntypes,
+ const gchar *type)
+{
+ guint i;
+ if (!type)
+ return -1;
+
+ for (i = 0; i < ntypes; i++)
+ if (strcmp(types[i], type) == 0)
+ return i;
+
+ return -1;
+}
diff --git a/src/util.h b/src/util.h
index 4304bac..61e01c9 100644
--- a/src/util.h
+++ b/src/util.h
@@ -37,3 +37,31 @@ virtDBusUtilVirDomainListFree(virDomainPtr *domains);
G_DEFINE_AUTOPTR_CLEANUP_FUNC(virDomain, virDomainFree);
G_DEFINE_AUTOPTR_CLEANUP_FUNC(virDomainPtr, virtDBusUtilVirDomainListFree);
+
+gint
+virtDBusUtilEnumFromString(const gchar *const*types,
+ guint ntypes,
+ const gchar *type) G_GNUC_PURE;
+
+const gchar *
+virtDBusUtilEnumToString(const gchar *const*types,
+ guint ntypes,
+ gint type) G_GNUC_PURE;
+
+# define VIRT_DBUS_ENUM_IMPL(name, lastVal, ...) \
+ static const gchar *const name ##TypeList[] = { __VA_ARGS__ }; \
+ G_STATIC_ASSERT(G_N_ELEMENTS(name ##TypeList) == lastVal); \
+ const gchar *name ##TypeToString(int type) { \
+ return virtDBusUtilEnumToString(name ##TypeList, \
+ G_N_ELEMENTS(name ##TypeList), \
+ type); \
+ } \
+ gint name ##TypeFromString(const gchar *type) { \
+ return virtDBusUtilEnumFromString(name ##TypeList, \
+ G_N_ELEMENTS(name ##TypeList), \
+ type); \
+ }
+
+# define VIRT_DBUS_ENUM_DECL(name) \
+ const gchar *name ##TypeToString(gint type) G_GNUC_PURE; \
+ gint name ##TypeFromString(const gchar*type) G_GNUC_PURE;
--
2.15.0
7 a.m.
New subject: [libvirt] [dbus PATCH v2 3/5] utils: Introduce functions and macros for translating ENUMs to strings
On Thu, Mar 29, 2018 at 05:41:19PM +0200, Katerina Koukiou wrote:
The functions were copied from src/util/virutil.* files from
libvirt project
Signed-off-by: Katerina Koukiou <kkoukiou(a)redhat.com>
---
src/util.c | 27 +++++++++++++++++++++++++++
src/util.h | 28 ++++++++++++++++++++++++++++
2 files changed, 55 insertions(+)
diff --git a/src/util.c b/src/util.c
index d6c27f3..3179dd7 100644
--- a/src/util.c
+++ b/src/util.c
@@ -124,3 +124,30 @@ virtDBusUtilVirDomainListFree(virDomainPtr *domains)
g_free(domains);
}
+
+const gchar *
+virtDBusUtilEnumToString(const gchar *const*types,
s/*const*types/*const *types/
+ guint ntypes,
+ gint type)
+{
+ if (type < 0 || (unsigned)type >= ntypes)
s/unsigned/guint/
+ return NULL;
+
+ return types[type];
+}
+
+gint
+virtDBusUtilEnumFromString(const gchar *const*types,
s/*const*types/*const *types/
+ guint ntypes,
+ const gchar *type)
+{
+ guint i;
+ if (!type)
+ return -1;
+
+ for (i = 0; i < ntypes; i++)
+ if (strcmp(types[i], type) == 0)
You can use g_str_equal here.
+ return i;
+
+ return -1;
+}
diff --git a/src/util.h b/src/util.h
index 4304bac..61e01c9 100644
--- a/src/util.h
+++ b/src/util.h
@@ -37,3 +37,31 @@ virtDBusUtilVirDomainListFree(virDomainPtr *domains);
G_DEFINE_AUTOPTR_CLEANUP_FUNC(virDomain, virDomainFree);
G_DEFINE_AUTOPTR_CLEANUP_FUNC(virDomainPtr, virtDBusUtilVirDomainListFree);
+
+gint
+virtDBusUtilEnumFromString(const gchar *const*types,
s/*const*types/*const *types/
+ guint ntypes,
+ const gchar *type) G_GNUC_PURE;
+
+const gchar *
+virtDBusUtilEnumToString(const gchar *const*types,
s/*const*types/*const *types/
+ guint ntypes,
+ gint type) G_GNUC_PURE;
+
+# define VIRT_DBUS_ENUM_IMPL(name, lastVal, ...) \
Remove the extra space after #.
+ static const gchar *const name ##TypeList[] = { __VA_ARGS__ };
\
+ G_STATIC_ASSERT(G_N_ELEMENTS(name ##TypeList) == lastVal); \
+ const gchar *name ##TypeToString(int type) { \
s/int/gint/
+ return virtDBusUtilEnumToString(name ##TypeList, \
+ G_N_ELEMENTS(name ##TypeList), \
+ type); \
+ } \
+ gint name ##TypeFromString(const gchar *type) { \
+ return virtDBusUtilEnumFromString(name ##TypeList, \
+ G_N_ELEMENTS(name ##TypeList), \
+ type); \
+ }
+
+# define VIRT_DBUS_ENUM_DECL(name) \
Remove the extra space after #.
+ const gchar *name ##TypeToString(gint type) G_GNUC_PURE; \
+ gint name ##TypeFromString(const gchar*type) G_GNUC_PURE;
s/gchar*type/gchar *type/
If you fix these small issues mostly introduced by copying the code
Reviewed-by: Pavel Hrdina <phrdina(a)redhat.com>
10:41 a.m.
New subject: [libvirt] [dbus PATCH v2 4/5] events: Introduce virtDBusEventsDomainEventToString function
This function replaces the manual translation of Event ENUMS.
Note: VIR_ENUM_SENTINELS is needed for VIR_DOMAIN_EVENT_LAST.
Signed-off-by: Katerina Koukiou <kkoukiou(a)redhat.com>
---
src/events.c | 55 +++++++++++++++++++++----------------------------------
src/events.h | 3 +++
src/util.h | 1 +
3 files changed, 25 insertions(+), 34 deletions(-)
diff --git a/src/events.c b/src/events.c
index dada55f..0d6d5fc 100644
--- a/src/events.c
+++ b/src/events.c
@@ -4,6 +4,26 @@
#include <libvirt/libvirt.h>
+VIRT_DBUS_ENUM_DECL(virtDBusEventsDomainEvent)
+VIRT_DBUS_ENUM_IMPL(virtDBusEventsDomainEvent,
+ VIR_DOMAIN_EVENT_LAST,
+ "Defined",
+ "Undefined",
+ "Started",
+ "Suspended",
+ "Resumed",
+ "Stopped",
+ "Shutdown",
+ "PMSuspended",
+ "Crashed")
+
+const gchar *
+virtDBusEventsDomainEventToString(gint event)
+{
+ const gchar *str = virtDBusEventsDomainEventTypeToString(event);
+ return str ? str : "unknown";
+}
+
static gint
virtDBusEventsDomainLifecycle(virConnectPtr connection G_GNUC_UNUSED,
virDomainPtr domain,
@@ -12,42 +32,9 @@ virtDBusEventsDomainLifecycle(virConnectPtr connection G_GNUC_UNUSED,
gpointer opaque)
{
virtDBusConnect *connect = opaque;
- const gchar *signal = NULL;
const gchar *name;
g_autofree gchar *path = NULL;
- switch (event) {
- case VIR_DOMAIN_EVENT_DEFINED:
- signal = "DomainDefined";
- break;
- case VIR_DOMAIN_EVENT_UNDEFINED:
- signal = "DomainUndefined";
- break;
- case VIR_DOMAIN_EVENT_STARTED:
- signal = "DomainStarted";
- break;
- case VIR_DOMAIN_EVENT_SUSPENDED:
- signal = "DomainSuspended";
- break;
- case VIR_DOMAIN_EVENT_RESUMED:
- signal = "DomainResumed";
- break;
- case VIR_DOMAIN_EVENT_STOPPED:
- signal = "DomainStopped";
- break;
- case VIR_DOMAIN_EVENT_SHUTDOWN:
- signal = "DomainShutdown";
- break;
- case VIR_DOMAIN_EVENT_PMSUSPENDED:
- signal = "DomainPMSuspended";
- break;
- case VIR_DOMAIN_EVENT_CRASHED:
- signal = "DomainCrashed";
- break;
- default:
- return 0;
- }
-
name = virDomainGetName(domain);
path = virtDBusUtilBusPathForVirDomain(domain, connect->domainPath);
@@ -55,7 +42,7 @@ virtDBusEventsDomainLifecycle(virConnectPtr connection G_GNUC_UNUSED,
NULL,
connect->connectPath,
VIRT_DBUS_CONNECT_INTERFACE,
- signal,
+ virtDBusEventsDomainEventToString(event),
g_variant_new("(so)", name, path),
NULL);
diff --git a/src/events.h b/src/events.h
index c234c55..7eb5daa 100644
--- a/src/events.h
+++ b/src/events.h
@@ -4,3 +4,6 @@
void
virtDBusEventsRegister(virtDBusConnect *connect);
+
+const gchar *
+virtDBusEventsDomainEventToString(gint event) G_GNUC_PURE;
diff --git a/src/util.h b/src/util.h
index 61e01c9..203cd62 100644
--- a/src/util.h
+++ b/src/util.h
@@ -2,6 +2,7 @@
#include "gdbus.h"
+#define VIR_ENUM_SENTINELS
#include <libvirt/libvirt.h>
#define VIRT_DBUS_ERROR virtDBusErrorQuark()
--
2.15.0
9:26 a.m.
New subject: [libvirt] [dbus PATCH v2 4/5] events: Introduce virtDBusEventsDomainEventToString function
On Thu, Mar 29, 2018 at 05:41:20PM +0200, Katerina Koukiou wrote:
This function replaces the manual translation of Event ENUMS.
Note: VIR_ENUM_SENTINELS is needed for VIR_DOMAIN_EVENT_LAST.
Signed-off-by: Katerina Koukiou <kkoukiou(a)redhat.com>
---
src/events.c | 55 +++++++++++++++++++++----------------------------------
src/events.h | 3 +++
src/util.h | 1 +
3 files changed, 25 insertions(+), 34 deletions(-)
diff --git a/src/events.c b/src/events.c
index dada55f..0d6d5fc 100644
--- a/src/events.c
+++ b/src/events.c
@@ -4,6 +4,26 @@
#include <libvirt/libvirt.h>
+VIRT_DBUS_ENUM_DECL(virtDBusEventsDomainEvent)
+VIRT_DBUS_ENUM_IMPL(virtDBusEventsDomainEvent,
+ VIR_DOMAIN_EVENT_LAST,
+ "Defined",
+ "Undefined",
+ "Started",
+ "Suspended",
+ "Resumed",
+ "Stopped",
+ "Shutdown",
+ "PMSuspended",
+ "Crashed")
For this patch it should be with the "Domain" prefix, the next patch
should rename it.
+
+const gchar *
+virtDBusEventsDomainEventToString(gint event)
+{
+ const gchar *str = virtDBusEventsDomainEventTypeToString(event);
+ return str ? str : "unknown";
+}
+
static gint
virtDBusEventsDomainLifecycle(virConnectPtr connection G_GNUC_UNUSED,
virDomainPtr domain,
@@ -12,42 +32,9 @@ virtDBusEventsDomainLifecycle(virConnectPtr connection G_GNUC_UNUSED,
gpointer opaque)
{
virtDBusConnect *connect = opaque;
- const gchar *signal = NULL;
const gchar *name;
g_autofree gchar *path = NULL;
- switch (event) {
- case VIR_DOMAIN_EVENT_DEFINED:
- signal = "DomainDefined";
- break;
- case VIR_DOMAIN_EVENT_UNDEFINED:
- signal = "DomainUndefined";
- break;
- case VIR_DOMAIN_EVENT_STARTED:
- signal = "DomainStarted";
- break;
- case VIR_DOMAIN_EVENT_SUSPENDED:
- signal = "DomainSuspended";
- break;
- case VIR_DOMAIN_EVENT_RESUMED:
- signal = "DomainResumed";
- break;
- case VIR_DOMAIN_EVENT_STOPPED:
- signal = "DomainStopped";
- break;
- case VIR_DOMAIN_EVENT_SHUTDOWN:
- signal = "DomainShutdown";
- break;
- case VIR_DOMAIN_EVENT_PMSUSPENDED:
- signal = "DomainPMSuspended";
- break;
- case VIR_DOMAIN_EVENT_CRASHED:
- signal = "DomainCrashed";
- break;
- default:
- return 0;
- }
-
name = virDomainGetName(domain);
path = virtDBusUtilBusPathForVirDomain(domain, connect->domainPath);
@@ -55,7 +42,7 @@ virtDBusEventsDomainLifecycle(virConnectPtr connection G_GNUC_UNUSED,
NULL,
connect->connectPath,
VIRT_DBUS_CONNECT_INTERFACE,
- signal,
+ virtDBusEventsDomainEventToString(event),
g_variant_new("(so)", name, path),
NULL);
diff --git a/src/events.h b/src/events.h
index c234c55..7eb5daa 100644
--- a/src/events.h
+++ b/src/events.h
@@ -4,3 +4,6 @@
void
virtDBusEventsRegister(virtDBusConnect *connect);
+
+const gchar *
+virtDBusEventsDomainEventToString(gint event) G_GNUC_PURE;
There is no need to export this function in header file, it's used only
in events.c as a static function.
diff --git a/src/util.h b/src/util.h
index 61e01c9..203cd62 100644
--- a/src/util.h
+++ b/src/util.h
@@ -2,6 +2,7 @@
#include "gdbus.h"
+#define VIR_ENUM_SENTINELS
Unrelated change and not needed, you can remove it.
Pavel
10:41 a.m.
New subject: [libvirt] [dbus PATCH v2 5/5] Merge all Domain lifecycle signals into one signal called Domain.
Instead of having multiple signals regarding to domain events,
like DomainStarted, DomainUndefined etc, we will have only one
called DomainEvents, and the specific event type will be specified in
the signals arguments.
The domain name argument in not needed in the signal since we can
fetch it from path.
The tests are adjusted to call correctly connect_to_signal method
on the new signal.
Signed-off-by: Katerina Koukiou <kkoukiou(a)redhat.com>
---
data/org.libvirt.Connect.xml | 54 +++-----------------------------------------
src/events.c | 6 ++---
test/test_connect.py | 10 ++++----
test/test_domain.py | 20 +++++++---------
4 files changed, 17 insertions(+), 73 deletions(-)
diff --git a/data/org.libvirt.Connect.xml b/data/org.libvirt.Connect.xml
index f24dff4..1695100 100644
--- a/data/org.libvirt.Connect.xml
+++ b/data/org.libvirt.Connect.xml
@@ -44,59 +44,11 @@
<arg name="uuid" type="s" direction="in"/>
<arg name="domain" type="o" direction="out"/>
</method>
- <signal name="DomainCrashed">
+ <signal name="DomainEvent">
<annotation name="org.gtk.GDBus.DocString"
- value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_EVENT_CRA...
- <arg name="reason" type="s"/>
- <arg name="domain" type="o"/>
- </signal>
- <signal name="DomainDefined">
- <annotation name="org.gtk.GDBus.DocString"
- value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_EVENT_DEF...
- <arg name="reason" type="s"/>
- <arg name="domain" type="o"/>
- </signal>
- <signal name="DomainPMSuspended">
- <annotation name="org.gtk.GDBus.DocString"
- value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_EVENT_SUS...
- <arg name="reason" type="s"/>
- <arg name="domain" type="o"/>
- </signal>
- <signal name="DomainResumed">
- <annotation name="org.gtk.GDBus.DocString"
- value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_EVENT_RES...
- <arg name="reason" type="s"/>
- <arg name="domain" type="o"/>
- </signal>
- <signal name="DomainShutdown">
- <annotation name="org.gtk.GDBus.DocString"
- value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_EVENT_SHU...
- <arg name="reason" type="s"/>
- <arg name="domain" type="o"/>
- </signal>
- <signal name="DomainStarted">
- <annotation name="org.gtk.GDBus.DocString"
- value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_EVENT_STA...
- <arg name="reason" type="s"/>
- <arg name="domain" type="o"/>
- </signal>
- <signal name="DomainStopped">
- <annotation name="org.gtk.GDBus.DocString"
- value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_EVENT_STO...
- <arg name="reason" type="s"/>
- <arg name="domain" type="o"/>
- </signal>
- <signal name="DomainSuspended">
- <annotation name="org.gtk.GDBus.DocString"
- value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_EVENT_SUS...
- <arg name="reason" type="s"/>
- <arg name="domain" type="o"/>
- </signal>
- <signal name="DomainUndefined">
- <annotation name="org.gtk.GDBus.DocString"
- value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_EVENT_UND...
- <arg name="reason" type="s"/>
+ value="See
https://libvirt.org/html/libvirt-libvirt-domain.html#virConnectDomainEven...
<arg name="domain" type="o"/>
+ <arg name="event" type="s"/>
</signal>
</interface>
</node>
diff --git a/src/events.c b/src/events.c
index 0d6d5fc..1aad5a3 100644
--- a/src/events.c
+++ b/src/events.c
@@ -32,18 +32,16 @@ virtDBusEventsDomainLifecycle(virConnectPtr connection G_GNUC_UNUSED,
gpointer opaque)
{
virtDBusConnect *connect = opaque;
- const gchar *name;
g_autofree gchar *path = NULL;
- name = virDomainGetName(domain);
path = virtDBusUtilBusPathForVirDomain(domain, connect->domainPath);
g_dbus_connection_emit_signal(connect->bus,
NULL,
connect->connectPath,
VIRT_DBUS_CONNECT_INTERFACE,
- virtDBusEventsDomainEventToString(event),
- g_variant_new("(so)", name, path),
+ "DomainEvent",
+ g_variant_new("(os)", path,
virtDBusEventsDomainEventToString(event)),
NULL);
return 0;
diff --git a/test/test_connect.py b/test/test_connect.py
index 48e25cb..e8b9f92 100755
--- a/test/test_connect.py
+++ b/test/test_connect.py
@@ -29,12 +29,11 @@ class TestConnect(libvirttest.BaseTestClass):
domain.Introspect(dbus_interface=dbus.INTROSPECTABLE_IFACE)
def test_create(self):
- def domain_started(name, path):
- assert name == 'foo'
+ def domain_started(path, _event):
assert isinstance(path, dbus.ObjectPath)
self.loop.quit()
- self.connect.connect_to_signal('DomainStarted', domain_started)
+ self.connect.connect_to_signal('DomainEvent', domain_started,
arg2='Started')
path = self.connect.CreateXML(self.minimal_xml, 0)
assert isinstance(path, dbus.ObjectPath)
@@ -42,12 +41,11 @@ class TestConnect(libvirttest.BaseTestClass):
self.main_loop()
def test_define(self):
- def domain_defined(name, path):
- assert name == 'foo'
+ def domain_defined(path, _event):
assert isinstance(path, dbus.ObjectPath)
self.loop.quit()
- self.connect.connect_to_signal('DomainDefined', domain_defined)
+ self.connect.connect_to_signal('DomainEvent', domain_defined,
arg2='Defined')
path = self.connect.DefineXML(self.minimal_xml)
assert isinstance(path, dbus.ObjectPath)
diff --git a/test/test_domain.py b/test/test_domain.py
index 1433ff5..fa30e7c 100755
--- a/test/test_domain.py
+++ b/test/test_domain.py
@@ -38,12 +38,11 @@ class TestDomain(libvirttest.BaseTestClass):
domain.Undefine(0)
def test_shutdown(self):
- def domain_stopped(name, path):
- assert name == 'test'
+ def domain_stopped(path, _event):
assert isinstance(path, dbus.ObjectPath)
self.loop.quit()
- self.connect.connect_to_signal('DomainStopped', domain_stopped)
+ self.connect.connect_to_signal('DomainEvent', domain_stopped,
arg2='Stopped')
obj, domain = self.domain()
domain.Shutdown(0)
@@ -54,12 +53,11 @@ class TestDomain(libvirttest.BaseTestClass):
self.main_loop()
def test_undefine(self):
- def domain_undefined(name, path):
- assert name == 'test'
+ def domain_undefined(path, _event):
assert isinstance(path, dbus.ObjectPath)
self.loop.quit()
- self.connect.connect_to_signal('DomainUndefined', domain_undefined)
+ self.connect.connect_to_signal('DomainEvent', domain_undefined,
arg2='Undefined')
_, domain = self.domain()
domain.Shutdown(0)
@@ -68,12 +66,11 @@ class TestDomain(libvirttest.BaseTestClass):
self.main_loop()
def test_suspend(self):
- def domain_suspended(name, path):
- assert name == 'test'
+ def domain_suspended(path, _event):
assert isinstance(path, dbus.ObjectPath)
self.loop.quit()
- self.connect.connect_to_signal('DomainSuspended', domain_suspended)
+ self.connect.connect_to_signal('DomainEvent', domain_suspended,
arg2='Suspended')
obj, domain = self.domain()
domain.Suspend()
@@ -84,12 +81,11 @@ class TestDomain(libvirttest.BaseTestClass):
self.main_loop()
def test_resume(self):
- def domain_resumed(name, path):
- assert name == 'test'
+ def domain_resumed(path, _event):
assert isinstance(path, dbus.ObjectPath)
self.loop.quit()
- self.connect.connect_to_signal('DomainResumed', domain_resumed)
+ self.connect.connect_to_signal('DomainEvent', domain_resumed,
arg2='Resumed')
obj, domain = self.domain()
domain.Suspend()
--
2.15.0
2458
days inactive
2460
days old
7 comments
2 participants
participants (2)
-
Katerina Koukiou -
Pavel Hrdina