From Tal Nisan <tnisan(a)redhat.com>:
Tal Nisan has submitted this change and it was merged.
Change subject: backend: sync all luns when upgrading a data center
......................................................................
backend: sync all luns when upgrading a data center
Up until now, when upgrading a data center from version <= 4.0 to
version >= 4.1, all the storage domains' luns were synchronized in order
to update their discard related new data from vdsm.
However, direct luns were never synchronized and updated, so their
discard related data was not automatically updated.
Now, also direct luns that are attached to a VM in the data center are
synchronized.
Also, since UpdateStoragePoolCommand now uses SyncAllUsedLunsCommand and
doesn't call directly to SyncLunsInfoForBlockStorageDomainCommand for
all n active block storage domain, synchronizing all their luns should
take ~ n times faster than it did before.
Change-Id: Id2f191a5f3580d7c73764b9dcd80fe8b4ed43b1e
Bug-Url:https://bugzilla.redhat.com/1455547
Bug-Url:https://bugzilla.redhat.com/1455548
Signed-off-by: Idan Shaby <ishaby(a)redhat.com>
---
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
2 files changed, 14 insertions(+), 106 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Daniel Erez: Looks good to me, approved
Idan Shaby: Verified
--
To view, visit
https://gerrit.ovirt.org/78061
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id2f191a5f3580d7c73764b9dcd80fe8b4ed43b1e
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>