)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"c38ea927ccd9afc945113b6e26955c803f861393","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Salvatore Orlando \u003csalv.orlando@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-08-20 05:42:11 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove auto-generation of db schema from models at startupwq"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug 1207402"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAARn%2F%2F4Bc%3D","line":7,"updated":"2013-08-20 20:35:55.000000000","message":"this is what happens when you witch from vi to nano for editing your commit mesage","commit_id":"2b9771489bcd524f015b437cb1dc6487d6238556"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"55e433ae4748150b842d6379fb084af22bcde18b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     154bca32 (Fix brocade and embrane plugin migrations)"},{"line_number":2,"context_line":"Author:     Salvatore Orlando \u003csalv.orlando@gmail.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2013-08-05 08:07:02 -0700"},{"line_number":4,"context_line":"Commit:     Henry Gessau \u003cgessau@cisco.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-04-08 20:41:21 -0400"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":22,"id":"AAAAW3%2F%2F4po%3D","line":2,"updated":"2014-04-09 17:39:21.000000000","message":"I think it is better to add co-author line. Henry have made a big input to this patch. (I just made small changes. Please ignore me)","commit_id":"f48e56ccd107493bc778972cda801ba143b772c2"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"2bbfc793129e9f98ab056d373cc2a0d3ad55ef2e","unresolved":false,"context_lines":[{"line_number":28,"context_line":"Added VMware NSX plugin (nsx) to folsom_initial to ensure"},{"line_number":29,"context_line":"to populate quotas table."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Closes-Bug: #1277379"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Change-Id: Ie4ee5507888ecad5f6dc32ce7a029c43014687a2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":22,"id":"AAAAW3%2F%2F7hk%3D","line":31,"updated":"2014-04-09 14:22:47.000000000","message":"It\u0027s better to keep the closes-bug lines together","commit_id":"f48e56ccd107493bc778972cda801ba143b772c2"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"55e433ae4748150b842d6379fb084af22bcde18b","unresolved":false,"context_lines":[{"line_number":28,"context_line":"Added VMware NSX plugin (nsx) to folsom_initial to ensure"},{"line_number":29,"context_line":"to populate quotas table."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Closes-Bug: #1277379"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Change-Id: Ie4ee5507888ecad5f6dc32ce7a029c43014687a2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":22,"id":"AAAAW3%2F%2F4qc%3D","line":31,"in_reply_to":"AAAAW3%2F%2F7hk%3D","updated":"2014-04-09 17:39:21.000000000","message":"I placed two closes-bug lines in separate places because L.7-21 is related to the first bug and L.23-29 is related to the second bug.\nIt may be better I should fix the second bug in a separate patch, but I didn\u0027t thought this bug lives so long at that time.","commit_id":"f48e56ccd107493bc778972cda801ba143b772c2"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"2bbfc793129e9f98ab056d373cc2a0d3ad55ef2e","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Closes-Bug: #1277379"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Change-Id: Ie4ee5507888ecad5f6dc32ce7a029c43014687a2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":22,"id":"AAAAW3%2F%2F7hI%3D","line":33,"updated":"2014-04-09 14:22:47.000000000","message":"I think this patch needs a docimpact.\nThere are users who believe neutron does automagically migrations, as they do not realize the schemas are being auto-generated","commit_id":"f48e56ccd107493bc778972cda801ba143b772c2"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"ffafcc0ac5e21654fddecbe5e0562c9af13f7294","unresolved":false,"context_lines":[{"line_number":17,"context_line":"The dhcpopts table was found to be missing from the ml2 plugin"},{"line_number":18,"context_line":"migrations, and are added here to pass the gate. Other plugins"},{"line_number":19,"context_line":"with missing migrations will be fixed separately."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: #1207402"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: Ie4ee5507888ecad5f6dc32ce7a029c43014687a2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":25,"id":"baada198_902013fa","line":20,"updated":"2014-07-23 04:54:23.000000000","message":"We should also DocImpact this because some people might be used to the old way.","commit_id":"92124121d8548070426d62647dba793606e4f442"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"d30cb2d2fe1ef1872331f4bea9f02c468542ee2a","unresolved":false,"context_lines":[{"line_number":17,"context_line":"The dhcpopts table was found to be missing from the ml2 plugin"},{"line_number":18,"context_line":"migrations, and are added here to pass the gate. Other plugins"},{"line_number":19,"context_line":"with missing migrations will be fixed separately."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: #1207402"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: Ie4ee5507888ecad5f6dc32ce7a029c43014687a2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":25,"id":"baada198_122b6e7c","line":20,"in_reply_to":"baada198_3b9f1167","updated":"2014-07-28 19:51:00.000000000","message":"Aargh, I forgot to update this with patch set 26. I promise to try harder to remember for patch set 27. :)","commit_id":"92124121d8548070426d62647dba793606e4f442"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"848fc0a989d0e4f7651fd2ced287170a2d39e1be","unresolved":false,"context_lines":[{"line_number":17,"context_line":"The dhcpopts table was found to be missing from the ml2 plugin"},{"line_number":18,"context_line":"migrations, and are added here to pass the gate. Other plugins"},{"line_number":19,"context_line":"with missing migrations will be fixed separately."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: #1207402"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: Ie4ee5507888ecad5f6dc32ce7a029c43014687a2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":25,"id":"baada198_389e376a","line":20,"in_reply_to":"baada198_902013fa","updated":"2014-07-30 19:23:24.000000000","message":"Done","commit_id":"92124121d8548070426d62647dba793606e4f442"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"c8dde381bcec5a38f4923cdb9b14223e08ac25be","unresolved":false,"context_lines":[{"line_number":17,"context_line":"The dhcpopts table was found to be missing from the ml2 plugin"},{"line_number":18,"context_line":"migrations, and are added here to pass the gate. Other plugins"},{"line_number":19,"context_line":"with missing migrations will be fixed separately."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: #1207402"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: Ie4ee5507888ecad5f6dc32ce7a029c43014687a2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":25,"id":"baada198_3b9f1167","line":20,"in_reply_to":"baada198_902013fa","updated":"2014-07-23 13:46:16.000000000","message":"Yes, good catch.","commit_id":"92124121d8548070426d62647dba793606e4f442"}],"neutron/db/migration/alembic_migrations/common_ext_ops.py":[{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"ffafcc0ac5e21654fddecbe5e0562c9af13f7294","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"def upgrade_quota(options\u003dNone):"},{"line_number":63,"context_line":"    if not (options or {}).get(\u0027folsom_quota_db_enabled\u0027):"},{"line_number":64,"context_line":"        return"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    op.create_table("}],"source_content_type":"text/x-python","patch_set":25,"id":"baada198_d0dfebd4","side":"PARENT","line":63,"updated":"2014-07-23 04:54:23.000000000","message":"I don\u0027t really think we need to remove this right?","commit_id":"895cf8ee86c4de8668f4f7a343435abe2a3fea4c"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"c8dde381bcec5a38f4923cdb9b14223e08ac25be","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"def upgrade_quota(options\u003dNone):"},{"line_number":63,"context_line":"    if not (options or {}).get(\u0027folsom_quota_db_enabled\u0027):"},{"line_number":64,"context_line":"        return"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    op.create_table("}],"source_content_type":"text/x-python","patch_set":25,"id":"baada198_3b8d512b","side":"PARENT","line":63,"in_reply_to":"baada198_d0dfebd4","updated":"2014-07-23 13:46:16.000000000","message":"I am not familiar with the history behind this option. Can you explain why we would want to keep it?","commit_id":"895cf8ee86c4de8668f4f7a343435abe2a3fea4c"}],"neutron/db/migration/alembic_migrations/versions/5ac71e65402c_ml2_initial.py":[{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"8b275275f42b86fbef222c1148cbe31efe042feb","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        sa.Column(\u0027resource\u0027, sa.String(length\u003d255), nullable\u003dTrue),"},{"line_number":78,"context_line":"        sa.Column(\u0027limit\u0027, sa.Integer(), nullable\u003dTrue),"},{"line_number":79,"context_line":"        sa.PrimaryKeyConstraint(\u0027id\u0027)"},{"line_number":80,"context_line":"    )"},{"line_number":81,"context_line":"    ### end Alembic commands ###"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAWH%2F%2FHpE%3D","line":80,"updated":"2014-03-20 04:58:25.000000000","message":"I put this in here to pass the gate. I suspect this is not the right fix for this table.","commit_id":"726bdce681b477ce4edf42736ef7e7a8e18c3814"}],"neutron/plugins/metaplugin/meta_neutron_plugin.py":[{"author":{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},"change_message_id":"ac916e911f365af847e225d22ffb7efd3e4270dd","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        cfg._is_opt_registered \u003d _is_opt_registered"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        # Keep existing tables if multiple plugin use same table name."},{"line_number":67,"context_line":"        model_base.NeutronBase.__table_args__ \u003d {\u0027keep_existing\u0027: True}"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        self.plugins \u003d {}"},{"line_number":70,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAARn%2F%2Fzeo%3D","line":67,"updated":"2013-08-21 18:40:29.000000000","message":"This line is not needed now.\nbecause we are removing table generation","commit_id":"e5d5826036e293f691cc3b5a429e76b27f0a0cfe"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"3ebbfd5ab5abec8c6a6dc39ef2a7bf40007b5291","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        cfg._is_opt_registered \u003d _is_opt_registered"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        # Keep existing tables if multiple plugin use same table name."},{"line_number":67,"context_line":"        model_base.NeutronBase.__table_args__ \u003d {\u0027keep_existing\u0027: True}"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        self.plugins \u003d {}"},{"line_number":70,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAARn%2F%2FzWg%3D","line":67,"in_reply_to":"AAAARn%2F%2Fzeo%3D","updated":"2013-08-21 19:21:41.000000000","message":"I wasn\u0027t sure about this so I left it to avoid mistakes.\nThanks for telling me!","commit_id":"e5d5826036e293f691cc3b5a429e76b27f0a0cfe"}],"neutron/tests/unit/testlib_api.py":[{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"841da23aa44af93cb90983723cad707ed6cd3684","unresolved":false,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def setUp(self):"},{"line_number":59,"context_line":"        super(SqlTestCase, self).setUp()"},{"line_number":60,"context_line":"        self.config_parse()"},{"line_number":61,"context_line":"        # Register all data models"},{"line_number":62,"context_line":"        engine \u003d db_api.get_engine()"},{"line_number":63,"context_line":"        model_base.BASEV2.metadata.create_all(engine)"}],"source_content_type":"text/x-python","patch_set":27,"id":"baada198_b12b47e9","line":60,"updated":"2014-07-30 17:48:28.000000000","message":"This line should be changed to self.setup_config()","commit_id":"dc4c662e284034a569544a0d1b673fb9287a582e"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"bc5374ecb2581a800b3932900185973c667e2865","unresolved":false,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def setUp(self):"},{"line_number":59,"context_line":"        super(SqlTestCase, self).setUp()"},{"line_number":60,"context_line":"        self.config_parse()"},{"line_number":61,"context_line":"        # Register all data models"},{"line_number":62,"context_line":"        engine \u003d db_api.get_engine()"},{"line_number":63,"context_line":"        model_base.BASEV2.metadata.create_all(engine)"}],"source_content_type":"text/x-python","patch_set":27,"id":"baada198_47a146a7","line":60,"in_reply_to":"baada198_b12b47e9","updated":"2014-07-30 21:35:47.000000000","message":"And BaseTestCase calls setup_config() so it is not needed here.","commit_id":"dc4c662e284034a569544a0d1b673fb9287a582e"}]}