Allon Mureinik has submitted this change and it was merged.

Change subject: core, webadmin: Warn about incompatible upgrade
......................................................................


core, webadmin: Warn about incompatible upgrade

When upgrading a DC, a warning is issued if this upgrade would cause
upgrading the domains' storage format.

This patch improves that message by adding information that this upgrade
will cause the domains to be unusable with products of version XYZ.

The patch includes:
1. A method in VersionStorageFormatUtil to retrieve the earliest
   supported version of a given format.
2. A unit test for said method
3. Migrating the message from UIConstants to UIMessages, and adding a
   version parameter.
4. The call in DataCenterListModel for said message.

Change-Id: I3fa7345caad8603c03fb042dec2cebf6fdc03f44
Bug-Url: https://bugzilla.redhat.com/957939
Signed-off-by: Allon Mureinik <[email protected]>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VersionStorageFormatUtil.java
M 
backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/utils/VersionStorageFormatUtilTest.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java
M 
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
M 
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java
5 files changed, 44 insertions(+), 7 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3fa7345caad8603c03fb042dec2cebf6fdc03f44
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Cheryn Tan <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to