)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f1bb948fa5aec0db28f4bcf243bed95291d0ebff","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jay S. Bryant \u003cjsbryant@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-13 15:28:52 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix BVT cinder test_volume_from_image failure in DB2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As db2 does not support SQL insert via raw number type if"},{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F%2FI%2BQ%3D","line":7,"updated":"2013-11-13 22:17:54.000000000","message":"What is \"BVT\"?","commit_id":"31dcc53852be77155831750641456981dc6403f0"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"18e4b3872493a6f5a444077281ed68ab5b9f1ba2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jay S. Bryant \u003cjsbryant@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-13 15:28:52 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix BVT cinder test_volume_from_image failure in DB2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As db2 does not support SQL insert via raw number type if"},{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F%2FI9g%3D","line":7,"in_reply_to":"AAAATn%2F%2FI%2BQ%3D","updated":"2013-11-13 22:21:06.000000000","message":"Ooops, I removed the other Internal IBM stuff.  \u0027Build Verification Test\u0027.  I will fix that.","commit_id":"31dcc53852be77155831750641456981dc6403f0"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a5678b16b8e6a93914e366f4de22ec96ffb6c9a3","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jay S. Bryant \u003cjsbryant@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-13 16:20:30 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix cinder test_volume_from_image failure in DB2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As db2 does not support SQL insert via raw number type if"},{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F%2FIVQ%3D","line":7,"updated":"2013-11-14 02:44:10.000000000","message":"So you\u0027re fixing this to fix a test but shouldn\u0027t there be a bug report related to the test?  I\u0027m assuming this is Tempest, or not?","commit_id":"638676f69ee893126e487ced8f3ac6a230532e85"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"cd9e27d9a597a60154fe567608409abc28357c60","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jay S. Bryant \u003cjsbryant@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-13 16:20:30 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix cinder test_volume_from_image failure in DB2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As db2 does not support SQL insert via raw number type if"},{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F%2FDfs%3D","line":7,"in_reply_to":"AAAATn%2F%2FEmA%3D","updated":"2013-11-15 03:52:20.000000000","message":"Yeah, but I can\u0027t seem to find \u0027test_volume_from_image\u0027 in tempest now, or in the stable/branches.  Do you have details on that?  Otherwise it might look kind of weird to refer to a test in tempest that doesn\u0027t exist.","commit_id":"638676f69ee893126e487ced8f3ac6a230532e85"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"54d046bebdcb2b89580e28d167fa69bc88d1ba3e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jay S. Bryant \u003cjsbryant@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-13 16:20:30 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix cinder test_volume_from_image failure in DB2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As db2 does not support SQL insert via raw number type if"},{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F%2FEmA%3D","line":7,"in_reply_to":"AAAATn%2F%2FIVQ%3D","updated":"2013-11-14 20:14:01.000000000","message":"No, at this point in the development cycle we are not requiring a bug for every commit.  Yes this was a tempest failure.  I can update the descrption accordingly.","commit_id":"638676f69ee893126e487ced8f3ac6a230532e85"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a5678b16b8e6a93914e366f4de22ec96ffb6c9a3","unresolved":false,"context_lines":[{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"},{"line_number":11,"context_line":"to convert numerical values to string before insert operations."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Partial-blueprint db2-database"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5b378cce5b639df23d7003548f49bdab1eecf6a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F%2FIVM%3D","line":13,"updated":"2013-11-14 02:44:10.000000000","message":"Seems like this is more for fixing a bug with running tests against DB2 rather than the blueprint, and if it were reported as a bug you\u0027d have some stack trace to show the issue.","commit_id":"638676f69ee893126e487ced8f3ac6a230532e85"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"54d046bebdcb2b89580e28d167fa69bc88d1ba3e","unresolved":false,"context_lines":[{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"},{"line_number":11,"context_line":"to convert numerical values to string before insert operations."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Partial-blueprint db2-database"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5b378cce5b639df23d7003548f49bdab1eecf6a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F%2FEkg%3D","line":13,"in_reply_to":"AAAATn%2F%2FIVM%3D","updated":"2013-11-14 20:14:01.000000000","message":"This is required for Cinder to be working properly on DB2 and is probably one of the only commits required to address Cinder DB2 issues.  I feel that this is acceptable.","commit_id":"638676f69ee893126e487ced8f3ac6a230532e85"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"cb6af3a23c419bbb3896d242b0cde0e338758cea","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jay S. Bryant \u003cjsbryant@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-14 14:18:44 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix test_volume_from_image tempest failure in DB2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As db2 does not support SQL insert via raw number type if"},{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAATn%2F%2FDeM%3D","line":7,"updated":"2013-11-15 04:03:57.000000000","message":"I can\u0027t find this test anymore in Tempest, so we probably want to get to the bottom of that (renamed, removed?), otherwise it looks weird to refer to a failing tempest test that doesn\u0027t exist in the current tree.","commit_id":"eb205935693a3a57d9d1cc42f9a73bdf2614b5fd"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"2cebb8c268197e505f86a66493783a7970885740","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jay S. Bryant \u003cjsbryant@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-14 14:18:44 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix test_volume_from_image tempest failure in DB2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As db2 does not support SQL insert via raw number type if"},{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAATn%2F%2BvM4%3D","line":7,"in_reply_to":"AAAATn%2F%2FDeM%3D","updated":"2013-11-20 21:34:56.000000000","message":"Ok, so I did some more investigation and recreated to make sure that this wasn\u0027t something that was fixed via other changes that have been made for DB2 and here is what I found:  The failure still exists.  The test case has been moved to test_volumes_get and is now called test_volume_create_get_update_delete_from_image.  I am going to update this commit message accordingly.","commit_id":"eb205935693a3a57d9d1cc42f9a73bdf2614b5fd"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"36aaada2a5c41c04e28c7e8add4ac91d5b394bca","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jay S. Bryant \u003cjsbryant@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-20 15:44:27 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make volume_glance_metadata_create campat with DB2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As db2 does not support SQL insert via raw number type if"},{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F%2BvF8%3D","line":7,"updated":"2013-11-20 22:21:35.000000000","message":"s/campat/compat/","commit_id":"e940f8b8aceeb16518d236bf97bb1da5ea27f714"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"3cd8c8f8fcf78934413d0e287d90b5576d23bfbc","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jay S. Bryant \u003cjsbryant@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-20 15:44:27 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make volume_glance_metadata_create campat with DB2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As db2 does not support SQL insert via raw number type if"},{"line_number":10,"context_line":"table column data type is \"CLOB\"(\"TEXT\" in sqlalchemy), we need"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F%2BcZA%3D","line":7,"in_reply_to":"AAAATn%2F%2BvF8%3D","updated":"2013-11-26 00:19:40.000000000","message":"Done","commit_id":"e940f8b8aceeb16518d236bf97bb1da5ea27f714"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6883b21855108743f540193f162dd0044dbd251d","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This problem was discovered through the"},{"line_number":14,"context_line":"test_volume_create_get_update_delete_from_image test case in"},{"line_number":15,"context_line":"tempest.api.volume.test_volume_get.  The test case fails with"},{"line_number":16,"context_line":"a DBError without this change in place."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Partial-blueprint db2-database"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F%2BjRk%3D","line":15,"updated":"2013-11-24 03:14:53.000000000","message":"s/test_volume_get/test_volumes_get/","commit_id":"e940f8b8aceeb16518d236bf97bb1da5ea27f714"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"3cd8c8f8fcf78934413d0e287d90b5576d23bfbc","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This problem was discovered through the"},{"line_number":14,"context_line":"test_volume_create_get_update_delete_from_image test case in"},{"line_number":15,"context_line":"tempest.api.volume.test_volume_get.  The test case fails with"},{"line_number":16,"context_line":"a DBError without this change in place."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Partial-blueprint db2-database"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F%2BcY8%3D","line":15,"in_reply_to":"AAAATn%2F%2BjRk%3D","updated":"2013-11-26 00:19:40.000000000","message":"Done","commit_id":"e940f8b8aceeb16518d236bf97bb1da5ea27f714"}],"cinder/db/sqlalchemy/api.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a5678b16b8e6a93914e366f4de22ec96ffb6c9a3","unresolved":false,"context_lines":[{"line_number":2376,"context_line":"        vol_glance_metadata \u003d models.VolumeGlanceMetadata()"},{"line_number":2377,"context_line":"        vol_glance_metadata.volume_id \u003d volume_id"},{"line_number":2378,"context_line":"        vol_glance_metadata.key \u003d key"},{"line_number":2379,"context_line":"        vol_glance_metadata.value \u003d str(value)"},{"line_number":2380,"context_line":""},{"line_number":2381,"context_line":"        vol_glance_metadata.save(session\u003dsession)"},{"line_number":2382,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2FIVA%3D","line":2379,"updated":"2013-11-14 02:44:10.000000000","message":"Should be str() or six.text_type()?","commit_id":"638676f69ee893126e487ced8f3ac6a230532e85"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"54d046bebdcb2b89580e28d167fa69bc88d1ba3e","unresolved":false,"context_lines":[{"line_number":2376,"context_line":"        vol_glance_metadata \u003d models.VolumeGlanceMetadata()"},{"line_number":2377,"context_line":"        vol_glance_metadata.volume_id \u003d volume_id"},{"line_number":2378,"context_line":"        vol_glance_metadata.key \u003d key"},{"line_number":2379,"context_line":"        vol_glance_metadata.value \u003d str(value)"},{"line_number":2380,"context_line":""},{"line_number":2381,"context_line":"        vol_glance_metadata.save(session\u003dsession)"},{"line_number":2382,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2FEiw%3D","line":2379,"in_reply_to":"AAAATn%2F%2FIVA%3D","updated":"2013-11-14 20:14:01.000000000","message":"Given that six.text_type() isn\u0027t being used anywhere else in the file only str() is being used it seems appropriate and consistent to use str().","commit_id":"638676f69ee893126e487ced8f3ac6a230532e85"}]}