Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1302834

Summary: Backport validation of filters in snapshot*, backup* in db.api in OSP6
Product: Red Hat OpenStack Reporter: Jaison Raju <jraju>
Component: python-django-horizonAssignee: Matthias Runge <mrunge>
Status: CLOSED DUPLICATE QA Contact: Ido Ovadia <iovadia>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0 (Juno)CC: aortega, athomas, dcadzow, eharney, jmelvin, jraju, mrunge, pmukhedk, sgotliv, yeylon
Target Milestone: ---Keywords: FeatureBackport, ZStream
Target Release: 6.0 (Juno)   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-03-04 09:22:24 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

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 ***