Change in ovirt-engine[master]: core: last ovf store update

laravot at redhat.com laravot at redhat.com
Sun Mar 23 14:45:20 UTC 2014


Liron Ar has submitted this change and it was merged.

Change subject: core: last ovf store update
......................................................................


core: last ovf store update

In a case in which the last ovf store was previously updated, we should
attempt to update it only if update to any of the previous domains
was succesfull.
Otherwise, we can attempt to update it immediately as it doesn't contain
any ovf data so there's no point to consider the other updates results.

Change-Id: I21491330681c333faf9e97b3d348d2af8c2b2fb5
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java
1 file changed, 3 insertions(+), 1 deletion(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I21491330681c333faf9e97b3d348d2af8c2b2fb5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list