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