[queues] Return shard listing as object, not array

Bug #1243898 reported by Allele Dev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
Medium
Allele Dev

Bug Description

Return the array encapsulated in an object (e.g., {}). This will allow for extensions and adding fields that won't break backwards-compatibility.

Currently, it is returned as an array.

Allele Dev (alleledev)
Changed in marconi:
status: New → Triaged
importance: Undecided → Medium
Allele Dev (alleledev)
Changed in marconi:
assignee: nobody → Alejandro Cabrera (cpp-cabrera)
Changed in marconi:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to marconi (master)

Reviewed: https://review.openstack.org/50998
Committed: http://github.com/openstack/marconi/commit/f6c321dd8b3f4f921c3cbf82fc385bb6e71cf9c0
Submitter: Jenkins
Branch: master

commit f6c321dd8b3f4f921c3cbf82fc385bb6e71cf9c0
Author: Alejandro Cabrera <email address hidden>
Date: Thu Oct 10 09:12:53 2013 -0400

    feat: integrate shard storage with transport

    This patch refines the interface and storage implementation defined in
    the last patch and integrates it with the transport layer.

    A few updates have been made:
    - 'name' -> 'href' for listing shards
    - limiting, markers, and detailed are all used
    - use of common_utils.fields to clean up shards transport PATCH
    - add missing init for schemas
    - fix schema issues found: 'location' -> 'uri', __init__.py
    - shard resource correctly implements PUT semantics (replaces)

    Transport: the admin API concept has been expanded to include
    functionality from the public interface *in addition* to admin
    functionality. Part of the rationale behind this is to simplify unit
    testing. The other part of this is that an admin should be able to do
    everything a normal user can do in addition to their special
    functions.

    Storage: now divided into control and data plane. The bootstrap passes
    a control driver down to the transport which *can* be used for
    endpoints as needed.

    A test suite has been added that exercises the functionality from the
    transport side of the shard registry resource.

    Finally, the way the FaultyStorage driver tests were handled was
    changed. Something about the setattr magic in that suite's setup made
    it such that *all* tests would use the Faulty storage driver. This is
    possibly related to the use of lazy_property decorators. To address
    this issue, this patch promotes the faulty storage driver to setup.cfg
    visibility and removes the setattrs.

    Change-Id: I5b8cdb3a11d29422762b52f1e15e33167eecb867
    Partitally-implements: blueprint storage-sharding
    Partially-Closes: 1241686
    Closes-Bug: 1243898

Changed in marconi:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in marconi:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in marconi:
milestone: icehouse-1 → 2014.1
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.