Roy Golan has posted comments on this change.

Change subject: core: Add Windows 8/2012 to VM OS list
......................................................................


Patch Set 2: (1 inline comment)

also this commit isn't verifying sysprep for both. 
will open a preintegration ticket for that

....................................................
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmOsType.java
Line 23:     RHEL6(18, OsType.Linux, false),
Line 24:     RHEL6x64(19, OsType.Linux, true),
Line 25:     Windows8(20, OsType.Windows, false),
Line 26:     Windows8x64(21, OsType.Windows, true),
Line 27:     Windows2012(22, OsType.Windows, false),
Done
Line 28:     Windows2012x64(23, OsType.Windows, true);
Line 29: 
Line 30:     private final int intValue;
Line 31:     private final OsType osType;


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I21be8a69feddc89adcc671e307a96e4dd7897993
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: Yaniv Kaul <yk...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to