Unify the type of value for what are True/False values

Bug #1589865 reported by Hiroyuki Eguchi on 2016-06-07
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Undecided
Unassigned
tempest
Undecided
Hiroyuki Eguchi

Bug Description

A bootable field in cinder volume data returned from cinder-api is a string data type.
However, a crypted field is a boolean data type.

{"volumes": [
 {"status": "available",
 "user_id": "b398614e552a4250a8e9d6c5079c74fa",
 "attachments": [],
 "links": [],
 "availability_zone": "nova",
★ "bootable": "false",
★ "encrypted": false,
     ...

The type of value which can be True or False should be boolean.

This issue is causing a another issue in horizon.
https://bugs.launchpad.net/horizon/+bug/1587263

Changed in cinder:
assignee: nobody → Hiroyuki Eguchi (h-eguchi)

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

Changed in cinder:
status: New → In Progress
Changed in tempest:
assignee: nobody → Hiroyuki Eguchi (h-eguchi)

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

Changed in tempest:
status: New → In Progress

Change abandoned by Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/330339
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Change abandoned by Hiroyuki Eguchi (<email address hidden>) on branch: master
Review: https://review.openstack.org/331400

Ken'ichi Ohmichi (oomichi) wrote :

The change of cinder was denied on the cinder side, and I think this report of tempest is not necessary now.

Changed in tempest:
status: In Progress → Won't Fix

Unassigning due to no activity for > 6 months.

Changed in cinder:
assignee: Hiroyuki Eguchi (h-eguchi) → nobody
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers