)]}'
{"setup.cfg":[{"author":{"_account_id":13702,"name":"yushiro","username":"yushiro","inactive":true},"change_message_id":"fc6add28fd4db6136cc0e2d87850af978e29fdcf","unresolved":false,"context_lines":[{"line_number":117,"context_line":"neutron.agent.l2.extensions \u003d"},{"line_number":118,"context_line":"    qos \u003d neutron.agent.l2.extensions.qos:QosAgentExtension"},{"line_number":119,"context_line":"    fdb \u003d neutron.agent.l2.extensions.fdb_population:FdbPopulationAgentExtension"},{"line_number":120,"context_line":"neutron.qos.agent_drivers \u003d"},{"line_number":121,"context_line":"    ovs \u003d neutron.plugins.ml2.drivers.openvswitch.agent.extension_drivers.qos_driver:QosOVSAgentDriver"},{"line_number":122,"context_line":"    sriov \u003d neutron.plugins.ml2.drivers.mech_sriov.agent.extension_drivers.qos_driver:QosSRIOVAgentDriver"},{"line_number":123,"context_line":"    linuxbridge \u003d neutron.plugins.ml2.drivers.linuxbridge.agent.extension_drivers.qos_driver:QosLinuxbridgeAgentDriver"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"fa7ab95a_6c64e021","line":120,"range":{"start_line":120,"start_character":0,"end_line":120,"end_character":25},"updated":"2016-08-29 00:08:42.000000000","message":"I think following definition is also necessary.\n\n* fwaasv2 \u003d neutron_fwaas.services.firewall.agents.v2.l2.extensions.fwaas:FWaaSV2AgentExtension\n\nCurrent definition depends on following directory constructure.\n\n* https://review.openstack.org/#/c/323971/","commit_id":"81ea826d635f29254a219f1920d636c3bf6cef0e"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"4bcdf6307da6e29e0e5196f42c04952a8a47358f","unresolved":false,"context_lines":[{"line_number":117,"context_line":"neutron.agent.l2.extensions \u003d"},{"line_number":118,"context_line":"    qos \u003d neutron.agent.l2.extensions.qos:QosAgentExtension"},{"line_number":119,"context_line":"    fdb \u003d neutron.agent.l2.extensions.fdb_population:FdbPopulationAgentExtension"},{"line_number":120,"context_line":"neutron.qos.agent_drivers \u003d"},{"line_number":121,"context_line":"    ovs \u003d neutron.plugins.ml2.drivers.openvswitch.agent.extension_drivers.qos_driver:QosOVSAgentDriver"},{"line_number":122,"context_line":"    sriov \u003d neutron.plugins.ml2.drivers.mech_sriov.agent.extension_drivers.qos_driver:QosSRIOVAgentDriver"},{"line_number":123,"context_line":"    linuxbridge \u003d neutron.plugins.ml2.drivers.linuxbridge.agent.extension_drivers.qos_driver:QosLinuxbridgeAgentDriver"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"fa7ab95a_e7a489a7","line":120,"range":{"start_line":120,"start_character":0,"end_line":120,"end_character":25},"in_reply_to":"fa7ab95a_6c64e021","updated":"2016-08-29 01:32:39.000000000","message":"Thanks, Yushiro!  I think that should be a separate change.  That way this change can merge for basic L3 functionality, but if L2 takes a little longer and doesn\u0027t make the release cutoff then we still have a functioning L3 FWaaS.\n\nI have spun a separate change for that, with the requisite dependencies: https://review.openstack.org/361786","commit_id":"81ea826d635f29254a219f1920d636c3bf6cef0e"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"2111c487cb06668c6fc968f31965250ff0b86b86","unresolved":false,"context_lines":[{"line_number":69,"context_line":"neutron.service_plugins \u003d"},{"line_number":70,"context_line":"    dummy \u003d neutron.tests.unit.dummy_plugin:DummyServicePlugin"},{"line_number":71,"context_line":"    router \u003d neutron.services.l3_router.l3_router_plugin:L3RouterPlugin"},{"line_number":72,"context_line":"    firewall \u003d neutron_fwaas.services.firewall.fwaas_plugin:FirewallPlugin"},{"line_number":73,"context_line":"    firewall_v2 \u003d neutron_fwaas.services.firewall.fwaas_plugin_v2:FirewallPluginV2"},{"line_number":74,"context_line":"    lbaas \u003d neutron_lbaas.services.loadbalancer.plugin:LoadBalancerPlugin"},{"line_number":75,"context_line":"    vpnaas \u003d neutron_vpnaas.services.vpn.plugin:VPNDriverPlugin"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"fa7ab95a_911bcac9","line":72,"updated":"2016-08-29 17:45:47.000000000","message":"One might argue that firewall, lbaas and vpnaas are a reminiscence, and should be removed too. These entry points should belong to the project that controls them","commit_id":"a722e8ee50d684e74103fe9df9bd41d4c5b8a0d9"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"907a10fbde6e39f836d5587d282d871d845386c6","unresolved":false,"context_lines":[{"line_number":69,"context_line":"neutron.service_plugins \u003d"},{"line_number":70,"context_line":"    dummy \u003d neutron.tests.unit.dummy_plugin:DummyServicePlugin"},{"line_number":71,"context_line":"    router \u003d neutron.services.l3_router.l3_router_plugin:L3RouterPlugin"},{"line_number":72,"context_line":"    firewall \u003d neutron_fwaas.services.firewall.fwaas_plugin:FirewallPlugin"},{"line_number":73,"context_line":"    firewall_v2 \u003d neutron_fwaas.services.firewall.fwaas_plugin_v2:FirewallPluginV2"},{"line_number":74,"context_line":"    lbaas \u003d neutron_lbaas.services.loadbalancer.plugin:LoadBalancerPlugin"},{"line_number":75,"context_line":"    vpnaas \u003d neutron_vpnaas.services.vpn.plugin:VPNDriverPlugin"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"fa7ab95a_5724d2c3","line":72,"in_reply_to":"fa7ab95a_911bcac9","updated":"2016-08-29 18:24:21.000000000","message":"Totally agreed.","commit_id":"a722e8ee50d684e74103fe9df9bd41d4c5b8a0d9"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"907a10fbde6e39f836d5587d282d871d845386c6","unresolved":false,"context_lines":[{"line_number":75,"context_line":"    vpnaas \u003d neutron_vpnaas.services.vpn.plugin:VPNDriverPlugin"},{"line_number":76,"context_line":"    metering \u003d neutron.services.metering.metering_plugin:MeteringPlugin"},{"line_number":77,"context_line":"    neutron.services.firewall.fwaas_plugin.FirewallPlugin \u003d neutron_fwaas.services.firewall.fwaas_plugin:FirewallPlugin"},{"line_number":78,"context_line":"    neutron.services.firewall.fwaas_plugin_v2.FirewallPlugin_v2 \u003d neutron_fwaas.services.firewall.fwaas_plugin_v2:FirewallPluginV2"},{"line_number":79,"context_line":"    neutron.services.loadbalancer.plugin.LoadBalancerPlugin \u003d neutron_lbaas.services.loadbalancer.plugin:LoadBalancerPlugin"},{"line_number":80,"context_line":"    neutron.services.vpn.plugin.VPNDriverPlugin \u003d neutron_vpnaas.services.vpn.plugin:VPNDriverPlugin"},{"line_number":81,"context_line":"    qos \u003d neutron.services.qos.qos_plugin:QoSPlugin"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"fa7ab95a_4215ee3d","line":78,"updated":"2016-08-29 18:24:21.000000000","message":"This should not exist at all. The fact that we exposed others is for compatibility matters when we moved code from neutron into neutron-fwaas repo.","commit_id":"a722e8ee50d684e74103fe9df9bd41d4c5b8a0d9"}]}