Change in ovirt-engine[master]: core: synchronize lun device size on refreshLunsInfo

derez at redhat.com derez at redhat.com
Sun Jan 5 10:24:26 UTC 2014


Daniel Erez has submitted this change and it was merged.

Change subject: core: synchronize lun device size on refreshLunsInfo
......................................................................


core: synchronize lun device size on refreshLunsInfo

SyncLunsInfoForIscsiStorageDomainCommand:
When detecting a mismatch in LUN device size,
store the updated size data in DB.

* Added 'updateLUNsDeviceSize' method to LunDAO.
* Updated storages_san_sp.sql accordingly.

Note: the command is invoked when activating iSCSI SD.

Change-Id: I618880c7b49c226e39f0a0830f1d4f9b71a5f304
Bug-Url: https://bugzilla.redhat.com/961532
Signed-off-by: Daniel Erez <derez at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/SyncLunsInfoForIscsiStorageDomainCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/SyncLunsInfoForIscsiStorageDomainCommandTest.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/LunDAO.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/LunDAODbFacadeImpl.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/LunDAOTest.java
M packaging/dbscripts/storages_san_sp.sql
6 files changed, 68 insertions(+), 1 deletion(-)

Approvals:
  Daniel Erez: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/22887
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I618880c7b49c226e39f0a0830f1d4f9b71a5f304
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list