Glance

Merge lp://qastaging/~rackspace-titan/glance/registry-marker-lp819551 into lp://qastaging/~hudson-openstack/glance/trunk

  1. registry-marker-lp819551
  2. Merge into trunk
Proposed by Brian Waldon
Status: Merged
Approved by: Brian Lamar
Approved revision: 174
Merged at revision: 176
Proposed branch: lp://qastaging/~rackspace-titan/glance/registry-marker-lp819551
Merge into: lp://qastaging/~hudson-openstack/glance/trunk
Diff against target: 58 lines (+32/-5)
2 files modified
glance/registry/db/api.py (+12/-5)
tests/functional/test_httplib2_api.py (+20/-0)
To merge this branch: bzr merge lp://qastaging/~rackspace-titan/glance/registry-marker-lp819551
Reviewer Review Type Date Requested Status
Brian Lamar (community) Approve
Jay Pipes (community) Approve
Review via email: mp+70094@code.qastaging.launchpad.net

Description of the change

Update registry db api to properly handle pagination through sorted results.

To post a comment you must log in.
Revision history for this message
Jay Pipes (jaypipes) wrote :

Good stuff, Brian2.

review: Approve
Revision history for this message
Brian Waldon (bcwaldon) wrote :

> Good stuff, Brian2.

Excuse me? You know for a fact I am Brian 1 :)

Revision history for this message
Jay Pipes (jaypipes) wrote :

On Mon, Aug 1, 2011 at 9:13 PM, Brian Waldon <email address hidden> wrote:
>> Good stuff, Brian2.
>
> Excuse me? You know for a fact I am Brian 1 :)

W comes after L.

Nuf said. :P

-jay

Revision history for this message
Brian Lamar (blamar) wrote :

Looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches