[node-patches] Change in ovirt-node[master]: tui: Request user input upon application exception

fabiand at fedoraproject.org fabiand at fedoraproject.org
Wed Feb 6 13:05:48 UTC 2013


Fabian Deutsch has uploaded a new change for review.

Change subject: tui: Request user input upon application exception
......................................................................

tui: Request user input upon application exception

The user needs to confirm that he has seen the exception after one was
raised in the UI.

Change-Id: I76c7373fb7f7b8a19abf4976be47be01e942fb25
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/tui/src/ovirt/node/app.py
M scripts/tui/src/ovirt/node/utils/console.py
2 files changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/03/11803/1

diff --git a/scripts/tui/src/ovirt/node/app.py b/scripts/tui/src/ovirt/node/app.py
index 597e088..7b67e5f 100644
--- a/scripts/tui/src/ovirt/node/app.py
+++ b/scripts/tui/src/ovirt/node/app.py
@@ -291,6 +291,8 @@
             self.ui.run()
         except Exception as e:
             self.logger.error("An error appeared in the UI: %s" % repr(e))
+            utils.console.writeln("Press any key to continue ...")
+            utils.console.wait_for_keypress()
 
     def quit(self):
         self.logger.info("Quitting")
diff --git a/scripts/tui/src/ovirt/node/utils/console.py b/scripts/tui/src/ovirt/node/utils/console.py
index 029ad96..53339de 100644
--- a/scripts/tui/src/ovirt/node/utils/console.py
+++ b/scripts/tui/src/ovirt/node/utils/console.py
@@ -35,7 +35,7 @@
 
 
 def wait_for_keypress():
-    sys.stdin.read(1)
+    return sys.stdin.read(1)
 
 
 class TransactionProgress(base.Base):


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I76c7373fb7f7b8a19abf4976be47be01e942fb25
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