[node-patches] Change in ovirt-node[master]: ui: Add UIThreadConnection

jboggs at redhat.com jboggs at redhat.com
Mon Apr 22 21:51:02 UTC 2013


Joey Boggs has submitted this change and it was merged.

Change subject: ui: Add UIThreadConnection
......................................................................


ui: Add UIThreadConnection

This class can be used by other threads to interact/modify the UI
through the UI thread's mainloop.
If the interaction with the UI is not done through such a mechanism,
race conditions will (sooner or later) occur and can lead to all sorts
of UI errors (e.g. inconsitencies, incorrect rendering, ...).

This new class is directly used in the installer (thread).

rhbz#920049

Change-Id: I099a1fbb9106d5501c1f78c6274730b4801ccaf3
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/tui/src/ovirt/node/installer/progress_page.py
M scripts/tui/src/ovirt/node/ui/__init__.py
M scripts/tui/src/ovirt/node/ui/urwid_builder.py
M scripts/tui/src/ovirt/node/ui/widgets.py
M scripts/tui/src/ovirt/node/utils/storage.py
5 files changed, 98 insertions(+), 20 deletions(-)

Approvals:
  Joey Boggs: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I099a1fbb9106d5501c1f78c6274730b4801ccaf3
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Joey Boggs <jboggs at redhat.com>



More information about the node-patches mailing list