Change in ovirt-engine[master]: core:Print clusters when upgrading DC version(#844425)
Maor Lipchuk has submitted this change and it was merged.
Change subject: core:Print clusters when upgrading DC version(#844425)
......................................................................
core:Print clusters when upgrading DC version(#844425)
https://bugzilla.redhat.com/show_bug.cgi?id=844425
When we upgrade Data-Center we check if all cluster also has a validate
level (which is the same as the DC or higher), if not, the user gets a
message that he should upgrade the clusters, but there are no mention
which clusters it should be upgraded.
The proposed fix, adds the cluster names that should be upgraded to the
message, and add a test to cover it.
Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com>
Change-Id: Ia9c92d1c477354f7f5fa66895ecf2ab38c10bdf6
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStoragePoolCommand.java
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/UpdateStoragePoolCommandTest.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M backend/manager/modules/utils/src/test/resources/AppErrors.properties
M
frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M
frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M
frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
7 files changed, 57 insertions(+), 17 deletions(-)
Approvals:
Maor Lipchuk: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/7132
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia9c92d1c477354f7f5fa66895ecf2ab38c10bdf6
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server