Pass boolean instead of string for --all-stores, --allow-failure options
Bug #1871674 reported by
Abhishek Kekane
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Glance Client |
Fix Released
|
Critical
|
Unassigned | ||
Bug Description
Newly added command line options --all-stores, --allow-failure are boolean but from python-glanceclient we are passing it as a string to glance API. This will cause problem if those API parameters are not converted to boolean at API side.
Also as per documentation we expect these parameters as boolean in the API as well.
| Changed in python-glanceclient: | |
| importance: | Undecided → Critical |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to python-glanceclient (master) | #1 |
| Changed in python-glanceclient: | |
| status: | In Progress → Fix Released |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to python-glanceclient (stable/ussuri) | #2 |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to python-glanceclient (stable/ussuri) | #3 |
| tags: | added: in-stable-ussuri |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix included in openstack/python-glanceclient 3.1.1 | #4 |
To post a comment you must log in.
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit dff5c881bdbc7f0
Author: Abhishek Kekane <email address hidden>
Date: Wed Apr 8 17:11:18 2020 +0000
Pass --all-stores, --allow-failure as bool to API
Newly added command line options --all-stores, --allow-failure are boolean
but we are passing it as a string to glance API. This will cause problem if
those parameters are not converted to boolean at API side.
Passing these parameters as boolean instead of string to API.
Change-Id: I8d4eab9241fc9b
Closes-Bug: #1871674