)]}'
{"manifests/agents/lbaas.pp":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"e024fb6defb783bffe37e583751c7386ac14ecb0","unresolved":false,"context_lines":[{"line_number":55,"context_line":"  $user_group             \u003d $::neutron::params::nobody_user_group,"},{"line_number":56,"context_line":"  $manage_haproxy_package \u003d true,"},{"line_number":57,"context_line":"  $purge_config           \u003d false,"},{"line_number":58,"context_line":"  $enable_v1              \u003d false,"},{"line_number":59,"context_line":"  $enable_v2              \u003d true,"},{"line_number":60,"context_line":") {"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa7ab95a_896308b1","line":58,"updated":"2016-08-26 15:37:55.000000000","message":"My understanding is that enable_v1 is no longer a valid option at all in \u003e\u003d newton, this should be deprecated with no effect and a deprecation notice.","commit_id":"6133db8428fe92c9da00e5cea0abe7739c1ea410"},{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"52cb2df7d20640fa13e8b6fec2943f4e9b533867","unresolved":false,"context_lines":[{"line_number":55,"context_line":"  $manage_service         \u003d true,"},{"line_number":56,"context_line":"  $debug                  \u003d $::os_service_default,"},{"line_number":57,"context_line":"  $interface_driver       \u003d \u0027neutron.agent.linux.interface.OVSInterfaceDriver\u0027,"},{"line_number":58,"context_line":"  $device_driver          \u003d \u0027neutron_lbaas.drivers.haproxy.namespace_driver.HaproxyNSDriver\u0027,"},{"line_number":59,"context_line":"  $user_group             \u003d $::neutron::params::nobody_user_group,"},{"line_number":60,"context_line":"  $manage_haproxy_package \u003d true,"},{"line_number":61,"context_line":"  $purge_config           \u003d false,"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"fa7ab95a_d7475b49","line":58,"updated":"2016-08-26 20:35:24.000000000","message":"I\u0027m wondering if this can be $::os_service_default? That being said, this is blocking beaker and I\u0027d be happy to look into it later on. beaker must live again!","commit_id":"7b60d759fcc81113984f55d5820f1cc44195cf58"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"3d10173a84a8c50a9db503bbf0b7bf7f1a547ac6","unresolved":false,"context_lines":[{"line_number":55,"context_line":"  $manage_service         \u003d true,"},{"line_number":56,"context_line":"  $debug                  \u003d $::os_service_default,"},{"line_number":57,"context_line":"  $interface_driver       \u003d \u0027neutron.agent.linux.interface.OVSInterfaceDriver\u0027,"},{"line_number":58,"context_line":"  $device_driver          \u003d \u0027neutron_lbaas.drivers.haproxy.namespace_driver.HaproxyNSDriver\u0027,"},{"line_number":59,"context_line":"  $user_group             \u003d $::neutron::params::nobody_user_group,"},{"line_number":60,"context_line":"  $manage_haproxy_package \u003d true,"},{"line_number":61,"context_line":"  $purge_config           \u003d false,"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"fa7ab95a_67007902","line":58,"in_reply_to":"fa7ab95a_d7475b49","updated":"2016-08-29 00:32:22.000000000","message":"I haven\u0027t found in Neutron where this is the default. I would like to make sure we switch puppet-neutron to the right driver by default now, maybe use os_service_default later.","commit_id":"7b60d759fcc81113984f55d5820f1cc44195cf58"}]}