Cinder api "No operator matches the given name and argument type" ast filters

Bug #1494475 reported by Sean McCully
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Sean McCully

Bug Description

ast evaluation of integer fails and incorrectly converts integer to string.

'Could not evaluate value %s, assuming string'

Tags: api kilo
Revision history for this message
Sean McCully (sean-mccully) wrote :
Changed in cinder:
assignee: nobody → Sean McCully (sean-mccully)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/222393

Changed in cinder:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/225947

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Sean McCully (<email address hidden>) on branch: master
Review: https://review.openstack.org/225558

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/225947
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=96cacc6225a3fe48d06e0e41d6e74e6af1595f7f
Submitter: Jenkins
Branch: master

commit 96cacc6225a3fe48d06e0e41d6e74e6af1595f7f
Author: Sean McCully <email address hidden>
Date: Thu Sep 10 16:33:37 2015 -0400

    Use of ast for integers doesn't changes type

    ast.literal_eval doesn't change types for Integer
    Booleans resulting in DBAPIError exceptions with
    at least postgresql. Boolean columns need to be
    explicitly cast to boolean types. This changes
    the way filters are processed in the volume API.

    APIImpact
    Change-Id: Ice9c57fa99c17e4c87de6362bf30ed30503a1bed
    Closes-Bug: #1494475

Changed in cinder:
status: In Progress → Fix Committed
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/cinder 8.0.0.0b1

This issue was fixed in the openstack/cinder 8.0.0.0b1 development milestone.

Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/kilo)

Change abandoned by Sean McGinnis (<email address hidden>) on branch: stable/kilo
Review: https://review.openstack.org/222393
Reason: This review is > 4 weeks without comment and currently blocked by a core reviewer with a -2. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and contacting the reviewer with the -2 on this review to ensure you address their concerns.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.