From 7ca04daa71910b578303845a363883fa036358f7 Mon Sep 17 00:00:00 2001 From: Hynek Mlnarik Date: Tue, 12 Jan 2016 09:10:25 +0100 Subject: [PATCH] Log INFO message when setting admin state up flag to False for OVS port Disabling admin state can cause confusion in OVS agent logs as to why VLAN 4095 is set to a particular port. This commit adds an extra INFO log message when admin state is disabled. There is no log emitted upon enabling admin state as there is already a detailed log entry logged in treat_devices_added_or_updated() function. Change-Id: I1301db8fea75af211f66a1914d1f0b706a6d5b36 Closes-Bug: 1463891 --- .../plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py b/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py index 08323bef6..1fc1e6e09 100644 --- a/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py +++ b/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py @@ -1333,6 +1333,9 @@ class OVSNeutronAgent(sg_rpc.SecurityGroupAgentRpcCallbackMixin, physical_network, segmentation_id, fixed_ips, device_owner, ovs_restarted) else: + LOG.info(_LI("VIF port: %s admin state up disabled, " + "putting on the dead VLAN"), vif_port.vif_id) + self.port_dead(vif_port) port_needs_binding = False else: -- 2.45.2