Change in ovirt-engine[master]: core:Print clusters when upgrading DC version(#844425)

mlipchuk at redhat.com mlipchuk at redhat.com
Sun Aug 19 09:29:44 UTC 2012


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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list