summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlex Meade <alex.meade@rackspace.com>2011-06-30 11:18:19 -0400
committerAlex Meade <alex.meade@rackspace.com>2011-06-30 11:18:19 -0400
commit5f772ea10c22549a7149f608cfc2ff932878d6fe (patch)
treebd442b4bd0ca230228a0296daa62046b96232bb8
parent8133b9af105f7924f03b710b30cf4f0acb52f143 (diff)
downloadnova-5f772ea10c22549a7149f608cfc2ff932878d6fe.tar.gz
nova-5f772ea10c22549a7149f608cfc2ff932878d6fe.tar.xz
nova-5f772ea10c22549a7149f608cfc2ff932878d6fe.zip
updated servers
-rw-r--r--nova/api/openstack/views/servers.py17
-rw-r--r--nova/tests/api/openstack/test_servers.py16
2 files changed, 12 insertions, 21 deletions
diff --git a/nova/api/openstack/views/servers.py b/nova/api/openstack/views/servers.py
index cbfa5aae7..1c6dbf87d 100644
--- a/nova/api/openstack/views/servers.py
+++ b/nova/api/openstack/views/servers.py
@@ -156,6 +156,7 @@ class ViewBuilderV11(ViewBuilder):
def _build_links(self, response, inst):
href = self.generate_href(inst["id"])
+ bookmark = self.generate_bookmark(inst["id"])
links = [
{
@@ -164,13 +165,7 @@ class ViewBuilderV11(ViewBuilder):
},
{
"rel": "bookmark",
- "type": "application/json",
- "href": href,
- },
- {
- "rel": "bookmark",
- "type": "application/xml",
- "href": href,
+ "href": bookmark,
},
]
@@ -179,3 +174,11 @@ class ViewBuilderV11(ViewBuilder):
def generate_href(self, server_id):
"""Create an url that refers to a specific server id."""
return os.path.join(self.base_url, "servers", str(server_id))
+
+ def generate_bookmark(self, server_id):
+ """Create an url that refers to a specific flavor id."""
+ return os.path.join(self._remove_version(self.base_url),
+ "servers", str(server_id))
+
+ def _remove_version(self, base_url):
+ return base_url.rsplit('/', 1).pop(0)
diff --git a/nova/tests/api/openstack/test_servers.py b/nova/tests/api/openstack/test_servers.py
index b53c6c9be..110445935 100644
--- a/nova/tests/api/openstack/test_servers.py
+++ b/nova/tests/api/openstack/test_servers.py
@@ -290,13 +290,7 @@ class ServersTest(test.TestCase):
},
{
"rel": "bookmark",
- "type": "application/json",
- "href": "http://localhost/v1.1/servers/1",
- },
- {
- "rel": "bookmark",
- "type": "application/xml",
- "href": "http://localhost/v1.1/servers/1",
+ "href": "http://localhost/servers/1",
},
]
@@ -514,13 +508,7 @@ class ServersTest(test.TestCase):
},
{
"rel": "bookmark",
- "type": "application/json",
- "href": "http://localhost/v1.1/servers/%d" % (i,),
- },
- {
- "rel": "bookmark",
- "type": "application/xml",
- "href": "http://localhost/v1.1/servers/%d" % (i,),
+ "href": "http://localhost/servers/%d" % (i,),
},
]