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.
Reviewed: https:/ /review. openstack. org/505661 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=affb25ecef8 6537bfaebc69eb0 af3b84e9cad4de
Committed: https:/
Submitter: Zuul
Branch: master
commit affb25ecef86537 bfaebc69eb0af3b 84e9cad4de
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 65c36a5d5dbba1b f1c214aa20b
Change-Id: I3ca2a167c902d5