Tal Nisan has submitted this change and it was merged.
Change subject: backend: call getVgInfo after extending a storage domain
......................................................................
backend: call getVgInfo after extending a storage domain
When extending a storage domain, we get the new luns' information from
vdsm when calling ConnectAllHostsToLun (in validate()) and save it to
the db. This causes an inconsistency between the luns' real info from
the storage and their info as we save it in the db, for example a
different physical volume id.
Calling getVgInfo right after the extend itself gets us the new info,
and saving the new info fixes this bug.
Change-Id: I6a987f1f81f6ecad56a6854090fbba21131cbabd
Bug-Url:
https://bugzilla.redhat.com/1371960
Signed-off-by: Idan Shaby <ishaby(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/ExtendSANStorageDomainCommand.java
1 file changed, 26 insertions(+), 0 deletions(-)
Approvals:
Daniel Erez: Looks good to me, approved
Jenkins CI: Passed CI tests
Idan Shaby: Verified
Liron Aravot: Looks good to me, but someone else must approve
--
To view, visit
https://gerrit.ovirt.org/63084
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6a987f1f81f6ecad56a6854090fbba21131cbabd
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
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: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>