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

Bug #1589865 reported by Hiroyuki Eguchi
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
In Progress
Undecided
Unassigned
tempest
Won't Fix
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)
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/330339

Changed in cinder:
status: New → In Progress
Changed in tempest:
assignee: nobody → Hiroyuki Eguchi (h-eguchi)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

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

Changed in tempest:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

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.

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

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

Revision history for this message
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
Revision history for this message
Sean McGinnis (sean-mcginnis) wrote : Bug Assignee Expired

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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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