commit f1cb6f1647025539093dc2cdd52da3ed129a6f5c
Author: Matt Riedemann <email address hidden>
Date: Mon Jun 18 10:27:43 2018 -0400
Fix regression when listing build_requests with marker and ip filter
Change Ic02206e887e3fea7752d615bbed680510c482097 attempted
to optimize the GET /servers flow by skipping filtering on
build requests if the ip or ip6 filters were used, since
servers that are not yet scheduled (build requests) can't have
ips. However, if a marker is provided and the marker is in the
build_requests table, we fail to look there and then check the
cells for the marker, which won't exist and result in a 400
MarkerNotFound error.
This fixes the issue by *not* skipping build requests if there
is a marker specified. A functional test is added which will
show the 400 MarkerNotFound error if the code fix is removed.
Change-Id: Ibdd157d06252c3c0219539ec798c8d9d3a8ae26f
Closes-Bug: #1777458
(cherry picked from commit 8cea14abf30c40da4ce5ffcfb4fb37dd79083255)
Reviewed: https:/ /review. openstack. org/579561 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=f1cb6f16470 25539093dc2cdd5 2da3ed129a6f5c
Committed: https:/
Submitter: Zuul
Branch: stable/queens
commit f1cb6f164702553 9093dc2cdd52da3 ed129a6f5c
Author: Matt Riedemann <email address hidden>
Date: Mon Jun 18 10:27:43 2018 -0400
Fix regression when listing build_requests with marker and ip filter
Change Ic02206e887e3fe a7752d615bbed68 0510c482097 attempted
to optimize the GET /servers flow by skipping filtering on
build requests if the ip or ip6 filters were used, since
servers that are not yet scheduled (build requests) can't have
ips. However, if a marker is provided and the marker is in the
build_requests table, we fail to look there and then check the
cells for the marker, which won't exist and result in a 400
MarkerNotFound error.
This fixes the issue by *not* skipping build requests if there
is a marker specified. A functional test is added which will
show the 400 MarkerNotFound error if the code fix is removed.
Change-Id: Ibdd157d06252c3 c0219539ec798c8 d9d3a8ae26f a4ce5ffcfb4fb37 dd79083255)
Closes-Bug: #1777458
(cherry picked from commit 8cea14abf30c40d