Change in ovirt-engine[master]: core: Add IDs to ImageStatus

amureini at redhat.com amureini at redhat.com
Fri Apr 26 13:49:41 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Add IDs to ImageStatus
......................................................................


core: Add IDs to ImageStatus

Added IDs to the ImageStauts enum, as per the FIXME comment at the
head of the class, in order to decouple the enum values from their
ordinals.

This is done in preparation to remove the unused INVALID status, so
that ILLEGAL will retain its value when INVALID is removed.

Change-Id: I364f95ea384e1742842066043780183e3d4244b6
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ImageStatus.java
1 file changed, 23 insertions(+), 9 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I364f95ea384e1742842066043780183e3d4244b6
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas 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: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>



More information about the Engine-commits mailing list