From 843dcedcd95edac79dd84c146ba412864d7d66b3 Mon Sep 17 00:00:00 2001 From: Dirk Mueller Date: Tue, 16 Apr 2019 11:31:28 +0200 Subject: [PATCH] neutron: disable metering agent if no ceilometer Also cleanup code a bit to use the proper entry points rather than dotted path notations (which cause a warning) --- chef/cookbooks/neutron/recipes/common_config.rb | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/chef/cookbooks/neutron/recipes/common_config.rb b/chef/cookbooks/neutron/recipes/common_config.rb index 05c34c101b..6a728c0f5c 100644 --- a/chef/cookbooks/neutron/recipes/common_config.rb +++ b/chef/cookbooks/neutron/recipes/common_config.rb @@ -78,8 +78,9 @@ nova_config = Barclamp::Config.load("openstack", "nova") nova_insecure = CrowbarOpenStackHelper.insecure(nova_config) || keystone_settings["insecure"] -service_plugins = ["neutron.services.metering.metering_plugin.MeteringPlugin", - "neutron_fwaas.services.firewall.fwaas_plugin.FirewallPlugin"] +service_plugins = ["firewall"] +service_plugins.push("metering") if node.roles.include? "ceilometer-agent" + if neutron[:neutron][:use_lbaas] lbaas_plugin = if neutron[:neutron][:lbaasv2_driver] == "octavia" "neutron_lbaas.services.loadbalancer.proxy_plugin.LoadBalancerProxyPluginv2" @@ -91,11 +92,11 @@ end if neutron[:neutron][:networking_plugin] == "ml2" - service_plugins.unshift("neutron.services.l3_router.l3_router_plugin.L3RouterPlugin") + service_plugins.unshift("router") if neutron[:neutron][:ml2_mechanism_drivers].include?("linuxbridge") || neutron[:neutron][:ml2_mechanism_drivers].include?("openvswitch") - service_plugins.push("neutron.services.trunk.plugin.TrunkPlugin") + service_plugins.push("trunk") end if neutron[:neutron][:ml2_mechanism_drivers].include?("cisco_apic_ml2")