)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"01d4fa78a6a3743e11b91e189e69479e55b5df8f","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Better defaults in config files"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The defaults in the current configuration files of Neutron are"},{"line_number":10,"context_line":"sometimes missleading, and make it hard to parse or use sed to"},{"line_number":11,"context_line":"switch values. This patch is what I normally apply to the"},{"line_number":12,"context_line":"Debian package, and which I would really love to see applied"},{"line_number":13,"context_line":"also \"upstream\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWH%2F9fIs%3D","line":10,"updated":"2014-04-04 07:19:36.000000000","message":"grammar nazi warning on \"missleading\"!","commit_id":"b23f0edc0a7c758febfe8177427d2d50bb50bb4b"}],"etc/l3_agent.ini":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"01d4fa78a6a3743e11b91e189e69479e55b5df8f","unresolved":false,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":"# L3 requires that an interface driver be set. Choose the one that best"},{"line_number":6,"context_line":"# matches your plugin."},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"# Example of interface_driver option for OVS based plugins (OVS, Ryu, NEC)"},{"line_number":9,"context_line":"# that supports L3 agent"},{"line_number":10,"context_line":"interface_driver \u003d neutron.agent.linux.interface.OVSInterfaceDriver"}],"source_content_type":"text/x-properties","patch_set":1,"id":"AAAAWH%2F9fJk%3D","line":7,"updated":"2014-04-04 07:19:36.000000000","message":"blank line here not needed anymore","commit_id":"b23f0edc0a7c758febfe8177427d2d50bb50bb4b"}],"etc/lbaas_agent.ini":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"26a7a4cd5ba55142c4fac32bc427ec651c46ba6b","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"# Example of interface_driver option for OVS based plugins (OVS, Ryu, NEC, NVP,"},{"line_number":14,"context_line":"# BigSwitch/Floodlight)"},{"line_number":15,"context_line":"interface_driver \u003d neutron.agent.linux.interface.OVSInterfaceDriver"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"# Use veth for an OVS interface or not."},{"line_number":18,"context_line":"# Support kernels with limited namespace support"}],"source_content_type":"text/x-properties","patch_set":1,"id":"AAAAWH%2F9iO4%3D","line":15,"updated":"2014-04-03 21:15:38.000000000","message":"example line is commented out because neutron.agent.linux.interface.OVSInterfaceDriver is the default value for the option that is set in the code.\nThat\u0027s a practice across all config files.","commit_id":"b23f0edc0a7c758febfe8177427d2d50bb50bb4b"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"13177f9a6649811b507797a1b7421cc3f0ff5b8a","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"# Example of interface_driver option for OVS based plugins (OVS, Ryu, NEC, NVP,"},{"line_number":14,"context_line":"# BigSwitch/Floodlight)"},{"line_number":15,"context_line":"interface_driver \u003d neutron.agent.linux.interface.OVSInterfaceDriver"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"# Use veth for an OVS interface or not."},{"line_number":18,"context_line":"# Support kernels with limited namespace support"}],"source_content_type":"text/x-properties","patch_set":1,"id":"AAAAWH%2F9Nio%3D","line":15,"in_reply_to":"AAAAWH%2F9iO4%3D","updated":"2014-04-07 22:32:55.000000000","message":"The default value of interface_driver is None, so it is valid.\nhttp://git.openstack.org/cgit/openstack/neutron/tree/neutron/agent/common/config.py#n41","commit_id":"b23f0edc0a7c758febfe8177427d2d50bb50bb4b"}],"etc/neutron.conf":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"26a7a4cd5ba55142c4fac32bc427ec651c46ba6b","unresolved":false,"context_lines":[{"line_number":60,"context_line":"# of its entrypoint name."},{"line_number":61,"context_line":"#"},{"line_number":62,"context_line":"# service_plugins \u003d"},{"line_number":63,"context_line":"# Example: service_plugins \u003d router,firewall,lbaas,vpnaas,metering"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"# Paste configuration file"},{"line_number":66,"context_line":"# api_paste_config \u003d api-paste.ini"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAWH%2F9iOs%3D","side":"PARENT","line":63,"updated":"2014-04-03 21:15:38.000000000","message":"it makes sense to change the comment to correspond to actual plugin alias names","commit_id":"b62fb20ec22abaaf8f5da58fac60fafdaa3bd86e"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"01d4fa78a6a3743e11b91e189e69479e55b5df8f","unresolved":false,"context_lines":[{"line_number":58,"context_line":"# with previous versions, the class name of a plugin can be specified instead"},{"line_number":59,"context_line":"# of its entrypoint name."},{"line_number":60,"context_line":"#"},{"line_number":61,"context_line":"# Example: service_plugins \u003d router,firewall,lbaas,vpnaas,metering"},{"line_number":62,"context_line":"service_plugins \u003d neutron.services.loadbalancer.plugin.LoadBalancerPlugin,neutron.services.metering.metering_plugin.MeteringPlugin,neutron.services.l3_router.l3_router_plugin.L3RouterPlugin"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"# Paste configuration file"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAWH%2F9fKw%3D","line":61,"updated":"2014-04-04 07:19:36.000000000","message":"I would also remove this line. It\u0027s confusing.","commit_id":"b23f0edc0a7c758febfe8177427d2d50bb50bb4b"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"26a7a4cd5ba55142c4fac32bc427ec651c46ba6b","unresolved":false,"context_lines":[{"line_number":59,"context_line":"# of its entrypoint name."},{"line_number":60,"context_line":"#"},{"line_number":61,"context_line":"# Example: service_plugins \u003d router,firewall,lbaas,vpnaas,metering"},{"line_number":62,"context_line":"service_plugins \u003d neutron.services.loadbalancer.plugin.LoadBalancerPlugin,neutron.services.metering.metering_plugin.MeteringPlugin,neutron.services.l3_router.l3_router_plugin.L3RouterPlugin"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"# Paste configuration file"},{"line_number":65,"context_line":"# api_paste_config \u003d api-paste.ini"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAWH%2F9iOU%3D","line":62,"updated":"2014-04-03 21:15:38.000000000","message":"that\u0027s really up to a deployment configuration, not a default set of plugins","commit_id":"b23f0edc0a7c758febfe8177427d2d50bb50bb4b"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"01d4fa78a6a3743e11b91e189e69479e55b5df8f","unresolved":false,"context_lines":[{"line_number":59,"context_line":"# of its entrypoint name."},{"line_number":60,"context_line":"#"},{"line_number":61,"context_line":"# Example: service_plugins \u003d router,firewall,lbaas,vpnaas,metering"},{"line_number":62,"context_line":"service_plugins \u003d neutron.services.loadbalancer.plugin.LoadBalancerPlugin,neutron.services.metering.metering_plugin.MeteringPlugin,neutron.services.l3_router.l3_router_plugin.L3RouterPlugin"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"# Paste configuration file"},{"line_number":65,"context_line":"# api_paste_config \u003d api-paste.ini"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAWH%2F9fK0%3D","line":62,"in_reply_to":"AAAAWH%2F9iOU%3D","updated":"2014-04-04 07:19:36.000000000","message":"One problem with this setting is that it differs from the default parameter in the conf variable (which is nothing).\n\nHowever, if one want to have a workable configuration just removing leading hashes, then with ml2 you would need at least the l3_router service plugin.\n\nInstead of going into changes which might become complex for packaging plugins different from ML2, I might be ok with something like the following\n\n# The following minimal service plugins should be used when ML2 is enabled\n# service_plugins \u003d neutron.services.l3_router.l3_router_plugin.L3RouterPlugin","commit_id":"b23f0edc0a7c758febfe8177427d2d50bb50bb4b"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"01d4fa78a6a3743e11b91e189e69479e55b5df8f","unresolved":false,"context_lines":[{"line_number":399,"context_line":""},{"line_number":400,"context_line":"[database]"},{"line_number":401,"context_line":"# This line MUST be changed to actually run the plugin."},{"line_number":402,"context_line":"# Replace 127.0.0.1 above with the IP address of the database used by the"},{"line_number":403,"context_line":"# main neutron server. (Leave it as is if the database runs on this host.)"},{"line_number":404,"context_line":"connection \u003d sqlite:///var/lib/neutron/neutrondb"},{"line_number":405,"context_line":""}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAWH%2F9fKY%3D","line":402,"updated":"2014-04-04 07:19:36.000000000","message":"if you make this change 401-403 won\u0027t be useful anymore.\n\nIt might also be interesting to see how ubuntu, fedora and others are currently packaging.\nBut probably whatever they\u0027re doing won\u0027t work for debian.","commit_id":"b23f0edc0a7c758febfe8177427d2d50bb50bb4b"}],"etc/neutron/plugins/openvswitch/ovs_neutron_plugin.ini":[{"author":{"_account_id":8873,"name":"Assaf Muller","email":"amuller@redhat.com","username":"amuller"},"change_message_id":"9b2e47655fc78131592be775b631ca0a4c7e343f","unresolved":false,"context_lines":[{"line_number":135,"context_line":""},{"line_number":136,"context_line":"# Controls if neutron security group is enabled or not."},{"line_number":137,"context_line":"# It should be false when you use nova security group."},{"line_number":138,"context_line":"# enable_security_group \u003d True"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"#-----------------------------------------------------------------------------"},{"line_number":141,"context_line":"# Sample Configurations."}],"source_content_type":"text/x-properties","patch_set":1,"id":"AAAAWn%2F%2F%2BwA%3D","side":"PARENT","line":138,"updated":"2014-04-08 05:29:10.000000000","message":"I\u0027ve always found this extremely effective documentation. Why remove it?","commit_id":"b62fb20ec22abaaf8f5da58fac60fafdaa3bd86e"}]}