The API doesn't return nullable fields when they're null
Bug #1398314 reported by
Flavio Percoco
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Glance |
Fix Released
|
Medium
|
Flavio Percoco | ||
Glance Client |
Won't Fix
|
Medium
|
Flavio Percoco |
Bug Description
The schema filter method removes all fields that are null from the object schema. This is causing incompatibilities from the client side and it also makes glance's responses inconsistent.
Changed in glance: | |
assignee: | nobody → Flavio Percoco (flaper87) |
Changed in glance: | |
status: | New → In Progress |
Changed in glance: | |
importance: | Undecided → Medium |
Changed in python-glanceclient: | |
importance: | Undecided → Medium |
assignee: | nobody → Flavio Percoco (flaper87) |
Changed in glance: | |
milestone: | none → kilo-1 |
Changed in glance: | |
status: | Fix Committed → Fix Released |
Changed in python-glanceclient: | |
milestone: | none → v0.16.0 |
Changed in python-glanceclient: | |
milestone: | v0.16.0 → none |
Changed in glance: | |
milestone: | kilo-1 → 2015.1.0 |
To post a comment you must log in.
Reviewed: https:/ /review. openstack. org/138184 /git.openstack. org/cgit/ openstack/ glance/ commit/ ?id=d17a1ed78a2 59789ec7d526580 57ca92a6935ead
Committed: https:/
Submitter: Jenkins
Branch: master
commit d17a1ed78a25978 9ec7d52658057ca 92a6935ead
Author: Flavio Percoco <email address hidden>
Date: Mon Dec 1 21:52:22 2014 +0100
Allow None values to be returned from the API
Currently, Glance's API v2 doesn't return fields whose value is None.
This, unfortunately, is wrong for a client perspective since it would
create inconsistencies between calls and images due to the lack of
fields in the response.
The API should guarantee consistency in its replies and ensure all
fields have a value, even if it's None.
NOTE: This work is part of the migration to v2. It fixes inconsistencies
in the API and improves the interaction between the client library and
Glance.
NOTE2: A follow-up patch will bump the minor API version, wait for it.
ApiImpact
DocImpact
Closes-bug: #1398314
Change-Id: Ieaddd8a686cf73 61f18cb1ee83b78 87cdca22bd6