)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"edeed20631280f2c829c9273fe7c8aafe2bae023","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove downgrade from existing migrations"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove downgrade from all existing migrations and create test that"},{"line_number":10,"context_line":"verifies that all migrations in repo do not has downgrade."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related cross-project spec: https://review.openstack.org/152337"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a80dd14_2a92db9e","line":10,"updated":"2015-03-20 19:21:08.000000000","message":"do not have","commit_id":"082e75a6df80c4d973e7c09a5158222f85dd2cb2"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"edeed20631280f2c829c9273fe7c8aafe2bae023","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related cross-project spec: https://review.openstack.org/152337"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I2aad5f40f9aec9aea7ea869c9f30bc5ea586f3fe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a80dd14_2ad9fb82","line":14,"updated":"2015-03-20 19:21:08.000000000","message":"Closes-Bug: #1434103?","commit_id":"082e75a6df80c4d973e7c09a5158222f85dd2cb2"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"e55e1574eed31d98c3c24d1aae43e65310a23bb6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove downgrade from existing migrations"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove downgrade from all existing migrations and create test that"},{"line_number":10,"context_line":"verifies that all migrations in repo do not has downgrade."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related cross-project spec: https://review.openstack.org/152337"},{"line_number":13,"context_line":"Closes-Bug: 1434103"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9a80dd14_caa85f8e","line":10,"updated":"2015-03-20 19:21:46.000000000","message":"do not have","commit_id":"56c0ad040bfe9ac22cdf7b405487c2e97a800f87"}],"neutron/db/migration/alembic_migrations/versions/1680e1f0c4dc_remove_cisco_nexus_plugin.py":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"edeed20631280f2c829c9273fe7c8aafe2bae023","unresolved":false,"context_lines":[{"line_number":29,"context_line":"import sqlalchemy as sa"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"def upgrade(active_plugins\u003dNone, options\u003dNone):"},{"line_number":33,"context_line":"    op.execute(\u0027INSERT INTO cisco_ml2_nexusport_bindings (port_id, \u0027"},{"line_number":34,"context_line":"               \u0027vlan_id, switch_ip, instance_id) SELECT \u0027"},{"line_number":35,"context_line":"               \u0027port_id, vlan_id, switch_ip, instance_id FROM \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_6a4db33a","side":"PARENT","line":32,"updated":"2015-03-20 19:21:08.000000000","message":"This cleanup probably does not belong here (you leave similar garbage intact in other files).","commit_id":"e36753dfa3d3c34c9ed1b7b9f965b5d7c671ceb6"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"7c5b13059ce1706294203b77cc16965a1d7f3052","unresolved":false,"context_lines":[{"line_number":29,"context_line":"import sqlalchemy as sa"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"def upgrade(active_plugins\u003dNone, options\u003dNone):"},{"line_number":33,"context_line":"    op.execute(\u0027INSERT INTO cisco_ml2_nexusport_bindings (port_id, \u0027"},{"line_number":34,"context_line":"               \u0027vlan_id, switch_ip, instance_id) SELECT \u0027"},{"line_number":35,"context_line":"               \u0027port_id, vlan_id, switch_ip, instance_id FROM \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a6ced46_27965695","side":"PARENT","line":32,"in_reply_to":"7a75e904_5a7c2573","updated":"2015-03-23 13:42:13.000000000","message":"Nah, I haven\u0027t said it\u0027s worth a respin on its own.","commit_id":"e36753dfa3d3c34c9ed1b7b9f965b5d7c671ceb6"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"7107e54e5e25381515e8741899fd573a7993bc99","unresolved":false,"context_lines":[{"line_number":29,"context_line":"import sqlalchemy as sa"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"def upgrade(active_plugins\u003dNone, options\u003dNone):"},{"line_number":33,"context_line":"    op.execute(\u0027INSERT INTO cisco_ml2_nexusport_bindings (port_id, \u0027"},{"line_number":34,"context_line":"               \u0027vlan_id, switch_ip, instance_id) SELECT \u0027"},{"line_number":35,"context_line":"               \u0027port_id, vlan_id, switch_ip, instance_id FROM \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a75e904_5a7c2573","side":"PARENT","line":32,"in_reply_to":"9a80dd14_6a4db33a","updated":"2015-03-21 06:44:10.000000000","message":"This is only one more file with similar garbage, I though it is OK to remove it here, but if you think that this need a separate change I will split it.In my opinion it is not harmful to hold this together.","commit_id":"e36753dfa3d3c34c9ed1b7b9f965b5d7c671ceb6"}],"neutron/db/migration/alembic_migrations/versions/5446f2a45467_set_server_default.py":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"82acb32206670b341976ae170e352118091bd286","unresolved":false,"context_lines":[{"line_number":128,"context_line":""},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"def set_default_ml2():"},{"line_number":131,"context_line":"    default \u003d sqlalchemy.sql.false()"},{"line_number":132,"context_line":"    migration.alter_column_if_exists("},{"line_number":133,"context_line":"        \u0027ml2_gre_allocations\u0027, \u0027allocated\u0027,"},{"line_number":134,"context_line":"        server_default\u003ddefault,"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a6ced46_42a0b7f4","line":131,"updated":"2015-03-22 22:47:37.000000000","message":"seems like you could have made this \u0027default\u0027 a module-level constant, but it\u0027s a really minor thing not worth addressing","commit_id":"6a7bce54272ec50c34dad0bd1cfa42c51187d985"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"d28e23f6d3635d6489a3af44d0953d732564657f","unresolved":false,"context_lines":[{"line_number":128,"context_line":""},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"def set_default_ml2():"},{"line_number":131,"context_line":"    default \u003d sqlalchemy.sql.false()"},{"line_number":132,"context_line":"    migration.alter_column_if_exists("},{"line_number":133,"context_line":"        \u0027ml2_gre_allocations\u0027, \u0027allocated\u0027,"},{"line_number":134,"context_line":"        server_default\u003ddefault,"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a6ced46_f1bdeb03","line":131,"in_reply_to":"1a6ced46_42a0b7f4","updated":"2015-03-23 06:27:38.000000000","message":"Anyway I need to push new patch set with rebase, so I\u0027ll address this. Thanks!","commit_id":"6a7bce54272ec50c34dad0bd1cfa42c51187d985"}],"neutron/tests/functional/db/test_migrations.py":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"82acb32206670b341976ae170e352118091bd286","unresolved":false,"context_lines":[{"line_number":281,"context_line":"                failed_revisions.append(version.revision)"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":"        if failed_revisions:"},{"line_number":284,"context_line":"            self.fail(\u0027Migrations %s has downgrade\u0027 % failed_revisions)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a6ced46_62a5b303","line":284,"updated":"2015-03-22 22:47:37.000000000","message":"technically if there is more than one migration the error report should say \"have\" ;)","commit_id":"6a7bce54272ec50c34dad0bd1cfa42c51187d985"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"d28e23f6d3635d6489a3af44d0953d732564657f","unresolved":false,"context_lines":[{"line_number":281,"context_line":"                failed_revisions.append(version.revision)"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":"        if failed_revisions:"},{"line_number":284,"context_line":"            self.fail(\u0027Migrations %s has downgrade\u0027 % failed_revisions)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a6ced46_d192af9e","line":284,"in_reply_to":"1a6ced46_62a5b303","updated":"2015-03-23 06:27:38.000000000","message":"Oops :)","commit_id":"6a7bce54272ec50c34dad0bd1cfa42c51187d985"}]}