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

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 08323bef69a388f97b711237909df0d9cb837a9f..1fc1e6e09f4777e7c68417454adc92dc8aaaaba5 100644 (file)
--- 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: