yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #72549
[Bug 1765204] Re: Listing resource providers in placement with a postgresql backend gets a group by error
Reviewed: https://review.openstack.org/562379
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=979b066be4df4e92b44efe35e0b39f43e2079be6
Submitter: Zuul
Branch: master
commit 979b066be4df4e92b44efe35e0b39f43e2079be6
Author: Chris Dent <cdent@xxxxxxxxxxxxx>
Date: Wed Apr 18 21:25:56 2018 +0100
Add root and parent provider uuid to group by clause
Without the group by, postgres complains with:
column "root_rp.uuid" must appear in the GROUP BY clause or be
used in an aggregate function
which is a common complaint between mysql and postgresql. After
that is fixed, it complains about parent_rp.uuid.
This problem was discovered using my placedock container[1] with
a postgres database. The change that causes this to be necessary
must be relatively recent as this wasn't happening last week.
The problem can show up when the outer join to the root_rp and
parent_rp copies of the resource_providers table can only produce
a single row. The joins to those aliased tables are what is
upsetting postgresql.
[1] https://hub.docker.com/r/cdent/placedock/
Closes-Bug: 1765204
Change-Id: I9fb3db0d7e025fff8836d01f60bb035fb1c59758
** 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/1765204
Title:
Listing resource providers in placement with a postgresql backend gets
a group by error
Status in OpenStack Compute (nova):
Fix Released
Bug description:
When listing resource providers in placement with a postgresql
database, this error recently started showing up:
column "root_rp.uuid" must appear in the GROUP BY clause or be
used in an aggregate function
And then once you fix that it says the same for parent_rp.uuid.
It's not clear when this problem came on the scene, since we don't
test regularly with postgresql, but I do use it in my experiments with
placement in a container [1] and it wasn't there as of April 6th, and
probably a bit later.
Work is in progress to fix this as well as turn on an experimental job
for checking with postgres every now and again.
[1] https://hub.docker.com/r/cdent/placedock/
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1765204/+subscriptions
References