[Engine-devel] [CORE] broken backward compatibility for vm creation

[Eyal Edri]

----- Original Message -----
> From: "Michael Pasternak" <mpastern at redhat.com>
> To: "Vitor de Lima" <vitor.lima at eldorado.org.br>, "engine-devel" <engine-devel at ovirt.org>, "Eyal Edri"
> <eedri at redhat.com>
> Cc: "Michal Skrivanek" <mskrivan at redhat.com>
> Sent: Tuesday, December 24, 2013 2:10:13 PM
> Subject: Re: [Engine-devel] [CORE] broken backward compatibility for vm creation
> 
> 
> Thanks Vitor,
> 
> Such changes should be merged all together, otherwise they create significant
> noise in master,
> and make testing not possible what can lead to not stable master afterwards,
> 
> btw eyal, shouldn't Jenkins job be alerting on such changes?

we don't have currently any tests on engine rest other than unit tests/dao tests running on oVirt.


> 
> On 12/24/2013 02:04 PM, Vitor de Lima wrote:
> > This is temporarily broken due to the introduction of multi-architecture
> > support. Changes #18227 and #20667 are going to fix this by blocking the
> > addition of VMs into clusters without a defined architecture and properly
> > selecting the default OS when creating VMs.
> > 
> >> -----Original Message-----
> >> From: engine-devel-bounces at ovirt.org [mailto:engine-devel-
> >> bounces at ovirt.org] On Behalf Of Michael Pasternak
> >> Sent: terça-feira, 24 de dezembro de 2013 07:30
> >> To: engine-devel
> >> Cc: Michal Skrivanek
> >> Subject: [Engine-devel] [CORE] broken backward compatibility for vm
> >> creation
> >>
> >>
> >> [oVirt shell (connected)]# add vm --name test --cluster-id 00000001-0001-
> >> 0001-0001-000000000001 --template-id 00000000-0000-0000-0000-
> >> 000000000000
> >>
> >>
> >> ==========================================================
> >> ================= ERROR
> >> ==========================================================
> >> ===============
> >>   status: 400
> >>   reason: Bad Request
> >>   detail: Cannot add VM. Selected operating system is not supported by the
> >> architecture.
> >>
> >> ==========================================================
> >> ==========================================================
> >> =======================================
> >>
> >> [oVirt shell (connected)]# show template 00000000-0000-0000-0000-
> >> 000000000000
> >>
> >> id                        : 00000000-0000-0000-0000-000000000000
> >> name                      : Blank
> >> description               : Blank template
> >> cluster-id                : 00000001-0001-0001-0001-000000000001
> >> cpu-architecture          : UNDEFINED
> >> cpu-topology-cores        : 1
> >> cpu-topology-sockets      : 1
> >> cpu_shares                : 0
> >> creation_time             : 2008-04-01 00:00:00+03:00
> >> delete_protected          : False
> >> display-allow_override    : False
> >> display-monitors          : 1
> >> display-single_qxl_pci    : False
> >> display-smartcard_enabled : False
> >> display-type              : spice
> >> high_availability-enabled : False
> >> high_availability-priority: 0
> >> memory                    : 1073741824
> >> origin                    : rhev
> >> os-boot-dev               : hd
> >> os-type                   : other
> >> stateless                 : False
> >> status-state              : ok
> >> type                      : desktop
> >> usb-enabled               : False
> >>
> >> [oVirt shell (connected)]# show cluster 00000001-0001-0001-0001-
> >> 000000000001
> >>
> >> id                                         :
> >> 00000001-0001-0001-0001-000000000001
> >> name                                       : Default
> >> description                                : The default server cluster
> >> ballooning_enabled                         : False
> >> data_center-id                             :
> >> 00000002-0002-0002-0002-000000000002
> >> error_handling-on_error                    : migrate
> >> gluster_service                            : False
> >> memory_policy-overcommit-percent           : 100
> >> memory_policy-transparent_hugepages-enabled: True
> >> scheduling_policy-policy                   : none
> >> threads_as_cores                           : False
> >> trusted_service                            : False
> >> tunnel_migration                           : False
> >> version-major                              : 3
> >> version-minor                              : 3
> >> virt_service                               : True
> >>
> >> --
> >>
> >> Michael Pasternak
> >> RedHat, ENG-Virtualization R&D
> >> _______________________________________________
> >> Engine-devel mailing list
> >> Engine-devel at ovirt.org
> >> http://lists.ovirt.org/mailman/listinfo/engine-devel
> 
> 
> --
> 
> Michael Pasternak
> RedHat, ENG-Virtualization R&D
>