- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: after running vm as stateless we can no longer run the...
by emesika＠redhat.com 10 May '12

10 May '12

Eli Mesika has submitted this change and it was merged. Change subject: core: after running vm as stateless we can no longer run the vm ...................................................................... core: after running vm as stateless we can no longer run the vm Was unable to reproduce the bug. Reported scenario worked in both run and run-once no matter if VM was stateless and stateless set to false in run-once or vice-versa. Fixied a bug in stateless VM unmanaged devices cleanup that were cleaned up to now only when VM was suspended and then stop rather than on stop from any state as it should be. Change-Id: Ia139e98596715bb3b09a589da6dfae98a38b7249 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StopVmCommandBase.java 1 file changed, 17 insertions(+), 13 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4153 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia139e98596715bb3b09a589da6dfae98a38b7249 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: rest: fix payload attribute definition for json support
by oliel＠redhat.com 10 May '12

10 May '12

Ori Liel has submitted this change and it was merged. Change subject: rest: fix payload attribute definition for json support ...................................................................... rest: fix payload attribute definition for json support Change-Id: I7f966888b9c296828ba19d2afdcfd9221ca4ebcf Signed-off-by: Shahar Havivi<shavivi(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 2 files changed, 16 insertions(+), 4 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4246 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f966888b9c296828ba19d2afdcfd9221ca4ebcf Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: release 1.6.5-1
by mpastern＠redhat.com 10 May '12

10 May '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: release 1.6.5-1 ...................................................................... sdk: release 1.6.5-1 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> Change-Id: I5cc7c90c4e6cb422f204088c7aa221b547fb524c --- M Makefile M ovirt-engine-sdk.spec.in M setup.py 3 files changed, 31 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/4290 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5cc7c90c4e6cb422f204088c7aa221b547fb524c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: rebase to latest api
by mpastern＠redhat.com 10 May '12

10 May '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: rebase to latest api ...................................................................... sdk: rebase to latest api sdk: Add session support sdk: Infinite recursion when calling non-existent method #808124 Network.update(): + [@param network.usages: collection] + { + [@ivar usage: string] + } HostNIC + [@param hostnic.ip.mtu: int] StorageDomainTemplate/StorageDomainVM.add(): + [@param action.storagedomain.id|name: string] + [@param action.vm.disks.disk: collection] + { + [@ivar id: string] + } StorageDomains.add(): + Overload 4: + @param storagdomain.name: string + @param storagdomain.host.id|name: string + @param storagdomain.type: string + @param storagdomain.storage.type: string + @param storagdomain.format: boolean + @param storagdomain.storage.path: string + @param storagedomain.storage.vfs_type: string + [@param storagdomain.storage.address: string] + [@param storagdomain.storage.mount_options: string] Template/VM: + [@param template.display.allow_reconnect: boolean] Template/VM.add(): + [@param template.display.allow_reconnect: boolean] + [@param template.vm.disks.disk: collection] + { + [@ivar id: string] + [@ivar storage_domains.storage_domain: collection] + { + [@param id: string] + } + } VMDisk.activate(): VMDisk.deactivate(): VMNic.activate(): VMNic.deactivate(): Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> Change-Id: I6cd29a14b7a7c322d307e6b386e5e428da4a75f6 --- M src/ovirtsdk/infrastructure/brokers.py M src/ovirtsdk/xml/params.py 2 files changed, 958 insertions(+), 176 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/4289 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6cd29a14b7a7c322d307e6b386e5e428da4a75f6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: fix split function and its use for spice security chan...
by oourfali＠redhat.com 10 May '12

10 May '12

Oved Ourfali has submitted this change and it was merged. Change subject: core: fix split function and its use for spice security channels ...................................................................... core: fix split function and its use for spice security channels Previous commit added the split function, but it didn't take into account what if the entries are not there at all (new install). It assumed they are there, as they would appear in the add section. Better implementation is not to assume that, creating the old entries with old value in install phase, or taking the original value from the DB in case of upgrade. Change-Id: I32b9b89c0c28180738b8532561e110fcd3e61d1b Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M backend/manager/dbscripts/common_sp.sql M backend/manager/dbscripts/upgrade/pre_upgrade/config.sql 2 files changed, 24 insertions(+), 20 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4271 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32b9b89c0c28180738b8532561e110fcd3e61d1b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0