[Yahoo-eng-team] [Bug 1777458] Re: Listing instances with a marker that is in the build_requests table and the ip/ip6 filters results in an error

2018-06-18 Thread Matt Riedemann
** Also affects: nova/queens
   Importance: Undecided
   Status: New

** Changed in: nova/queens
   Status: New => Confirmed

** Changed in: nova/queens
   Importance: Undecided => Medium

** Changed in: nova
   Importance: High => Medium

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1777458

Title:
  Listing instances with a marker that is in the build_requests table
  and the ip/ip6 filters results in an error

Status in OpenStack Compute (nova):
  In Progress
Status in OpenStack Compute (nova) queens series:
  Confirmed

Bug description:
  This is very similar to bug 1773945 but has a specific recreate:

  1. create server
  2. while it's a build request (before scheduling), immediately list servers 
with the id from server in #1 as the marker and filter using the 'ip' filter - 
it doesn't matter what the value is

  Due to change https://review.openstack.org/#/c/539469/ this will
  bypass the build_requests table because of the ip filter and then
  attempt to look for the marker in the cell database instances tables,
  and not find it and fail with a 400 error.

  If you do this on top of change
  https://review.openstack.org/#/c/575556/ then it's a 500 error.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1777458/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1777458] Re: Listing instances with a marker that is in the build_requests table and the ip/ip6 filters results in an error

2018-07-02 Thread OpenStack Infra
Reviewed:  https://review.openstack.org/576161
Committed: 
https://git.openstack.org/cgit/openstack/nova/commit/?id=8cea14abf30c40da4ce5ffcfb4fb37dd79083255
Submitter: Zuul
Branch:master

commit 8cea14abf30c40da4ce5ffcfb4fb37dd79083255
Author: Matt Riedemann 
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


** Changed in: nova
   Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1777458

Title:
  Listing instances with a marker that is in the build_requests table
  and the ip/ip6 filters results in an error

Status in OpenStack Compute (nova):
  Fix Released
Status in OpenStack Compute (nova) queens series:
  Confirmed

Bug description:
  This is very similar to bug 1773945 but has a specific recreate:

  1. create server
  2. while it's a build request (before scheduling), immediately list servers 
with the id from server in #1 as the marker and filter using the 'ip' filter - 
it doesn't matter what the value is

  Due to change https://review.openstack.org/#/c/539469/ this will
  bypass the build_requests table because of the ip filter and then
  attempt to look for the marker in the cell database instances tables,
  and not find it and fail with a 400 error.

  If you do this on top of change
  https://review.openstack.org/#/c/575556/ then it's a 500 error.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1777458/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp