* [Qemu-devel] [PATCH v4] qga: Add `guest-get-timezone` command
@ 2017-03-23 0:27 Vinzenz 'evilissimo' Feenstra
2017-03-23 0:27 ` Vinzenz 'evilissimo' Feenstra
0 siblings, 1 reply; 2+ messages in thread
From: Vinzenz 'evilissimo' Feenstra @ 2017-03-23 0:27 UTC (permalink / raw)
To: qemu-devel; +Cc: eblake, mdroth, sameeh, marcandre.lureau, Vinzenz Feenstra
From: Vinzenz Feenstra <vfeenstr@redhat.com>
Changes since v3:
- Made zone optional, and won't fail if the name can't be looked up.
- Removed unncessary check for g_new0 allocation failure
Vinzenz Feenstra (1):
qga: Add `guest-get-timezone` command
qga/commands.c | 34 ++++++++++++++++++++++++++++++++++
qga/qapi-schema.json | 26 ++++++++++++++++++++++++++
2 files changed, 60 insertions(+)
--
2.9.3
^ permalink raw reply [flat|nested] 2+ messages in thread
* [Qemu-devel] [PATCH v4] qga: Add `guest-get-timezone` command
2017-03-23 0:27 [Qemu-devel] [PATCH v4] qga: Add `guest-get-timezone` command Vinzenz 'evilissimo' Feenstra
@ 2017-03-23 0:27 ` Vinzenz 'evilissimo' Feenstra
0 siblings, 0 replies; 2+ messages in thread
From: Vinzenz 'evilissimo' Feenstra @ 2017-03-23 0:27 UTC (permalink / raw)
To: qemu-devel; +Cc: eblake, mdroth, sameeh, marcandre.lureau, Vinzenz Feenstra
From: Vinzenz Feenstra <vfeenstr@redhat.com>
Adds a new command `guest-get-timezone` reporting the currently
configured timezone on the system. The information on what timezone is
currently is configured is useful in case of Windows VMs where the
offset of the hardware clock is required to have the same offset. This
can be used for management systems like `oVirt` to detect the timezone
difference and warn administrators of the misconfiguration.
Signed-off-by: Vinzenz Feenstra <vfeenstr@redhat.com>
---
qga/commands.c | 34 ++++++++++++++++++++++++++++++++++
qga/qapi-schema.json | 26 ++++++++++++++++++++++++++
2 files changed, 60 insertions(+)
diff --git a/qga/commands.c b/qga/commands.c
index 4d92946..8a0ff0a 100644
--- a/qga/commands.c
+++ b/qga/commands.c
@@ -499,3 +499,37 @@ int ga_parse_whence(GuestFileWhence *whence, Error **errp)
error_setg(errp, "invalid whence code %"PRId64, whence->u.value);
return -1;
}
+
+GuestTimezone *qmp_guest_get_timezone(Error **errp)
+{
+ GuestTimezone *info = g_new0(GuestTimezone, 1);
+ GTimeZone *tz = g_time_zone_new_local();
+ if (tz == NULL) {
+ error_setg(errp, QERR_QGA_COMMAND_FAILED,
+ "Couldn't retrieve local timezone");
+ goto error;
+ }
+
+ gint32 interval = g_time_zone_find_interval(tz, G_TIME_TYPE_STANDARD, 0);
+ if (interval != -1) {
+ info->offset = g_time_zone_get_offset(tz, interval);
+ gchar const *name = g_time_zone_get_abbreviation(tz, interval);
+ if (name != NULL) {
+ info->zone = g_strdup(name);
+ } else {
+ info->zone = NULL;
+ }
+ } else {
+ error_setg(errp, QERR_QGA_COMMAND_FAILED,
+ "Failed to lookup timezone interval");
+ goto error;
+ }
+ g_time_zone_unref(tz);
+ return info;
+
+error:
+ g_time_zone_unref(tz);
+ g_free(info);
+ return NULL;
+}
+
diff --git a/qga/qapi-schema.json b/qga/qapi-schema.json
index a02dbf2..20d12a8 100644
--- a/qga/qapi-schema.json
+++ b/qga/qapi-schema.json
@@ -1042,3 +1042,29 @@
'data': { 'path': 'str', '*arg': ['str'], '*env': ['str'],
'*input-data': 'str', '*capture-output': 'bool' },
'returns': 'GuestExec' }
+
+
+##
+# @GuestTimezone:
+#
+# @zone: Timezone name
+# @offset: Offset to UTC in seconds. For western timezones the offset has a
+# negative value and for eastern the offset is positive value
+#
+# Since: 2.10
+##
+{ 'struct': 'GuestTimezone',
+ 'data': { '*zone': 'str', 'offset': 'int' } }
+
+
+##
+# @guest-get-timezone:
+#
+# Retrieves the timezone information from the guest.
+#
+# Returns: A GuestTimezone dictionary.
+#
+# Since: 2.10
+##
+{ 'command': 'guest-get-timezone',
+ 'returns': 'GuestTimezone' }
--
2.9.3
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-03-23 14:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-23 0:27 [Qemu-devel] [PATCH v4] qga: Add `guest-get-timezone` command Vinzenz 'evilissimo' Feenstra
2017-03-23 0:27 ` Vinzenz 'evilissimo' Feenstra
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.