Change in ovirt-engine[master]: webadmin: block reinitialize on uninitalized pool

laravot at redhat.com laravot at redhat.com
Wed Feb 26 08:47:30 UTC 2014


Liron Ar has submitted this change and it was merged.

Change subject: webadmin: block reinitialize on uninitalized pool
......................................................................


webadmin: block reinitialize on uninitalized pool

When a pool is uninitalized there's no need to allow reinitializing
it, any wanted domain can be attached to the pool.
Allowing to perform reinitalize in that status caused to NPE in the
engine which currently attempts to load the current master domain of the
pool.
As there's no meaning for recovery/reinitialize in that scenario, it can
be blocked - as this issue was never encountered, we can add an engine
verification if and when it will be necessary.

Change-Id: I480b1f99ae881d43d34342ccb3bb248e60f556ce
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1069581
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Daniel Erez: Looks good to me, approved
  Liron Ar: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I480b1f99ae881d43d34342ccb3bb248e60f556ce
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list