Change in ovirt-engine[master]: core: ProcessOvfUpdateForStorageDomain - updateDate may be null

laravot at redhat.com laravot at redhat.com
Sun Jun 15 13:57:43 UTC 2014


Liron Ar has submitted this change and it was merged.

Change subject: core: ProcessOvfUpdateForStorageDomain - updateDate may be null
......................................................................


core: ProcessOvfUpdateForStorageDomain - updateDate may be null

When a new ovf store is created for domain, there's no last update date
as the ovf data was never updated to that domain.
To make it obvious that the ovf store was never updated, we can put null
value to indicate to it was never updated rather than excluding the key
from the description.

Change-Id: Id9743b2658d16039865cd3470bf822b5e5e137ff
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1109567
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, 1 insertion(+), 1 deletion(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Liron Ar: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id9743b2658d16039865cd3470bf822b5e5e137ff
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list