yahoo-eng-team team mailing list archive

[Matt Riedemann <mriedem.os@xxxxxxxxx>]

Public bug reported:

The down-cell microversion 2.69 just recently merged and returns links
in the response for GET /servers/detail but not GET /servers/{server_id}
(show) which was an oversight because that API returns links normally:

https://developer.openstack.org/api-ref/compute/?expanded=show-server-
details-detail#show-server-details

We should include the links in the 'show' case as well and make sure to
update the docs in the compute API guide:

https://developer.openstack.org/api-guide/compute/down_cells.html

https://review.openstack.org/#/c/621474/28/nova/api/openstack/compute/views/servers.py@180

Typically this would require a microversion change but given the code
merged on Feb 15 (https://review.openstack.org/#/c/635146/) and is not
yet released I think we can bend the microversion rules and just fix
this oversight since I doubt anyone has deployed or is using this yet
since it's a pretty advanced use case.

** Affects: nova
     Importance: High
         Status: Triaged


** Tags: api cells stein-rc-potential

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1818131

Title:
  GET /servers/{server_id} response for a down-cell instance does not
  include links

Status in OpenStack Compute (nova):
  Triaged

Bug description:
  The down-cell microversion 2.69 just recently merged and returns links
  in the response for GET /servers/detail but not GET
  /servers/{server_id} (show) which was an oversight because that API
  returns links normally:

  https://developer.openstack.org/api-ref/compute/?expanded=show-server-
  details-detail#show-server-details

  We should include the links in the 'show' case as well and make sure
  to update the docs in the compute API guide:

  https://developer.openstack.org/api-guide/compute/down_cells.html

  https://review.openstack.org/#/c/621474/28/nova/api/openstack/compute/views/servers.py@180

  Typically this would require a microversion change but given the code
  merged on Feb 15 (https://review.openstack.org/#/c/635146/) and is not
  yet released I think we can bend the microversion rules and just fix
  this oversight since I doubt anyone has deployed or is using this yet
  since it's a pretty advanced use case.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1818131/+subscriptions