Liron Aravot has posted comments on this change.
Change subject: core: OvfDataUpdater - removal calls to updateVm/TemplateInSpm
......................................................................
Patch Set 16: (7 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmFromScratchCommand.java
Line 114:
Line 115:
VmHandler.LockVm(getParameters().getVmStaticData().getId());
Line 116: } else {
Line 117: // if no disks send update vm here
Line 118: getVmStaticDao().incrementDbGeneration(getVm().getId());
it being done in vm command endVmCommand().
Line 119: }
Line 120:
Line 121: return ret;
Line 122: }
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmCommand.java
Line 407:
OvfDataUpdater.getInstance().buildMetadataDictionaryForVm(getVm(),
metaDictionary);
Line 408: VmHandler.updateDisksFromDb(getVm());
Line 409: return
OvfDataUpdater.getInstance().executeUpdateVmInSpmCommand(getVm().getstorage_pool_id(),
Line 410: metaDictionary,
Line 411: getParameters().getStorageDomainId());
VmCommand.updateVmInSpm is removed. export command use the updateVmCommand in
order to perform vm export to a specific domain - therefore it needs to execute
this as part of the command.
Line 412: }
Line 413:
Line 414: @Override
Line 415: protected void endSuccessfully() {
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmTemplateCommand.java
Line 141:
OvfDataUpdater.getInstance().buildMetadataDictionaryForTemplate(getVmTemplate(),
metaDictionary);
Line 142:
OvfDataUpdater.getInstance().executeUpdateVmInSpmCommand(getVmTemplate().getstorage_pool_id().getValue(),
Line 143: metaDictionary,
Line 144: getParameters().getStorageDomainId());
Line 145: }
VmTemplateCommand.updateVmInSpm is removed. export command use the
updateVmTemplateCommand in order to perform template export to a specific
domain - therefore it needs to execute this as part of the command.
Line 146:
Line 147: @Override
Line 148: public Map<String, String> getJobMessageProperties() {
Line 149: if (jobProperties == null) {
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyTemplateCommand.java
Line 256:
log.warn("MoveOrCopyTemplateCommand::EndMoveOrCopyCommand: VmTemplate is null,
not performing full EndAction");
Line 257: }
Line 258: }
Line 259:
Line 260: protected void updateTemplateInSpm() {
Done
Line 261: VmTemplate vmt = getVmTemplate();
Line 262: getVmStaticDAO().incrementDbGeneration(vmt.getId());
Line 263: }
Line 264:
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveVmCommand.java
Line 176: setSucceeded(true);
Line 177:
Line 178: }
Line 179:
Line 180: protected void updateVmImSpm() {
Done
Line 181: getVmStaticDAO().incrementDbGeneration(getVm().getId());
Line 182: }
Line 183:
Line 184: @Override
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TryBackToAllSnapshotsOfVmCommand.java
Line 80: }
Line 81:
Line 82: @Override
Line 83: protected void endSuccessfully() {
Line 84: getVmStaticDAO().incrementDbGeneration(getVm().getId());
indeed.
Line 85: endActionOnDisks();
Line 86:
Line 87: if (getVm() != null) {
Line 88: VmHandler.unlockVm(getVm(), getCompensationContext());
....................................................
File backend/manager/modules/dal/src/test/resources/fixtures.xml
Line 896: <row>
Line 897: <value>77296e00-0cad-4e5a-9299-008a7b6f4360</value>
Line 898: <value>72b9e200-f48b-4687-83f2-62828f249a47</value>
Line 899: <value>1</value>
Line 900: </row>
Done
Line 901: </table>
Line 902:
Line 903: <table name="vm_static">
Line 904: <column>vm_guid</column>
--
To view, visit http://gerrit.ovirt.org/9340
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iedebceb9809dc0b11c0bbe8a2d4af63b0d848df1
Gerrit-PatchSet: 16
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
