From 751c8b4ff0e94b4f665af5541b9249637623d193 Mon Sep 17 00:00:00 2001
From: Ryu Ishimoto <ryu@midokura.jp>
Date: Wed, 17 Aug 2011 19:58:26 +0900
Subject: Added XML support and changed JSON output keys

---
 nova/api/openstack/contrib/virtual_interfaces.py | 28 +++++++++++++++---------
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/nova/api/openstack/contrib/virtual_interfaces.py b/nova/api/openstack/contrib/virtual_interfaces.py
index 86d1128fd..715a54d52 100644
--- a/nova/api/openstack/contrib/virtual_interfaces.py
+++ b/nova/api/openstack/contrib/virtual_interfaces.py
@@ -24,16 +24,17 @@ from nova import log as logging
 from nova.api.openstack import common
 from nova.api.openstack import extensions
 from nova.api.openstack import faults
+from nova.api.openstack import wsgi
 
 
 LOG = logging.getLogger("nova.api.virtual_interfaces")
 
 
 def _translate_vif_summary_view(_context, vif):
-    """Maps keys for attachment summary view."""
+    """Maps keys for VIF summary view."""
     d = {}
     d['id'] = vif['uuid']
-    d['macAddress'] = vif['address']
+    d['mac_address'] = vif['address']
     return d
 
 
@@ -41,12 +42,6 @@ class ServerVirtualInterfaceController(object):
     """The instance VIF API controller for the Openstack API.
     """
 
-    _serialization_metadata = {
-        'application/xml': {
-            'attributes': {
-                'serverVirtualInterface': ['id',
-                                           'macAddress']}}}
-
     def __init__(self):
         self.compute_api = compute.API()
         super(ServerVirtualInterfaceController, self).__init__()
@@ -63,7 +58,7 @@ class ServerVirtualInterfaceController(object):
         vifs = instance['virtual_interfaces']
         limited_list = common.limited(vifs, req)
         res = [entity_maker(context, vif) for vif in limited_list]
-        return {'serverVirtualInterfaces': res}
+        return {'virtual_interfaces': res}
 
     def index(self, req, server_id):
         """Returns the list of VIFs for a given instance."""
@@ -91,11 +86,24 @@ class Virtual_interfaces(extensions.ExtensionDescriptor):
     def get_resources(self):
         resources = []
 
+        metadata = _get_metadata()
+        body_serializers = {
+            'application/xml': wsgi.XMLDictSerializer(metadata=metadata,
+                                                      xmlns=wsgi.XMLNS_V11)}
+        serializer = wsgi.ResponseSerializer(body_serializers, None)
         res = extensions.ResourceExtension('os-virtual-interfaces',
                                            ServerVirtualInterfaceController(),
                                            parent=dict(
                                                member_name='server',
-                                               collection_name='servers'))
+                                               collection_name='servers'),
+                                           serializer=serializer)
         resources.append(res)
 
         return resources
+
+
+def _get_metadata():
+    metadata = {
+        "attributes": {
+                'virtual_interface': ["id", "mac_address"]}}
+    return metadata
-- 
cgit