)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e094db601a313edf740d26fafa7da8a56514fde6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     ChangBo Guo(gcb) \u003cglongwave@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-04 05:13:10 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix wrong usage of config opiton gpfs_images_share_mode"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Config option gpfs_images_share_mode with paramter choices\u003d[\u0027copy\u0027,"},{"line_number":10,"context_line":"\u0027copy_on_write\u0027, None], that means any other value will raise"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a77a97e_bb02bd07","line":7,"updated":"2016-11-16 10:15:33.000000000","message":"nit: option","commit_id":"98e9317db3c16952d6217bc21f84aa0023136689"},{"author":{"_account_id":9796,"name":"ChangBo Guo","email":"glongwave@gmail.com","username":"gcb"},"change_message_id":"98f4e740687c45cbd98eef41be958cccf1f525f8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     ChangBo Guo(gcb) \u003cglongwave@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-04 05:13:10 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix wrong usage of config opiton gpfs_images_share_mode"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Config option gpfs_images_share_mode with paramter choices\u003d[\u0027copy\u0027,"},{"line_number":10,"context_line":"\u0027copy_on_write\u0027, None], that means any other value will raise"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a77a97e_e88cf7db","line":7,"in_reply_to":"7a77a97e_bb02bd07","updated":"2016-11-18 09:30:47.000000000","message":"Done","commit_id":"98e9317db3c16952d6217bc21f84aa0023136689"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"c8467f701f73ca6eb04d5dc3b97e67e64d4da451","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix wrong usage of config opiton gpfs_images_share_mode"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Config option gpfs_images_share_mode with paramter choices\u003d[\u0027copy\u0027,"},{"line_number":10,"context_line":"\u0027copy_on_write\u0027, None], that means any other value will raise"},{"line_number":11,"context_line":"ValueError, so don\u0027t check other values."},{"line_number":12,"context_line":"For more details when we set method set_override with paramter"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ba5da102_3e3fbfd7","line":9,"updated":"2016-11-04 15:20:50.000000000","message":"parameter","commit_id":"98e9317db3c16952d6217bc21f84aa0023136689"},{"author":{"_account_id":9796,"name":"ChangBo Guo","email":"glongwave@gmail.com","username":"gcb"},"change_message_id":"98f4e740687c45cbd98eef41be958cccf1f525f8","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix wrong usage of config opiton gpfs_images_share_mode"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Config option gpfs_images_share_mode with paramter choices\u003d[\u0027copy\u0027,"},{"line_number":10,"context_line":"\u0027copy_on_write\u0027, None], that means any other value will raise"},{"line_number":11,"context_line":"ValueError, so don\u0027t check other values."},{"line_number":12,"context_line":"For more details when we set method set_override with paramter"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a77a97e_488ecbd0","line":9,"in_reply_to":"ba5da102_3e3fbfd7","updated":"2016-11-18 09:30:47.000000000","message":"Done","commit_id":"98e9317db3c16952d6217bc21f84aa0023136689"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e094db601a313edf740d26fafa7da8a56514fde6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Config option gpfs_images_share_mode with paramter choices\u003d[\u0027copy\u0027,"},{"line_number":10,"context_line":"\u0027copy_on_write\u0027, None], that means any other value will raise"},{"line_number":11,"context_line":"ValueError, so don\u0027t check other values."},{"line_number":12,"context_line":"For more details when we set method set_override with paramter"},{"line_number":13,"context_line":"enforce_type\u003dTrue, please see: http://paste.openstack.org/show/523740/"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a77a97e_3b3dcdbb","line":11,"range":{"start_line":11,"start_character":12,"end_line":11,"end_character":40},"updated":"2016-11-16 10:15:33.000000000","message":"nit: Maybe be more explicit about where we don\u0027t want to check it.  As in:  the check is done by the configuration parser, so no invalid value will be accepted and the driver doesn\u0027t need to check it again.","commit_id":"98e9317db3c16952d6217bc21f84aa0023136689"},{"author":{"_account_id":9796,"name":"ChangBo Guo","email":"glongwave@gmail.com","username":"gcb"},"change_message_id":"98f4e740687c45cbd98eef41be958cccf1f525f8","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Config option gpfs_images_share_mode with paramter choices\u003d[\u0027copy\u0027,"},{"line_number":10,"context_line":"\u0027copy_on_write\u0027, None], that means any other value will raise"},{"line_number":11,"context_line":"ValueError, so don\u0027t check other values."},{"line_number":12,"context_line":"For more details when we set method set_override with paramter"},{"line_number":13,"context_line":"enforce_type\u003dTrue, please see: http://paste.openstack.org/show/523740/"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a77a97e_48532b39","line":11,"range":{"start_line":11,"start_character":12,"end_line":11,"end_character":40},"in_reply_to":"7a77a97e_3b3dcdbb","updated":"2016-11-18 09:30:47.000000000","message":"Done","commit_id":"98e9317db3c16952d6217bc21f84aa0023136689"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"a0bc7b8dab59fd52d61a92646c282e0ad581d725","unresolved":false,"context_lines":[{"line_number":14,"context_line":"enforce_type\u003dTrue, please see: http://paste.openstack.org/show/523740/"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Related-Bug: 1517839"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I000ab639e68534e7be111cc3759b6ae1dffd5520"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"7a77a97e_b6e77d45","line":17,"updated":"2016-11-21 11:13:10.000000000","message":"nit: Empty line is not necessary here.","commit_id":"73b5710c97c30c1bf55c51aa3857cf7fa52452a2"}]}