)]}'
{"lib/neutron_plugins/ml2":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"c91acdb90d61564ca68e31046cdfaa89dc92f766","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    Q_PLUGIN_CONF_FILENAME\u003dml2_conf.ini"},{"line_number":47,"context_line":"    Q_DB_NAME\u003d\"neutron_ml2\""},{"line_number":48,"context_line":"    Q_PLUGIN_CLASS\u003d\"neutron.plugins.ml2.plugin.Ml2Plugin\""},{"line_number":49,"context_line":"    # The ML2 plugin delegates L3 routing/NAT functionality to "},{"line_number":50,"context_line":"    # the L3 service plugin which must therefore be specified."},{"line_number":51,"context_line":"    Q_L3_PLUGIN_CLASS\u003d${Q_L3_PLUGIN_CLASS:-\"neutron.services.l3_router.l3_router_plugin.L3RouterPlugin\"}"},{"line_number":52,"context_line":"    if  ini_has_option $NEUTRON_CONF DEFAULT service_plugins ; then"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAASn%2F%2FUns%3D","line":49,"updated":"2013-09-06 11:33:43.000000000","message":"what\u0027s devstack policy re: whitespaces?","commit_id":"78892fe34313f5e7e870b15873eb1953844229f7"},{"author":{"_account_id":162,"name":"Bob Melander","email":"bob.melander@gmail.com","username":"bob-melander"},"change_message_id":"649f8fe32bf33d68a1a74ebf9212b7441fdf57b2","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    Q_PLUGIN_CONF_FILENAME\u003dml2_conf.ini"},{"line_number":47,"context_line":"    Q_DB_NAME\u003d\"neutron_ml2\""},{"line_number":48,"context_line":"    Q_PLUGIN_CLASS\u003d\"neutron.plugins.ml2.plugin.Ml2Plugin\""},{"line_number":49,"context_line":"    # The ML2 plugin delegates L3 routing/NAT functionality to "},{"line_number":50,"context_line":"    # the L3 service plugin which must therefore be specified."},{"line_number":51,"context_line":"    Q_L3_PLUGIN_CLASS\u003d${Q_L3_PLUGIN_CLASS:-\"neutron.services.l3_router.l3_router_plugin.L3RouterPlugin\"}"},{"line_number":52,"context_line":"    if  ini_has_option $NEUTRON_CONF DEFAULT service_plugins ; then"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAASn%2F%2FOVA%3D","line":49,"in_reply_to":"AAAASn%2F%2FUSc%3D","updated":"2013-09-09 18:36:23.000000000","message":"Done","commit_id":"78892fe34313f5e7e870b15873eb1953844229f7"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"d7ea2a2b26775851b2aaf67ee28e69b6905b2f39","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    Q_PLUGIN_CONF_FILENAME\u003dml2_conf.ini"},{"line_number":47,"context_line":"    Q_DB_NAME\u003d\"neutron_ml2\""},{"line_number":48,"context_line":"    Q_PLUGIN_CLASS\u003d\"neutron.plugins.ml2.plugin.Ml2Plugin\""},{"line_number":49,"context_line":"    # The ML2 plugin delegates L3 routing/NAT functionality to "},{"line_number":50,"context_line":"    # the L3 service plugin which must therefore be specified."},{"line_number":51,"context_line":"    Q_L3_PLUGIN_CLASS\u003d${Q_L3_PLUGIN_CLASS:-\"neutron.services.l3_router.l3_router_plugin.L3RouterPlugin\"}"},{"line_number":52,"context_line":"    if  ini_has_option $NEUTRON_CONF DEFAULT service_plugins ; then"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAASn%2F%2FUSc%3D","line":49,"in_reply_to":"AAAASn%2F%2FUns%3D","updated":"2013-09-06 13:45:32.000000000","message":"no trailing whitespace, if it pops up red in gerrit it\u0027s a -1.","commit_id":"78892fe34313f5e7e870b15873eb1953844229f7"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"c91acdb90d61564ca68e31046cdfaa89dc92f766","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        srv_plugins\u003d$(iniget $NEUTRON_CONF DEFAULT service_plugins)\",\"$Q_L3_PLUGIN_CLASS"},{"line_number":54,"context_line":"    else"},{"line_number":55,"context_line":"        srv_plugins\u003d$Q_L3_PLUGIN_CLASS"},{"line_number":56,"context_line":"    fi"},{"line_number":57,"context_line":"    iniset $NEUTRON_CONF DEFAULT service_plugins $srv_plugins"},{"line_number":58,"context_line":"}"},{"line_number":59,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAASn%2F%2FUng%3D","line":56,"updated":"2013-09-06 11:33:43.000000000","message":"I am not sure about this but I think devstack now has facilities for handling multistr options, so you might not need to do this concatenation","commit_id":"78892fe34313f5e7e870b15873eb1953844229f7"},{"author":{"_account_id":6685,"name":"Baodong (Robert) Li","email":"baoli@cisco.com","username":"baoli"},"change_message_id":"153f65034169c76f843c20e7fa324ef961f89039","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        srv_plugins\u003d$(iniget $NEUTRON_CONF DEFAULT service_plugins)\",\"$Q_L3_PLUGIN_CLASS"},{"line_number":54,"context_line":"    else"},{"line_number":55,"context_line":"        srv_plugins\u003d$Q_L3_PLUGIN_CLASS"},{"line_number":56,"context_line":"    fi"},{"line_number":57,"context_line":"    iniset $NEUTRON_CONF DEFAULT service_plugins $srv_plugins"},{"line_number":58,"context_line":"}"},{"line_number":59,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAASn%2F%2FOAs%3D","line":56,"in_reply_to":"AAAASn%2F%2FOQQ%3D","updated":"2013-09-09 19:33:08.000000000","message":"service_plugins in neutron/common/config.py is defined as a ListOpt rather than a MultiStrOpt. In the later case, you can use iniadd $NEUTRON_CONF DEFAULT service_plugins $Q_L3_PLUGIN_CLASS.","commit_id":"78892fe34313f5e7e870b15873eb1953844229f7"},{"author":{"_account_id":162,"name":"Bob Melander","email":"bob.melander@gmail.com","username":"bob-melander"},"change_message_id":"649f8fe32bf33d68a1a74ebf9212b7441fdf57b2","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        srv_plugins\u003d$(iniget $NEUTRON_CONF DEFAULT service_plugins)\",\"$Q_L3_PLUGIN_CLASS"},{"line_number":54,"context_line":"    else"},{"line_number":55,"context_line":"        srv_plugins\u003d$Q_L3_PLUGIN_CLASS"},{"line_number":56,"context_line":"    fi"},{"line_number":57,"context_line":"    iniset $NEUTRON_CONF DEFAULT service_plugins $srv_plugins"},{"line_number":58,"context_line":"}"},{"line_number":59,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAASn%2F%2FOQQ%3D","line":56,"in_reply_to":"AAAASn%2F%2FUng%3D","updated":"2013-09-09 18:36:23.000000000","message":"I\u0027m not sure either. At least not in the sense that I feel confident how to use them so I\u0027ll leave it as is for now.","commit_id":"78892fe34313f5e7e870b15873eb1953844229f7"}]}