Comment 3 for bug 1090310

Revision history for this message
jiang, yunhong (yunhong-jiang) wrote :

I have a concern to changes required on nova to support kernel_id and ramdisk_id. The concern is, it will need to add 'kernel_id' and 'ramdisk_id' to the server details request. This is in fact a API changes, so not sure how strict nova is for the API changes.

The patch to add the ramdisk_id and kernel_id is simple as followed one, but a lot of test cases need be updated since these test case will check the key in the return value also.

So just want do discuss how important/priority to add back the kernel_id/ramdisk_id back to the meter.

--- a/nova/api/openstack/compute/views/servers.py
+++ b/nova/api/openstack/compute/views/servers.py
@@ -98,6 +98,8 @@ class ViewBuilder(common.ViewBuilder):
                 "metadata": self._get_metadata(instance),
                 "hostId": self._get_host_id(instance) or "",
                 "image": self._get_image(request, instance),
+ "kernel_id": instance['kernel_id'],
+ "ramdisk_id": instance['ramdisk_id'],
                 "flavor": self._get_flavor(request, instance),
                 "created": timeutils.isotime(instance["created_at"]),
                 "updated": timeutils.isotime(instance["updated_at"]),