Change in ovirt-engine[ovirt-engine-4.1.1.z]: backend: sync luns on live dc upgrade

Code Review gerrit at ovirt.org
Wed Mar 1 17:01:31 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: backend: sync luns on live dc upgrade
......................................................................


backend: sync luns on live dc upgrade

When running a live dc upgrade, active storage domains don't get updated
with the latest luns information from vdsm.
Since on new dc versions, it is possible that new information about the
storge domains' luns can be returned from vdsm, this patch synchronizes
these luns as a part of the upgrade process.

Change-Id: I81f9b247c73cffa688e7c2776b0da6fc937710ea
Bug-Url: https://bugzilla.redhat.com/1419327
Signed-off-by: Idan Shaby <ishaby at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/SyncLunsInfoForBlockStorageDomainCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/pool/UpdateStoragePoolCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/pool/UpdateStoragePoolCommandTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
5 files changed, 127 insertions(+), 2 deletions(-)

Approvals:
  Tal Nisan: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Idan Shaby: Verified



-- 
To view, visit https://gerrit.ovirt.org/73338
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I81f9b247c73cffa688e7c2776b0da6fc937710ea
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1.1.z
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>


More information about the Engine-commits mailing list