]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
NSX: lower the severity of messages about VIF's on external networks
authorarmando-migliaccio <armamig@gmail.com>
Thu, 13 Mar 2014 20:16:07 +0000 (13:16 -0700)
committerarmando-migliaccio <armamig@gmail.com>
Thu, 13 Mar 2014 20:16:53 +0000 (13:16 -0700)
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

neutron/plugins/vmware/plugins/base.py

index e378d21c76d3c594ceba94b2bf33f158b323b723..235c4b703365eef14249a9acd032c22248ebe6ae 100644 (file)
@@ -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