x/vmware-nsx
Code Issues Proposed changes

NSX: lower the severity of messages about VIF's on external networks

Browse Source 
Set the message to INFO as there is no error to recover from or
something bad about to happen. The only place where this occurs
is during the setup of probe interfaces in DevStack.

Change-Id: I2310af00676e508646783ce769a8830ab0b9308f
This commit is contained in:
armando-migliaccio 2014-03-13 13:16:07 -07:00
parent 736918f08b
commit d1b7b34f16
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
neutron/plugins/vmware/plugins/base.py
View File

@ -433,9 +433,9 @@ class NsxPluginV2(addr_pair_db.AllowedAddressPairsMixin,
# However, in order to not break unit tests, we need to still create # However, in order to not break unit tests, we need to still create
# the DB object and return success # the DB object and return success
if self._network_is_external(context, port_data['network_id']): if self._network_is_external(context, port_data['network_id']):
LOG.error(_("NSX plugin does not support regular VIF ports on " LOG.info(_("NSX plugin does not support regular VIF ports on "
"external networks. Port %s will be down."), "external networks. Port %s will be down."),
port_data['network_id']) port_data['network_id'])
# No need to actually update the DB state - the default is down # No need to actually update the DB state - the default is down
return port_data return port_data
lport = None lport = None
@ -485,9 +485,9 @@ class NsxPluginV2(addr_pair_db.AllowedAddressPairsMixin,
# does not make sense. However we cannot raise as this would break # does not make sense. However we cannot raise as this would break
# unit tests. # unit tests.
if self._network_is_external(context, port_data['network_id']): if self._network_is_external(context, port_data['network_id']):
LOG.error(_("NSX plugin does not support regular VIF ports on " LOG.info(_("NSX plugin does not support regular VIF ports on "
"external networks. Port %s will be down."), "external networks. Port %s will be down."),
port_data['network_id']) port_data['network_id'])
return return
nsx_switch_id, nsx_port_id = nsx_utils.get_nsx_switch_and_port_id( nsx_switch_id, nsx_port_id = nsx_utils.get_nsx_switch_and_port_id(
context.session, self.cluster, port_data['id']) context.session, self.cluster, port_data['id'])
@ -683,9 +683,9 @@ class NsxPluginV2(addr_pair_db.AllowedAddressPairsMixin,
# However, in order to not break unit tests, we need to still create # However, in order to not break unit tests, we need to still create
# the DB object and return success # the DB object and return success
if self._network_is_external(context, port_data['network_id']): if self._network_is_external(context, port_data['network_id']):
LOG.error(_("NSX plugin does not support regular VIF ports on " LOG.info(_("NSX plugin does not support regular VIF ports on "
"external networks. Port %s will be down."), "external networks. Port %s will be down."),
port_data['network_id']) port_data['network_id'])
# No need to actually update the DB state - the default is down # No need to actually update the DB state - the default is down
return port_data return port_data
lport = None lport = None