Edward Haas has uploaded a new change for review.

Change subject: net: Log a setup networks transaction failure when using ovs 
switch
......................................................................

net: Log a setup networks transaction failure when using ovs switch

When a failure occurs during the ovs setup process, an exception is
raised.
Log this failure in order to ease debugging.

Change-Id: I157a93a1cb800d47587eec5fef5a7c3eb08df58d
Signed-off-by: Edward Haas <edwa...@redhat.com>
---
M lib/vdsm/network/netconfpersistence.py
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/42/64242/1

diff --git a/lib/vdsm/network/netconfpersistence.py 
b/lib/vdsm/network/netconfpersistence.py
index 37b9e89..9763cc3 100644
--- a/lib/vdsm/network/netconfpersistence.py
+++ b/lib/vdsm/network/netconfpersistence.py
@@ -245,6 +245,10 @@
         else:
             config_diff = self.base_config.diffFrom(self.config)
             if config_diff:
+                logging.warning(
+                    'Failed setup transaction,'
+                    'reverting to last known good network.',
+                    exc_info=(ex_type, ex_value, ex_traceback))
                 raise ne.RollbackIncomplete(config_diff, ex_type, ex_value)
 
 


-- 
To view, visit https://gerrit.ovirt.org/64242
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I157a93a1cb800d47587eec5fef5a7c3eb08df58d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwa...@redhat.com>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to