This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


[Xen-devel] [PATCH] [xend] Fix Xen API Host.get_record method

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] [PATCH] [xend] Fix Xen API Host.get_record method
From: Jim Fehlig <jfehlig@xxxxxxxxxx>
Date: Thu, 03 May 2007 17:41:48 -0600
Delivery-date: Thu, 03 May 2007 16:40:30 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Thunderbird (X11/20060911)
Host.get_record returns supported_bootloaders as a string instead of
'string set' per the spec.  The c-bindings, honoring the spec, fail to
parse response.  Trivial fix attached.

# HG changeset patch
# User Jim Fehlig <jfehlig@xxxxxxxxxx>
# Date 1178235218 21600
# Node ID 6efe4b58684fa506763e0104d4f603ec45a16b4d
# Parent  3a5722420de74aafbe3e11e0faad501c7babe3e4
Fix Xen API's Host.get_record method.

Currently it returns supported_bootloaders as a string.  This patch returns
supported_bootloaders as a 'string set' - per the spec.  The c-bindings fail
to parse response, expecting a set but only finding a string.

Signed-off-by: Jim Fehlig <jfehlig@xxxxxxxxxx>

diff -r 3a5722420de7 -r 6efe4b58684f tools/python/xen/xend/XendAPI.py
--- a/tools/python/xen/xend/XendAPI.py  Thu May 03 19:25:47 2007 +0100
+++ b/tools/python/xen/xend/XendAPI.py  Thu May 03 17:33:38 2007 -0600
@@ -1009,7 +1009,7 @@ class XendAPI(object):
                   'cpu_configuration': node.get_cpu_configuration(),
                   'metrics': node.host_metrics_uuid,
                   'capabilities': node.get_capabilities(),
-                  'supported_bootloaders': 'pygrub',
+                  'supported_bootloaders': ['pygrub'],
                   'sched_policy': node.get_vcpus_policy()}
         return xen_api_success(record)
Xen-devel mailing list
<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-devel] [PATCH] [xend] Fix Xen API Host.get_record method, Jim Fehlig <=