From: armando-migliaccio <armamig@gmail.com>
Date: Thu, 13 Mar 2014 20:16:07 +0000 (-0700)
Subject: NSX: lower the severity of messages about VIF's on external networks
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=04a1834dc42fdb6fe19d5115ec6705ebb897aa03;p=openstack-build%2Fneutron-build.git

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

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

diff --git a/neutron/plugins/vmware/plugins/base.py b/neutron/plugins/vmware/plugins/base.py
index e378d21c7..235c4b703 100644
--- a/neutron/plugins/vmware/plugins/base.py
+++ b/neutron/plugins/vmware/plugins/base.py
@@ -433,9 +433,9 @@ class NsxPluginV2(addr_pair_db.AllowedAddressPairsMixin,
         # However, in order to not break unit tests, we need to still create
         # the DB object and return success
         if self._network_is_external(context, port_data['network_id']):
-            LOG.error(_("NSX plugin does not support regular VIF ports on "
-                        "external networks. Port %s will be down."),
-                      port_data['network_id'])
+            LOG.info(_("NSX plugin does not support regular VIF ports on "
+                       "external networks. Port %s will be down."),
+                     port_data['network_id'])
             # No need to actually update the DB state - the default is down
             return port_data
         lport = None
@@ -485,9 +485,9 @@ class NsxPluginV2(addr_pair_db.AllowedAddressPairsMixin,
         # does not make sense. However we cannot raise as this would break
         # unit tests.
         if self._network_is_external(context, port_data['network_id']):
-            LOG.error(_("NSX plugin does not support regular VIF ports on "
-                        "external networks. Port %s will be down."),
-                      port_data['network_id'])
+            LOG.info(_("NSX plugin does not support regular VIF ports on "
+                       "external networks. Port %s will be down."),
+                     port_data['network_id'])
             return
         nsx_switch_id, nsx_port_id = nsx_utils.get_nsx_switch_and_port_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
         # the DB object and return success
         if self._network_is_external(context, port_data['network_id']):
-            LOG.error(_("NSX plugin does not support regular VIF ports on "
-                        "external networks. Port %s will be down."),
-                      port_data['network_id'])
+            LOG.info(_("NSX plugin does not support regular VIF ports on "
+                       "external networks. Port %s will be down."),
+                     port_data['network_id'])
             # No need to actually update the DB state - the default is down
             return port_data
         lport = None