[node-patches] Change in ovirt-node[master]: setup: Remove kdump message after displaying it

fabiand at fedoraproject.org fabiand at fedoraproject.org
Mon Jan 28 09:25:07 UTC 2013


Fabian Deutsch has uploaded a new change for review.

Change subject: setup: Remove kdump message after displaying it
......................................................................

setup: Remove kdump message after displaying it

Previously every error message which appeared on the kdump page wasn't
discarded after it was shown, this way the error message was shown even
if it did not apply. E.g. when kdump was configured with NFS and th
epath was missing an appropriate message was shown, if in a second try
the restore option was used the NFS related message was still shown.
This patch discards kdump error messages after they were shown.

rhbz#903989

Change-Id: I383d3ac0b0f6b68633a2a7c67a2dc380989e472f
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/ovirt-config-setup.py
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/50/11450/1

diff --git a/scripts/ovirt-config-setup.py b/scripts/ovirt-config-setup.py
index c6dd507..6a70a95 100755
--- a/scripts/ovirt-config-setup.py
+++ b/scripts/ovirt-config-setup.py
@@ -1872,6 +1872,7 @@
                 self._create_warn_screen()
                 ButtonChoiceWindow(self.screen, "Configuration Check",
                                    self.INVALID_KDUMP_MSG, buttons=["Ok"])
+                self.INVALID_KDUMP_MSG = ""
                 return False
             if os.path.exists("/etc/kdump.conf"):
                 system("cp /etc/kdump.conf /etc/kdump.conf.old")


--
To view, visit http://gerrit.ovirt.org/11450
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I383d3ac0b0f6b68633a2a7c67a2dc380989e472f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>



More information about the node-patches mailing list