August 2012 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk[master]: sdk: use cached /filter argument rather than demanding it in...
by mpastern＠redhat.com 27 Aug '12

27 Aug '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: use cached /filter argument rather than demanding it in methods ...................................................................... sdk: use cached /filter argument rather than demanding it in methods Filter header value cached during proxy .ctr initiation and then reused instead of asking for it in different methods Change-Id: I45d2b7899035b64cfda0abce5a68287fd323ac1a Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/codegen/doc/documentation.py M src/codegen/imp/imprt.py M src/codegen/utils/headerutils.py M src/ovirtsdk/infrastructure/brokers.py 4 files changed, 51 insertions(+), 60 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7495 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I45d2b7899035b64cfda0abce5a68287fd323ac1a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

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

27 Aug '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: rebase to latest api ...................................................................... sdk: rebase to latest api added correlation_id to: - restore snapshot - detach HostNIC Change-Id: I9e53fc7d3cf85ea5704d48090f8fea4536693a3d Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/brokers.py M src/ovirtsdk/xml/params.py 2 files changed, 259 insertions(+), 165 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7489 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e53fc7d3cf85ea5704d48090f8fea4536693a3d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Add 3.1 Features To Capabilities
by oliel＠redhat.com 27 Aug '12

27 Aug '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Add 3.1 Features To Capabilities ...................................................................... restapi: Add 3.1 Features To Capabilities Change-Id: Ic20e90027722febc173c833b7c872856e7829147 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/utils/FeaturesHelper.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendCapabilityResourceTest.java 5 files changed, 262 insertions(+), 16 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6394 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic20e90027722febc173c833b7c872856e7829147 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Remove Wrong Template Links
by oliel＠redhat.com 27 Aug '12

27 Aug '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Remove Wrong Template Links ...................................................................... restapi: Remove Wrong Template Links Change-Id: Ic4665bbac339d4c3ed1e701bc3105fb407de20cc Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendTemplateResource.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7471 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4665bbac339d4c3ed1e701bc3105fb407de20cc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: 500 instead of 404 for api/events/search
by oliel＠redhat.com 27 Aug '12

27 Aug '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: 500 instead of 404 for api/events/search ...................................................................... restapi: 500 instead of 404 for api/events/search Change-Id: Icf2a7f2abcc3ddae675c6ec2f677380ed25ac121 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendEventsResource.java 1 file changed, 9 insertions(+), 5 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7488 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icf2a7f2abcc3ddae675c6ec2f677380ed25ac121 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Add 'unknown' status to Power Management
by oliel＠redhat.com 27 Aug '12

27 Aug '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Add 'unknown' status to Power Management ...................................................................... restapi: Add 'unknown' status to Power Management Change-Id: I53225ed72745d3a7c5ba97ed3a516f8bc8665ed6 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/PowerManagementStatus.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostResource.java 2 files changed, 4 insertions(+), 2 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7490 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53225ed72745d3a7c5ba97ed3a516f8bc8665ed6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: Move defaults out of service script
by juan.hernandez＠redhat.com 27 Aug '12

27 Aug '12

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Move defaults out of service script ...................................................................... packaging: Move defaults out of service script We used to have the defaults values for local engine configuration files hardcoded in the service start/stop script. This means that other tools need to duplicate these values. This change moves the default values to a separate plain file that can be easily consumed by all kinds of programs. The file will be stored in /usr/share/ovirt-engine because it is *not* a configuration file. Change-Id: I062bb033bf20db9a0d68b8286151345a3653cb03 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile A backend/manager/conf/engine.conf.defaults M packaging/fedora/engine-service.py M packaging/fedora/engine-service.sysconfig M packaging/fedora/engine-service.xml.in 5 files changed, 221 insertions(+), 146 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7387 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I062bb033bf20db9a0d68b8286151345a3653cb03 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: sysprep.inf contents are visible in engine.log (#84966...
by oourfali＠redhat.com 27 Aug '12

27 Aug '12

Oved Ourfali has submitted this change and it was merged. Change subject: core: sysprep.inf contents are visible in engine.log (#849666) ...................................................................... core: sysprep.inf contents are visible in engine.log (#849666) https://bugzilla.redhat.com/849666 When VDSM returs VM devices without address, an information message with the device details is printed to the log. In the sysprep case, it is a floppy device, which may contain sensitive information. This patch will not print the details of floppy devices. Change-Id: I5b9825ef299272ce2d6e9ea3bdecc92bcd1a78cc Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 32 insertions(+), 3 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7491 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5b9825ef299272ce2d6e9ea3bdecc92bcd1a78cc Gerrit-PatchSet: 2 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

Change in ovirt-engine[master]: webadmin: rename V.Disks sub-tabs to Disks (#850749)
by derez＠redhat.com 27 Aug '12

27 Aug '12

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: rename V.Disks sub-tabs to Disks (#850749) ...................................................................... webadmin: rename V.Disks sub-tabs to Disks (#850749) https://bugzilla.redhat.com/850749 Renamed all 'Virtual Disks' sub-tabs to 'Disks'. Change-Id: Id469b62f26e95c0e8d27a8ed8f1e9d9649ff6d81 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolDiskListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateDiskListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmDiskListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Constants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportTemplatePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportVmPopupView.java 8 files changed, 11 insertions(+), 14 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7480 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id469b62f26e95c0e8d27a8ed8f1e9d9649ff6d81 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: packaging: Check ENGINE_PROXY_ENABLED correctly
by ydary＠redhat.com 27 Aug '12

27 Aug '12

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: Check ENGINE_PROXY_ENABLED correctly ...................................................................... packaging: Check ENGINE_PROXY_ENABLED correctly Currently the setup only checks if this configuration is present, but it should check if it is present and if it is value is true. Change-Id: I7e7819b51ef8f6a91bc3c1c4e7389c034bc6567b Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M packaging/ovirt-engine-dwh-setup.py 1 file changed, 7 insertions(+), 4 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7454 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e7819b51ef8f6a91bc3c1c4e7389c034bc6567b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0