)]}'
{"manifests/neutron.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"31bb732ce6d26232b90343e3b5c175fbffb537c4","unresolved":false,"context_lines":[{"line_number":115,"context_line":"    rpc_workers         \u003d\u003e 2,"},{"line_number":116,"context_line":"    auth_uri            \u003d\u003e $::openstack_integration::config::keystone_auth_uri,"},{"line_number":117,"context_line":"    auth_url            \u003d\u003e $::openstack_integration::config::keystone_admin_uri,"},{"line_number":118,"context_line":"    service_providers   \u003d\u003e [\u0027LOADBALANCER:Haproxy:neutron_lbaas.services.loadbalancer.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver:default\u0027,"},{"line_number":119,"context_line":"                            \u0027LOADBALANCERV2:Haproxy:neutron_lbaas.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver\u0027],"},{"line_number":120,"context_line":"  }"},{"line_number":121,"context_line":"  class { \u0027::neutron::services::lbaas\u0027: }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bab6814e_584f11bd","line":118,"updated":"2016-05-25 12:06:22.000000000","message":"is it ok to load both?","commit_id":"1c6350ac5bd3093eea9e943ba982cdf65919adf6"},{"author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"change_message_id":"4339b419c2a17b1c5f85e7a4f3a00f52a85641a4","unresolved":false,"context_lines":[{"line_number":115,"context_line":"    rpc_workers         \u003d\u003e 2,"},{"line_number":116,"context_line":"    auth_uri            \u003d\u003e $::openstack_integration::config::keystone_auth_uri,"},{"line_number":117,"context_line":"    auth_url            \u003d\u003e $::openstack_integration::config::keystone_admin_uri,"},{"line_number":118,"context_line":"    service_providers   \u003d\u003e [\u0027LOADBALANCER:Haproxy:neutron_lbaas.services.loadbalancer.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver:default\u0027,"},{"line_number":119,"context_line":"                            \u0027LOADBALANCERV2:Haproxy:neutron_lbaas.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver\u0027],"},{"line_number":120,"context_line":"  }"},{"line_number":121,"context_line":"  class { \u0027::neutron::services::lbaas\u0027: }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bab6814e_5859d159","line":118,"in_reply_to":"bab6814e_584f11bd","updated":"2016-05-25 12:12:19.000000000","message":"It\u0027s a requirement to have at the very minimum the v1 service provider or neutron api server will continually restart complaining that it does not have the v1 service provider defined. The v2 service provider is not required if we\u0027re only testing v1.","commit_id":"1c6350ac5bd3093eea9e943ba982cdf65919adf6"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"31bb732ce6d26232b90343e3b5c175fbffb537c4","unresolved":false,"context_lines":[{"line_number":137,"context_line":"  }"},{"line_number":138,"context_line":"  class { \u0027::neutron::agents::lbaas\u0027:"},{"line_number":139,"context_line":"    interface_driver \u003d\u003e $driver,"},{"line_number":140,"context_line":"    $enable_v1       \u003d\u003e $lbaasv1,"},{"line_number":141,"context_line":"    $enable_v2       \u003d\u003e $lbaasv2,"},{"line_number":142,"context_line":"    debug            \u003d\u003e true,"},{"line_number":143,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bab6814e_98ee9989","line":140,"updated":"2016-05-25 12:06:22.000000000","message":"or:\n\n $enable_v1 \u003d\u003e ! $lbaasv2,\n $enable_v2 \u003d\u003e $lbaasv2,","commit_id":"1c6350ac5bd3093eea9e943ba982cdf65919adf6"},{"author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"change_message_id":"4339b419c2a17b1c5f85e7a4f3a00f52a85641a4","unresolved":false,"context_lines":[{"line_number":137,"context_line":"  }"},{"line_number":138,"context_line":"  class { \u0027::neutron::agents::lbaas\u0027:"},{"line_number":139,"context_line":"    interface_driver \u003d\u003e $driver,"},{"line_number":140,"context_line":"    $enable_v1       \u003d\u003e $lbaasv1,"},{"line_number":141,"context_line":"    $enable_v2       \u003d\u003e $lbaasv2,"},{"line_number":142,"context_line":"    debug            \u003d\u003e true,"},{"line_number":143,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bab6814e_b371feaa","line":140,"in_reply_to":"bab6814e_98ee9989","updated":"2016-05-25 12:12:19.000000000","message":"I\u0027ll switch it to this once we confirm that having both service providers does not cause issues. If we need to remove the v2 service provider, we\u0027ll need the conditional for that.","commit_id":"1c6350ac5bd3093eea9e943ba982cdf65919adf6"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"277814f766099e3bad4a85f83ad31a8898e7c214","unresolved":false,"context_lines":[{"line_number":6,"context_line":"#   Defaults to \u0027ml2_ovs\u0027."},{"line_number":7,"context_line":"#"},{"line_number":8,"context_line":"# [*lbaasv2*]"},{"line_number":9,"context_line":"#  (optional) Configure lbaas v2 instead of v1"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"class openstack_integration::neutron ("},{"line_number":12,"context_line":"  $driver  \u003d \u0027openvswitch\u0027,"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"9abb7d3a_0418da83","line":9,"updated":"2016-05-27 12:38:29.000000000","message":"Defaults to false","commit_id":"59f2a46bee86abc72bc9f6cc1ea3db04e24670ab"},{"author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"change_message_id":"caafaac739659c14f10e4f1bb4cb04af3924c8ef","unresolved":false,"context_lines":[{"line_number":6,"context_line":"#   Defaults to \u0027ml2_ovs\u0027."},{"line_number":7,"context_line":"#"},{"line_number":8,"context_line":"# [*lbaasv2*]"},{"line_number":9,"context_line":"#  (optional) Configure lbaas v2 instead of v1"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"class openstack_integration::neutron ("},{"line_number":12,"context_line":"  $driver  \u003d \u0027openvswitch\u0027,"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"9abb7d3a_4a0f89e1","line":9,"in_reply_to":"9abb7d3a_0418da83","updated":"2016-05-27 13:02:04.000000000","message":"Done","commit_id":"59f2a46bee86abc72bc9f6cc1ea3db04e24670ab"}]}