[node-patches] Change in ovirt-node[master]: ui: Fix kdump page type initialization

fabiand at fedoraproject.org fabiand at fedoraproject.org
Tue Jul 9 15:34:08 UTC 2013


Fabian Deutsch has submitted this change and it was merged.

Change subject: ui: Fix kdump page type initialization
......................................................................


ui: Fix kdump page type initialization

Previously the available kdump target types were initialized during
class creation, this broke doctests and it wouldn't work at runtime,
because the network status can change between page views, and the class
is only instantiated once at application start.
Now the network status is checked on each page view.

Change-Id: I331e27b03e4df5817a318e571a80ad504a0dff4a
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/setup/core/kdump_page.py
1 file changed, 20 insertions(+), 13 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I331e27b03e4df5817a318e571a80ad504a0dff4a
Gerrit-PatchSet: 3
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: oVirt Jenkins CI Server



More information about the node-patches mailing list