)]}'
{"tempest/api/volume/test_volumes_actions.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"6c4a0391f0cfbfb8f2b05d3e667e72b1ab77e4ab","unresolved":false,"context_lines":[{"line_number":74,"context_line":"            fetched_volume \u003d self.client.show_volume("},{"line_number":75,"context_line":"                self.volume[\u0027id\u0027])[\u0027volume\u0027]"},{"line_number":76,"context_line":"            # Get Volume information"},{"line_number":77,"context_line":"            self.assertEqual(str(bool_bootable).lower(),"},{"line_number":78,"context_line":"                             fetched_volume[\u0027bootable\u0027])"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    @test.idempotent_id(\u00279516a2c8-9135-488c-8dd6-5677a7e5f371\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_4f6d9ecb","line":77,"range":{"start_line":77,"start_character":29,"end_line":77,"end_character":56},"updated":"2016-11-30 02:40:30.000000000","message":"This code seems odd for me at my first looking.\nAs IRC conversation, this is because of Cinder API itself.\nSo it is nice to write some NOTE why we need to contain this odd code for the maintenance.","commit_id":"7eef89fbd02261d88eceef589cd9046f889d25b8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"491bace0c88f08daa43d1bb4fa59adb68e4ab8e4","unresolved":false,"context_lines":[{"line_number":75,"context_line":"                self.volume[\u0027id\u0027])[\u0027volume\u0027]"},{"line_number":76,"context_line":"            # Get Volume information"},{"line_number":77,"context_line":"            self.assertEqual(str(bool_bootable).lower(),"},{"line_number":78,"context_line":"                             fetched_volume[\u0027bootable\u0027])"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    @test.idempotent_id(\u00279516a2c8-9135-488c-8dd6-5677a7e5f371\u0027)"},{"line_number":81,"context_line":"    @test.services(\u0027compute\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_fa949c37","line":78,"range":{"start_line":78,"start_character":29,"end_line":78,"end_character":55},"updated":"2016-11-29 08:59:33.000000000","message":"can this be in upper case (or bool not str) like True or \u0027True\u0027 as it was compared in is_true function. \n\nin py3 it will error while comparing different type (if cinder can return bool but not sure).","commit_id":"7eef89fbd02261d88eceef589cd9046f889d25b8"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"c9488eeef5c03a0ece5609798d237ebd8c22b7f6","unresolved":false,"context_lines":[{"line_number":75,"context_line":"                self.volume[\u0027id\u0027])[\u0027volume\u0027]"},{"line_number":76,"context_line":"            # Get Volume information"},{"line_number":77,"context_line":"            self.assertEqual(str(bool_bootable).lower(),"},{"line_number":78,"context_line":"                             fetched_volume[\u0027bootable\u0027])"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    @test.idempotent_id(\u00279516a2c8-9135-488c-8dd6-5677a7e5f371\u0027)"},{"line_number":81,"context_line":"    @test.services(\u0027compute\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_48993dc3","line":78,"range":{"start_line":78,"start_character":29,"end_line":78,"end_character":55},"in_reply_to":"5a74a57a_fa949c37","updated":"2016-11-29 12:26:21.000000000","message":"Oh, really? I actually couldn\u0027t find any True/False/\u0027True\u0027/\u0027False\u0027 in the gate. I grepped a tempest.txt log like this. But I could get booleans only in requests not response.\n\n $ grep \u0027\"bootable\":\u0027 tempest.txt| grep -v \u0027\"bootable\": \"true\"\u0027|grep -v \u0027\"bootable\": \"false\"\u0027\n        Body: {\"os-set_bootable\": {\"bootable\": false}}\n        Body: {\"os-set_bootable\": {\"bootable\": true}}\n        Body: {\"os-set_bootable\": {\"bootable\": false}}\n        Body: {\"os-set_bootable\": {\"bootable\": true}}\n        Body: {\"os-set_bootable\": {\"bootable\": false}}\n\nSo, could you show me an example or url link for it?","commit_id":"7eef89fbd02261d88eceef589cd9046f889d25b8"}]}
