openstack/neutron
Code Issues Proposed changes

Log INFO message when setting admin state up flag to False for OVS port

Browse Source 
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
This commit is contained in:
Hynek Mlnarik 2016-01-12 09:10:25 +01:00
parent ebb0dffd2f
commit 7ca04daa71
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py
View File

@ -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: