)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"5671d266986ef984782c5311ba590f8d73322113","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactoring config options for ml2 plugin linuxbridge driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Refactoring neutron ml2 linuxbridge drivers configuration options to be in"},{"line_number":10,"context_line":"neutron/conf/plugins/ml2/drivers/linuxbridge. This would allow"},{"line_number":11,"context_line":"centralization of all configuration options and provides easy way to import."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dada55a8_6efb9979","line":9,"updated":"2016-07-21 21:15:02.000000000","message":"Commit message line exceeds 72 characters limit","commit_id":"626ff3085f8573d1d184b789da7bce7b349f5d3a"},{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"5671d266986ef984782c5311ba590f8d73322113","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Refactoring neutron ml2 linuxbridge drivers configuration options to be in"},{"line_number":10,"context_line":"neutron/conf/plugins/ml2/drivers/linuxbridge. This would allow"},{"line_number":11,"context_line":"centralization of all configuration options and provides easy way to import."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4d86831978a18c96a9cf60df743246d46e7a8a05"},{"line_number":14,"context_line":"Partial-Bug: #1563069"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dada55a8_8eec0dbc","line":11,"updated":"2016-07-21 21:15:02.000000000","message":"Same as above","commit_id":"626ff3085f8573d1d184b789da7bce7b349f5d3a"},{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"23a5f07ba367305e352f037d61fe1f794f12ec0e","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Refactoring neutron ml2 linuxbridge drivers configuration options to"},{"line_number":10,"context_line":"be in neutron/conf/plugins/ml2/drivers/linuxbridge. This would allow"},{"line_number":11,"context_line":"centralization of all configuration options and provides easy way to"},{"line_number":12,"context_line":"import."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I4d86831978a18c96a9cf60df743246d46e7a8a05"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bacf61ea_2c31c95a","line":11,"range":{"start_line":11,"start_character":57,"end_line":11,"end_character":61},"updated":"2016-08-01 18:48:17.000000000","message":"nit: an easy","commit_id":"a54bedb4d691c10a03a37f01d0ae2e619490930c"}],"neutron/conf/plugins/ml2/drivers/linuxbridge.py":[{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"5671d266986ef984782c5311ba590f8d73322113","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2012 Cisco Systems, Inc.  All rights reserved."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_6e4279af","line":1,"range":{"start_line":1,"start_character":12,"end_line":1,"end_character":16},"updated":"2016-07-21 21:15:02.000000000","message":"2016","commit_id":"626ff3085f8573d1d184b789da7bce7b349f5d3a"}],"neutron/plugins/ml2/drivers/linuxbridge/agent/common/config.py":[{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"23a5f07ba367305e352f037d61fe1f794f12ec0e","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"linuxbridge.register_linuxbridge_opts(linuxbridge.vxlan_opts, \"VXLAN\")"},{"line_number":19,"context_line":"linuxbridge.register_linuxbridge_opts(linuxbridge.bridge_opts, \"LINUX_BRIDGE\")"},{"line_number":20,"context_line":"linuxbridge.register_linuxbridge_opts(linuxbridge.qos_options, \"QOS\")"}],"source_content_type":"text/x-python","patch_set":4,"id":"bacf61ea_cfae1c2d","line":20,"updated":"2016-08-01 18:48:17.000000000","message":"Line (18-20): instead of making 3 different calls to register the opts can you please replace it with one call that registers all the opts at one shot in conf.plugins.ml2.drivers import linuxbridge","commit_id":"a54bedb4d691c10a03a37f01d0ae2e619490930c"},{"author":{"_account_id":6951,"name":"Brandon Logan","email":"brandon.logan@rackspace.com","username":"brandon-logan"},"change_message_id":"5e460d3586f2014ad3cffb5f203a3420af79b098","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"linuxbridge.register_linuxbridge_opts(linuxbridge.vxlan_opts, \"VXLAN\")"},{"line_number":19,"context_line":"linuxbridge.register_linuxbridge_opts(linuxbridge.bridge_opts, \"LINUX_BRIDGE\")"},{"line_number":20,"context_line":"linuxbridge.register_linuxbridge_opts(linuxbridge.qos_options, \"QOS\")"}],"source_content_type":"text/x-python","patch_set":4,"id":"bacf61ea_88141a4d","line":20,"in_reply_to":"bacf61ea_cfae1c2d","updated":"2016-08-01 20:05:58.000000000","message":"+1, the plugin_group parameter can just be done in the function itself for all 3 options.","commit_id":"a54bedb4d691c10a03a37f01d0ae2e619490930c"}]}