Change in ovirt-engine[master]: backend: call getVgInfo after extending a storage domain

[tnisan at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>