[node-patches] Change in ovirt-node[master]: installer: Don't reload, just re-parse

mburns at redhat.com mburns at redhat.com
Wed Mar 6 20:33:40 UTC 2013


Michael Burns has submitted this change and it was merged.

Change subject: installer: Don't reload, just re-parse
......................................................................


installer: Don't reload, just re-parse

Previously ovirtnode.ovirtfunctions were used by the main thread (running
the UI) and the installer thread (which is a different thread
running in the backgrouund to keep the ui respsonsive). This lead to
problems with th eaug variable in ovirtfunctions. To work around this
problem, the installer thread is now only explicitly using
parse_defaults to re-parse the defaults file, instead of reloading the
whole ovirtfunctions module.

Change-Id: I368b6cf56e0ede48b6a564ed4cea274a9198a858
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/tui/src/ovirt/node/installer/progress_page.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Michael Burns: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I368b6cf56e0ede48b6a564ed4cea274a9198a858
Gerrit-PatchSet: 4
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: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list