Roy Golan has uploaded a new change for review.

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

core: Add Windows 8/2012 to VM OS list

Change-Id: I21be8a69feddc89adcc671e307a96e4dd7897993
Signed-off-by: Roy Golan <rgo...@redhat.com>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmOsType.java
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M 
backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/OsType.java
M 
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/DataProvider.java
M 
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java
M 
frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties
8 files changed, 98 insertions(+), 14 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/30/9530/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmOsType.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmOsType.java
index 67d0dde..fcd4edd 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmOsType.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmOsType.java
@@ -21,7 +21,11 @@
     Windows2008x64(16, OsType.Windows, true),
     Windows2008R2x64(17, OsType.Windows, true),
     RHEL6(18, OsType.Linux, false),
-    RHEL6x64(19, OsType.Linux, true);
+    RHEL6x64(19, OsType.Linux, true),
+    Windows8(20, OsType.Windows, false),
+    Windows8x64(21, OsType.Windows, true),
+    Windows2012(22, OsType.Windows, false),
+    Windows2012x64(23, OsType.Windows, true);
 
     private final int intValue;
     private final OsType osType;
diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
index 6a46b74..b762537 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
@@ -1559,6 +1559,38 @@
     @DefaultValueAttribute("300")
     GlusterRefreshRateHeavy(396),
 
+    @TypeConverterAttribute(String.class)
+    @DefaultValueAttribute("")
+    SysPrepWindows8Path(397),
+
+    @TypeConverterAttribute(String.class)
+    @DefaultValueAttribute("")
+    ProductKeyWindows8(398),
+
+    @TypeConverterAttribute(String.class)
+    @DefaultValueAttribute("")
+    SysPrepWindows8x64Path(399),
+
+    @TypeConverterAttribute(String.class)
+    @DefaultValueAttribute("")
+    ProductKeyWindows8x64(400),
+
+    @TypeConverterAttribute(String.class)
+    @DefaultValueAttribute("")
+    SysPrepWindows2012Path(401),
+
+    @TypeConverterAttribute(String.class)
+    @DefaultValueAttribute("")
+    ProductKeyWindows2012(402),
+
+    @TypeConverterAttribute(String.class)
+    @DefaultValueAttribute("")
+    SysPrepWindows2012x64Path(403),
+
+    @TypeConverterAttribute(String.class)
+    @DefaultValueAttribute("")
+    ProductKeyWindows2012x64(404),
+
     Invalid(65535);
 
     private int intValue;
diff --git 
a/backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/OsType.java
 
b/backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/OsType.java
index 7adfdc3..6542326 100644
--- 
a/backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/OsType.java
+++ 
b/backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/OsType.java
@@ -35,7 +35,11 @@
     WINDOWS_2008X64,
     WINDOWS_2008R2,
     RHEL_6,
-    RHEL_6X64;
+    RHEL_6X64,
+    WINDOWS_8,
+    WINDOWS_8X64,
+    WINDOWS_2012,
+    WINDOWS_2012X64;
 
     public String value() {
         return name().toLowerCase();
diff --git 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
index 81cd791..1eda79c 100644
--- 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
+++ 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
@@ -808,6 +808,14 @@
             return OsType.RHEL_6;
         case RHEL6x64:
             return OsType.RHEL_6X64;
+        case Windows8:
+            return OsType.WINDOWS_8;
+        case Windows8x64:
+            return OsType.WINDOWS_8X64;
+        case Windows2012:
+            return OsType.WINDOWS_2012;
+        case Windows2012x64:
+            return OsType.WINDOWS_2012X64;
 
         default:
             return null;
@@ -855,6 +863,10 @@
             return VmOsType.RHEL6;
         case RHEL_6X64:
             return VmOsType.RHEL6x64;
+        case WINDOWS_8:
+            return VmOsType.Windows8;
+        case WINDOWS_8X64:
+            return VmOsType.Windows8x64;
 
         default:
             return null;
diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
index 79a5318..3a3287d 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
@@ -105,6 +105,36 @@
             sysPrepContent = replace(sysPrepContent, "$ProductKey$", 
Config.<String> GetValue(ConfigValues.ProductKeyWindow7x64));
             break;
 
+        case Windows8:
+            sysPrepContent = LoadFile(Config.<String> 
GetValue(ConfigValues.SysPrepWindows8Path));
+            sysPrepContent =
+                    replace(sysPrepContent, "$ProductKey$", Config.<String> 
GetValue(ConfigValues.ProductKeyWindows8));
+            break;
+
+        case Windows8x64:
+            sysPrepContent = LoadFile(Config.<String> 
GetValue(ConfigValues.SysPrepWindows8x64Path));
+            sysPrepContent =
+                    replace(sysPrepContent,
+                            "$ProductKey$",
+                            Config.<String> 
GetValue(ConfigValues.ProductKeyWindows8x64));
+            break;
+
+        case Windows2012:
+            sysPrepContent = LoadFile(Config.<String> 
GetValue(ConfigValues.SysPrepWindows2012Path));
+            sysPrepContent =
+                    replace(sysPrepContent,
+                            "$ProductKey$",
+                            Config.<String> 
GetValue(ConfigValues.ProductKeyWindows2012));
+            break;
+
+        case Windows2012x64:
+            sysPrepContent = LoadFile(Config.<String> 
GetValue(ConfigValues.SysPrepWindows2012x64Path));
+            sysPrepContent =
+                    replace(sysPrepContent,
+                            "$ProductKey$",
+                            Config.<String> 
GetValue(ConfigValues.ProductKeyWindows2012x64));
+            break;
+
         default:
             break;
         }
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/DataProvider.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/DataProvider.java
index 645b9a3..d28157c 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/DataProvider.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/DataProvider.java
@@ -2533,17 +2533,13 @@
         // return new List<VmOsType>();
 
         /***** TODO: remove once the gwt is using generic api instead of 
backend! *****/
-        windowsOsTypes =
-            new ArrayList<VmOsType>(Arrays.asList(new VmOsType[] {
-                VmOsType.Windows2003,
-                VmOsType.Windows2003x64,
-                VmOsType.Windows2008,
-                VmOsType.Windows2008R2x64,
-                VmOsType.Windows2008x64,
-                VmOsType.Windows7,
-                VmOsType.Windows7x64,
-                VmOsType.WindowsXP
-            }));
+        windowsOsTypes = new ArrayList<VmOsType>();
+
+        for (VmOsType type : VmOsType.values()) {
+            if (type.isWindows()) {
+                windowsOsTypes.add(type);
+            }
+        }
 
         return windowsOsTypes;
         
/*******************************************************************************/
diff --git 
a/frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java
 
b/frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java
index 190cf26..b491def 100644
--- 
a/frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java
+++ 
b/frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java
@@ -115,6 +115,10 @@
 
     String VmOsType___Windows7x64();
 
+    String VmOsType___Windows8();
+
+    String VmOsType___Windows8x64();
+
     String VmOsType___OtherLinux();
 
     String VmOsType___Other();
diff --git 
a/frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties
 
b/frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties
index 055a4e1..bdf0d4d 100644
--- 
a/frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties
+++ 
b/frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties
@@ -55,6 +55,8 @@
 VmOsType___Windows2008R2x64=Windows 2008 R2
 VmOsType___Windows7=Windows 7
 VmOsType___Windows7x64=Windows 7 x64
+VmOsType___Windows8=Windows 8
+VmOsType___Windows8x64=Windows 8 x64
 VmOsType___OtherLinux=Other Linux
 VmOsType___Other=Other
 VmOsType___RHEL6=Red Hat Enterprise Linux 6.x
@@ -174,4 +176,4 @@
 JobExecutionStatus___FINISHED=Finished
 JobExecutionStatus___FAILED=Failed
 JobExecutionStatus___ABORTED=Aborted
-JobExecutionStatus___UNKNOWN=Unknown
\ No newline at end of file
+JobExecutionStatus___UNKNOWN=Unknown


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I21be8a69feddc89adcc671e307a96e4dd7897993
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to