Bug 1109546

Summary:	Create vm via REST with parameter <os type="windows_2008r2"/> will create vm with <os type="other"/>
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	Artyom <alukiano>
Component:	ovirt-engine-restapi	Assignee:	Martin Betak <mbetak>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Petr Kubica <pkubica>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	3.4.0	CC:	bazulay, gklein, iheim, istein, juan.hernandez, oramraz, rbalakri, Rhev-m-bugs, yeylon
Target Milestone:	---
Target Release:	3.5.0
Hardware:	x86_64
OS:	Linux
Whiteboard:	virt
Fixed In Version:	vt2.2	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-02-17 08:27:55 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1142923, 1156165

Description Artyom 2014-06-15 07:59:38 UTC

Description of problem:
Create vm via REST with parameter <os type="windows_2008r2"/> will create vm with <os type="other"/>, from some reason engine change value "windows_2008r2" with value "other"

Version-Release number of selected component (if applicable):
av9.5

How reproducible:
Always

Steps to Reproduce:
1. Create new vm via REST:
<vm>
<name>vm2</name>
<cluster>
<name>cluster_name</name>
</cluster>
<template>
<name>Blank</name>
</template>
<os type="windows_2008r2"/>
</vm>

2.
3.

Actual results:
Vm with <os type="other"/>

Expected results:
Vm with <os type="windows_2008r2"/>

Additional info:
I also can't see windows_2008r2 in /api/capabilities for 3.4 version

Comment 1 Juan Hernández 2014-06-16 08:48:21 UTC

Apparently the list of operating systems provided in /capabilities isn't the same list that is used to check the value given by the caller. In /capabilities "windows_2008r2" is one of the values:

  <os_types>
    <os_type>UNASSIGNED</os_type>
    <os_type>WINDOWS_XP</os_type>
    <os_type>WINDOWS_2003</os_type>
    <os_type>WINDOWS_2008</os_type>
    <os_type>OTHER_LINUX</os_type>
    <os_type>OTHER</os_type>
    <os_type>RHEL_5</os_type>
    <os_type>RHEL_4</os_type>
    <os_type>RHEL_3</os_type>
    <os_type>WINDOWS_2003X64</os_type>
    <os_type>WINDOWS_7</os_type>
    <os_type>WINDOWS_7X64</os_type>
    <os_type>RHEL_5X64</os_type>
    <os_type>RHEL_4X64</os_type>
    <os_type>RHEL_3X64</os_type>
    <os_type>WINDOWS_2008X64</os_type>
    <os_type>WINDOWS_2008R2</os_type>  <-- Here it is, case isn't relevant
    <os_type>RHEL_6</os_type>
    <os_type>RHEL_6X64</os_type>
    <os_type>WINDOWS_8</os_type>
    <os_type>WINDOWS_8X64</os_type>
    <os_type>WINDOWS_2012X64</os_type>
  </os_types>

But when checking the OS type the OSRepositoryImpl.idToUnameLookup class contains the following:

  {
    0=other,
    1=windows_xp,
    3=windows_2003,
    4=windows_2008,
    5=other_linux,
    7=rhel_5,
    8=rhel_4,
    9=rhel_3,
    10=windows_2003x64,
    11=windows_7,
    12=windows_7x64,
    13=rhel_5x64,
    14=rhel_4x64, 
    15=rhel_3x64, 
    1001=other_ppc64, 
    17=windows_2008R2x64,  <-- This is the closest, but not equal
    16=windows_2008x64, 
    1003=rhel_6_ppc64, 
    19=rhel_6x64, 
    1002=other_linux_ppc64, 
    18=rhel_6, 
    21=windows_8x64, 
    20=windows_8, 
    1004=sles_11_ppc64, 
    23=windows_2012x64,
    24=rhel_7x64, 
    1255=ubuntu_13_10, 
    1254=ubuntu_13_04, 
    1253=ubuntu_12_10, 
    1252=ubuntu_12_04, 
    1193=sles_11
  }

I believe that these two lists should match.

Comment 2 Martin Betak 2014-07-29 09:45:33 UTC

*** Bug 1115350 has been marked as a duplicate of this bug. ***

Comment 4 Petr Kubica 2014-09-15 08:53:27 UTC

Verified 
with: <os type="windows_2008r2x64"/>
in: 3.5.0-0.10.master.el6ev

Comment 5 Petr Kubica 2014-09-15 08:59:38 UTC

EDIT: with <os type="windows_2008R2x64"/>

Comment 6 Omer Frenkel 2015-02-17 08:27:55 UTC

RHEV-M 3.5.0 has been released