)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"7f6245a8e664067baec29eb4f987b5d51ec8cb7c","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-06-16 15:03:59 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Metaplugin deprecation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I9cf36e1fd3a009c175e0d475af407a30f4e5c408"},{"line_number":10,"context_line":"Closes-Bug: #1465126"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa32b979_d71230b4","line":8,"updated":"2015-06-16 06:11:09.000000000","message":"DocImpact","commit_id":"c4c1f2f9d8206f6a9b07dfa7ebe1d657c02d3fb6"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"158dbbeb9cfad3204c4be9216f7615f6c2bffbbf","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-06-16 15:03:59 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Metaplugin deprecation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I9cf36e1fd3a009c175e0d475af407a30f4e5c408"},{"line_number":10,"context_line":"Closes-Bug: #1465126"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa32b979_b92757b5","line":8,"in_reply_to":"fa32b979_d71230b4","updated":"2015-06-16 10:26:54.000000000","message":"Done","commit_id":"c4c1f2f9d8206f6a9b07dfa7ebe1d657c02d3fb6"},{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"ba9356a9828ca55aa4ec7f06cac0e991562b215a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Metaplugin deprecation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Metaplugin was marked as deprecated in liberty."},{"line_number":10,"context_line":"Remove all metaplugin related codes."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"DocImpact"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa32b979_d92fbbf4","line":9,"updated":"2015-06-16 10:39:44.000000000","message":"I am lost, metaplugin is deprecated or removed in Liberty?\n\nAccording to the bug, it\u0027s only deprecated","commit_id":"d25f908f7bcfbe11fbedb4252c3bb819eb919ad4"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"47e3167fc98308fa3de6e874a3dff439f0bd4a56","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Metaplugin deprecation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Metaplugin was marked as deprecated in liberty."},{"line_number":10,"context_line":"Remove all metaplugin related codes."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"DocImpact"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa32b979_a24b8f86","line":9,"in_reply_to":"fa32b979_d92fbbf4","updated":"2015-06-16 15:10:00.000000000","message":"Metaplugin is removed in Liberty.\n\nI updated launchpad bug.","commit_id":"d25f908f7bcfbe11fbedb4252c3bb819eb919ad4"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"9adc13274d56cebb53d90df77038f5b3ce2a5b34","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Hirofumi Ichihara \u003cichihara.hirofumi@lab.ntt.co.jp\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-06-17 00:45:16 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Metaplugin deprecation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Metaplugin is removed in Liberty."},{"line_number":10,"context_line":"Remove all metaplugin related codes."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"fa32b979_aec5b23c","line":7,"updated":"2015-06-16 18:28:37.000000000","message":"This is a removal of metaplugin.\nThe commit message need to be updated.","commit_id":"579ff36787d036f0a5e7258f679146bd4539f569"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"786b99dfff8cf164e1d7a8c2564bbd2bd1a068f4","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Hirofumi Ichihara \u003cichihara.hirofumi@lab.ntt.co.jp\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-06-17 00:45:16 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Metaplugin deprecation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Metaplugin is removed in Liberty."},{"line_number":10,"context_line":"Remove all metaplugin related codes."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"fa32b979_19265c15","line":7,"in_reply_to":"fa32b979_aec5b23c","updated":"2015-06-17 01:36:28.000000000","message":"Done","commit_id":"579ff36787d036f0a5e7258f679146bd4539f569"}],"neutron/agent/linux/interface.py":[{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"7f6245a8e664067baec29eb4f987b5d51ec8cb7c","unresolved":false,"context_lines":[{"line_number":55,"context_line":"               help\u003d_(\"Authentication region\")),"},{"line_number":56,"context_line":"    cfg.StrOpt(\u0027endpoint_type\u0027,"},{"line_number":57,"context_line":"               default\u003d\u0027publicURL\u0027,"},{"line_number":58,"context_line":"               help\u003d_(\"Network service endpoint type to pull from \""},{"line_number":59,"context_line":"                      \"the keystone catalog\")),"},{"line_number":60,"context_line":"]"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa32b979_f72e54f9","line":58,"updated":"2015-06-16 06:11:09.000000000","message":"Line 43-59 must be deleted since they are used only with meta_flavor_driver_mappings.","commit_id":"c4c1f2f9d8206f6a9b07dfa7ebe1d657c02d3fb6"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"158dbbeb9cfad3204c4be9216f7615f6c2bffbbf","unresolved":false,"context_lines":[{"line_number":55,"context_line":"               help\u003d_(\"Authentication region\")),"},{"line_number":56,"context_line":"    cfg.StrOpt(\u0027endpoint_type\u0027,"},{"line_number":57,"context_line":"               default\u003d\u0027publicURL\u0027,"},{"line_number":58,"context_line":"               help\u003d_(\"Network service endpoint type to pull from \""},{"line_number":59,"context_line":"                      \"the keystone catalog\")),"},{"line_number":60,"context_line":"]"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa32b979_b9751799","line":58,"in_reply_to":"fa32b979_f72e54f9","updated":"2015-06-16 10:26:54.000000000","message":"Done","commit_id":"c4c1f2f9d8206f6a9b07dfa7ebe1d657c02d3fb6"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"0f9d0aeb63f0e1f804b1e9680d240a6e1fd8611d","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import netaddr"},{"line_number":19,"context_line":"from oslo_config import cfg"},{"line_number":20,"context_line":"from oslo_log import log as logging"},{"line_number":21,"context_line":"from oslo_utils import importutils"},{"line_number":22,"context_line":"import six"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from neutron.agent.common import ovs_lib"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa32b979_663906a4","line":21,"updated":"2015-06-16 22:41:32.000000000","message":"This line must be deleted since it is used by MetaInterfaceDriver only.\nIt is cause of pep8 error.","commit_id":"579ff36787d036f0a5e7258f679146bd4539f569"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"786b99dfff8cf164e1d7a8c2564bbd2bd1a068f4","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import netaddr"},{"line_number":19,"context_line":"from oslo_config import cfg"},{"line_number":20,"context_line":"from oslo_log import log as logging"},{"line_number":21,"context_line":"from oslo_utils import importutils"},{"line_number":22,"context_line":"import six"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from neutron.agent.common import ovs_lib"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa32b979_b91e50b9","line":21,"in_reply_to":"fa32b979_663906a4","updated":"2015-06-17 01:36:28.000000000","message":"Done","commit_id":"579ff36787d036f0a5e7258f679146bd4539f569"}],"neutron/db/migration/alembic_migrations/versions/2a16083502f3_metaplugin_deprecation.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"9adc13274d56cebb53d90df77038f5b3ce2a5b34","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"fa32b979_0eadbe30","updated":"2015-06-16 18:28:37.000000000","message":"This is actually not a renaming of files.\ngit or gerrit is sometimes confused for small files.\n\nIn addition, the filename contains \"deprecation\". It needs to be renamed to have \"removal\".","commit_id":"579ff36787d036f0a5e7258f679146bd4539f569"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"786b99dfff8cf164e1d7a8c2564bbd2bd1a068f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"fa32b979_797be8f5","in_reply_to":"fa32b979_0eadbe30","updated":"2015-06-17 01:36:28.000000000","message":"Done","commit_id":"579ff36787d036f0a5e7258f679146bd4539f569"}],"neutron/db/migration/alembic_migrations/versions/2a16083502f3_metaplugin_removal.py":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"830a90b08908451efdd953fd31af4add4423c2da","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def upgrade():"},{"line_number":32,"context_line":"    op.drop_table(\u0027networkflavors\u0027)"},{"line_number":33,"context_line":"    op.drop_table(\u0027routerflavors\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"fa32b979_3c305807","line":32,"updated":"2015-06-19 19:51:30.000000000","message":"It\u0027s a contract migration rule, as per online db migration scheme [1], so should be put on hold until we implement plan for online migrations.\n\n[1]: https://review.openstack.org/192937","commit_id":"9ddb4e11fb27f672dd717f26f05f82acb43f3e39"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"670d876a95927d1fdf88e74f628d8d1666a8e6e8","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def upgrade():"},{"line_number":32,"context_line":"    op.drop_table(\u0027networkflavors\u0027)"},{"line_number":33,"context_line":"    op.drop_table(\u0027routerflavors\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"fa32b979_ab9a5260","line":32,"in_reply_to":"fa32b979_11013d06","updated":"2015-06-24 08:10:41.000000000","message":"We should definitely kill it in L. I am working on a 3 phase db migration proposal from Mike and hope to come up with a patch in two weeks. Is it ok for you to rebase on top of it once it\u0027s in review?","commit_id":"9ddb4e11fb27f672dd717f26f05f82acb43f3e39"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"3641d8b367a8df8c3353f2991ecd6d2317c4e34d","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def upgrade():"},{"line_number":32,"context_line":"    op.drop_table(\u0027networkflavors\u0027)"},{"line_number":33,"context_line":"    op.drop_table(\u0027routerflavors\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"fa32b979_e9bd3403","line":32,"in_reply_to":"fa32b979_3c305807","updated":"2015-06-22 07:34:58.000000000","message":"Does this removal patch break the spec although I haven\u0027t read it yet? Can\u0027t we merge all patch including db migration until the implement is merged?","commit_id":"9ddb4e11fb27f672dd717f26f05f82acb43f3e39"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2a620f6e79713ea3c4002383d4949497d24e5b0b","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def upgrade():"},{"line_number":32,"context_line":"    op.drop_table(\u0027networkflavors\u0027)"},{"line_number":33,"context_line":"    op.drop_table(\u0027routerflavors\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"fa32b979_2bec02ba","line":32,"in_reply_to":"fa32b979_ab9a5260","updated":"2015-06-24 08:15:08.000000000","message":"That\u0027s OK. Thank you.","commit_id":"9ddb4e11fb27f672dd717f26f05f82acb43f3e39"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"351a78dd25577c2dd608a74c57312afd230f3441","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def upgrade():"},{"line_number":32,"context_line":"    op.drop_table(\u0027networkflavors\u0027)"},{"line_number":33,"context_line":"    op.drop_table(\u0027routerflavors\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"fa32b979_ec5462cb","line":32,"in_reply_to":"fa32b979_e9bd3403","updated":"2015-06-22 08:12:04.000000000","message":"It may break it or not, depending on how far we go with that. If we consider it as part of online db migration effort and not a mere rearrangement of migration rules, then yes, there is conflict that should be resolved.\n\nThe problem is merging everything now is that any non-additive patch landing in N cycle will postpone online db migration support till at least N+2. So we should be cautious.","commit_id":"9ddb4e11fb27f672dd717f26f05f82acb43f3e39"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"0b94ba9f7b674aa284353e4a45ce2d0442d338ed","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def upgrade():"},{"line_number":32,"context_line":"    op.drop_table(\u0027networkflavors\u0027)"},{"line_number":33,"context_line":"    op.drop_table(\u0027routerflavors\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"fa32b979_11013d06","line":32,"in_reply_to":"fa32b979_ec5462cb","updated":"2015-06-24 00:07:07.000000000","message":"I got it but I\u0027d like to remove Metaplugin in Liberty. Can I provide just removal patch? In a word, I will remove db/migration/* from this patch and besides, I will provide a patch for db migration after the discussion.","commit_id":"9ddb4e11fb27f672dd717f26f05f82acb43f3e39"}]}