Red Hat Bugzilla – Bug 1302834
Backport validation of filters in snapshot*, backup* in db.api in OSP6
Last modified: 2018-02-08 06:11:45 EST
Description of problem:
Validation of filters in snapshot*, backup* in db.api needs to be done in OSP6
Some more details on my issue:
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'
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:
This patch needs to be backported to OSP6.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
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  to OSP 6.0.
Sergey, I don't know if we agreed on this patch, since I don't know the impact of that on cinder, but I have backported a fix for horizon, on which we agreed. Form looking at the stack trace, it could be the same bug. like .
If I find some time tomorrow, I'll issue a build.
(In reply to Matthias Runge from comment #3)
> Sergey, I don't know if we agreed on this patch, since I don't know the
> impact of that on cinder, but I have backported a fix for horizon, on
> which we agreed. Form looking at the stack trace, it could be the same bug.
> like .
> 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.
>  https://code.engineering.redhat.com/gerrit/#/c/65755/
>  https://bugzilla.redhat.com/show_bug.cgi?id=1288654
Please, reassign to Cinder if Horizon build in comment#6 doesn't fix the problem.
*** This bug has been marked as a duplicate of bug 1288654 ***