Heat GET /stacks call doesn't have a "detail" option

Bug #1191117 reported by Gabriel Hurley
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Undecided
Liang Chen

Bug Description

In other projects (Nova, Cinder, etc.) there is an option on the "list" operations to get a "detail" version of the response which contains all the available information about the resources being returned. For Nova this looks like GET /instances/detail.

This functionality is somewhere between extremely helpful and downright necessary in order to build a useful dashboard interface for Heat stacks, and currently requires either making the "list" view practically useless or being terribly inefficient and looping through each stack to fetch the detailed data for it before returning the list.

Revision history for this message
Pavan (pavan0591) wrote :

I can take this up!

Doesn't heat describe < stack name > give the details of the stack?

Revision history for this message
Steve Baker (steve-stevebaker) wrote :

heat/api/openstack/v1/stacks.py

format_stack is called with some filtering keys in index(), but with no filtering in show()

Liang Chen (cbjchen)
Changed in heat:
assignee: nobody → Liang Chen (cbjchen)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

Fix proposed to branch: master
Review: https://review.openstack.org/35034

Changed in heat:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/35034
Committed: http://github.com/openstack/heat/commit/016133f538fd4136006e996d8a9da9130a29c85c
Submitter: Jenkins
Branch: master

commit 016133f538fd4136006e996d8a9da9130a29c85c
Author: Liang Chen <email address hidden>
Date: Fri Jun 28 22:27:59 2013 +0800

    add GET /{tenant_id}/stacks/detail to Heat API

    Enables the building of a useful dashboard interface for Heat stacks
    without looping through stacks returned from "/{tenant_id}/stacks/"
    to fetch the detailed data.

    Fixes bug #1191117

    Change-Id: I06b1100873de462c69302e43259df13104cad79f

Changed in heat:
status: In Progress → Fix Committed
Revision history for this message
Zane Bitter (zaneb) wrote :

Using this path is a bit problematic... we use /{tenant_id}/stacks/{stack_name} to look up the ID for a stack, so if a user ever creates a stack called "details", then they won't be able to access it by name. (This scheme works for Nova because only UUIDs ever appear in that path segment, so there is no namespace conflict.)

IMO the correct thing to do here is to use a query parameter, e.g.:

  /{tenant_id}/stacks?detail=all

Thierry Carrez (ttx)
Changed in heat:
milestone: none → havana-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: havana-2 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.