)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f8ef2457058a546ee18f45253b7599147a056cb9","unresolved":false,"context_lines":[{"line_number":12,"context_line":"which is the intended type."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Unfortunately, doing a migration from VARCHAR to Enum on PostgreSQL is"},{"line_number":15,"context_line":"a lot of work. Replace it with a noop migration."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Co-authored-by: Michael Still \u003cmikal@stillhq.com\u003e"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa81d914_244aee83","line":15,"updated":"2015-01-27 21:14:39.000000000","message":"Missed the \"Replace it with a noop migration.\" so ignore my comment about updating the commit message.","commit_id":"4e962992bfc081fd16481492a8aa67a2c2ef35fe"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"cb63537ad3d55b27cdba871bb30d66b577e29f7c","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Due to policy around never removing a migration, we replace it with a noop"},{"line_number":18,"context_line":"migration instead. For the deployers who might have created this column,"},{"line_number":19,"context_line":"we need to remember to delete this column if it exists later."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"UpgradeImpact: if you have deployed migration 272 before this revert and"},{"line_number":22,"context_line":"run hyper-v, you need to come talk to us."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"fa81d914_5f0fafe1","line":19,"updated":"2015-01-27 21:22:03.000000000","message":"Is this a TODO for another cleanup migration to check if it exists and remove it?","commit_id":"c8e9f8e71de64273f10498c5ad959634bfe79975"}],"nova/db/sqlalchemy/migrate_repo/versions/272_add_keypair_type.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b9c6eea33d4cad8d82e877a7fed5aa28dc761712","unresolved":false,"context_lines":[{"line_number":22,"context_line":"# will have a poor experience because of this code revert, if that deployer"},{"line_number":23,"context_line":"# is you, please contact the nova team at openstack-dev@lists.openstack.org"},{"line_number":24,"context_line":"# and we will walk you through the manual fix required for this problem."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def upgrade(migrate_engine):"},{"line_number":27,"context_line":"    pass"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"fa81d914_24958ee4","line":25,"updated":"2015-01-27 21:12:31.000000000","message":"Need another blank line here for pep8.","commit_id":"4e962992bfc081fd16481492a8aa67a2c2ef35fe"}]}
