Comment 15 for bug 1689692

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (master)

Reviewed: https://review.openstack.org/505661
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=affb25ecef86537bfaebc69eb0af3b84e9cad4de
Submitter: Zuul
Branch: master

commit affb25ecef86537bfaebc69eb0af3b84e9cad4de
Author: Dan Smith <email address hidden>
Date: Wed Sep 20 07:20:32 2017 -0700

    Fix a pagination logic bug in test_bug_1689692

    This test attempts to list all instances, then list them again with the
    first instance as the marker and ensure that the remaining instances
    are returned in the page. Now that we are doing queries to cells in
    parallel, consecutive unsorted list queries can return things in
    different orders as the cells may reply at different times. The fix
    in this patch is to ask for results to be sorted, which is the only way
    it makes sense. It worked before purely because we were always scanning
    the cells linearly and in the same order.

    Related-Bug: #1689692
    Change-Id: I3ca2a167c902d565c36a5d5dbba1bf1c214aa20b