Bug 1302834 - Backport validation of filters in snapshot*, backup* in db.api in OSP6
Summary: Backport validation of filters in snapshot*, backup* in db.api in OSP6
Keywords:
Status: CLOSED DUPLICATE of bug 1288654
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: python-django-horizon
Version: 6.0 (Juno)
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
: 6.0 (Juno)
Assignee: Matthias Runge
QA Contact: Ido Ovadia
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-28 18:22 UTC by Jaison Raju
Modified: 2019-11-14 07:22 UTC (History)
10 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-03-04 09:22:24 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Jaison Raju 2016-01-28 18:22:08 UTC
Description of problem:
Validation of filters in snapshot*, backup* in db.api needs to be done in OSP6
Some more details on my issue:
Cinder
2016-01-28 14:43:50.941 3126 TRACE cinder.api.middleware.fault InvalidRequestError: Entity '<class 'cinder.db.sqlalchemy.models.Snapshot'>' has no property 'tenant_id'

Horizon
Thu Jan 28 03:43:50.945988 2016] [:error] [pid 3356] ClientException: The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-bf612a1a-cf72-4396-be9b-756e491e991b)
This is fixed in liberty 7.0.0:
https://review.openstack.org/#/c/217110/
This patch needs to be backported to OSP6.

Version-Release number of selected component (if applicable):
RHOS6

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 2 Sergey Gotliv 2016-01-28 21:27:21 UTC
Matthias,

Please, read this BZ. I believe we agreed in the past that Horizon should use correct filters, I just don't remember if you've backported the relevant Horizon patch fixing that problem.

I don't like the idea that Cinder ignores wrong filters instead of throwing an error in Liberty, but if you didn't fix that BZ in Horizon yet then I'll definitely proceed with the Cinder backport [1] to OSP 6.0.

[1] https://review.openstack.org/#/c/217110/

Comment 3 Matthias Runge 2016-01-28 21:40:19 UTC
Sergey, I don't know if we agreed on this patch[1], since I don't know the impact of that on cinder, but I have backported a fix[2] for horizon, on which we agreed. Form looking at the stack trace, it could be the same bug. like [3].
If I find some time tomorrow, I'll issue a build.

[2] https://code.engineering.redhat.com/gerrit/#/c/65755/
[3] https://bugzilla.redhat.com/show_bug.cgi?id=1288654

Comment 7 Sergey Gotliv 2016-02-15 06:21:29 UTC
(In reply to Matthias Runge from comment #3)
> Sergey, I don't know if we agreed on this patch[1], since I don't know the
> impact of that on cinder, but I have backported a fix[2] for horizon, on
> which we agreed. Form looking at the stack trace, it could be the same bug.
> like [3].
> If I find some time tomorrow, I'll issue a build.

Matthias, thanks for the quick response and the horizon build! I have to take another look if it fixes the problem described here. The idea is the same, we have to use correct filters instead of validating them, but it seems that patch below fixes volumes' filters only while this issue seems reports issues with snapshots and backups. 

> 
> [2] https://code.engineering.redhat.com/gerrit/#/c/65755/
> [3] https://bugzilla.redhat.com/show_bug.cgi?id=1288654

Comment 10 Sergey Gotliv 2016-02-16 06:53:46 UTC
Please, reassign to Cinder if Horizon build in comment#6 doesn't fix the problem.

Comment 16 Matthias Runge 2016-03-04 09:22:24 UTC

*** This bug has been marked as a duplicate of bug 1288654 ***


Note You need to log in before you can comment on or make changes to this bug.