January 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: configure dialog should fetch data onReveal
by gchaplik＠redhat.com 16 Jan '13

16 Jan '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: configure dialog should fetch data onReveal ...................................................................... webadmin: configure dialog should fetch data onReveal Instead of onBind Change-Id: I85ad3cd9f77b346822f8b2482d34353d159de50a Related-Bug-Url: https://bugzilla.redhat.com/888492 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/configure/ConfigurePopupPresenterWidget.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11042 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85ad3cd9f77b346822f8b2482d34353d159de50a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine-sdk-java[master]: sdk: prepare for the next development iteration
by mpastern＠redhat.com 16 Jan '13

16 Jan '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: prepare for the next development iteration ...................................................................... sdk: prepare for the next development iteration Change-Id: I73ab527c3df4cfe2e145100b058f09fefbe2ba62 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M ovirt-engine-sdk-java/pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11105 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I73ab527c3df4cfe2e145100b058f09fefbe2ba62 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-sdk-java[master]: sdk: release 1.0.0.2-1
by mpastern＠redhat.com 16 Jan '13

16 Jan '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: release 1.0.0.2-1 ...................................................................... sdk: release 1.0.0.2-1 - in vm/vmnic device element renamed to reported_device - to host added libvirtVersion - fixed NULL params exception in constructor Change-Id: I4e3b6a9d459fad26119fb7447592972b3d998f0e Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M ovirt-engine-sdk-java/pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11099 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e3b6a9d459fad26119fb7447592972b3d998f0e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: tiny fix for make template dialog's focus traversa...
by gchaplik＠redhat.com 16 Jan '13

16 Jan '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: tiny fix for make template dialog's focus traversal ...................................................................... webadmin: tiny fix for make template dialog's focus traversal The tab index of 'Quota' combobox is wrong, resulting in wrong focus traversal order. This patch fix it by giving the 'Quota' combobox the right tab index so that it will be focused right after 'Host Cluster' combobox. Change-Id: If10096e8f9e1141d2d0a46a600b0e82e849b59ba Bug-Url: https://bugzilla.redhat.com/873133 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmMakeTemplatePopupWidget.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11096 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If10096e8f9e1141d2d0a46a600b0e82e849b59ba Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>

1 0

Change in ovirt-engine-sdk-java[master]: sdk: regenerate against latest api
by mpastern＠redhat.com 16 Jan '13

16 Jan '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: regenerate against latest api ...................................................................... sdk: regenerate against latest api - in vm/vmnic device element renamed to reported_device - to host added libvirtVersion Change-Id: I2014e9803a4be9f2edd85af32585cc740256eb71 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/entities/BaseResource.java M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/entities/Device.java M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/entities/Host.java M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/entities/ObjectFactory.java M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/entities/ReportedDevices.java 5 files changed, 206 insertions(+), 154 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11097 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2014e9803a4be9f2edd85af32585cc740256eb71 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: cleanup
by mpastern＠redhat.com 16 Jan '13

16 Jan '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: cleanup ...................................................................... sdk: cleanup Change-Id: I62a42b189f30505a5f68fe759b264c70c0094f11 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/codegen/entrypoint/entrypoint.py M src/codegen/subcollection/subresource.py M src/ovirtsdk/api.py M src/ovirtsdk/infrastructure/brokers.py 4 files changed, 37 insertions(+), 51 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11095 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I62a42b189f30505a5f68fe759b264c70c0094f11 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>

1 0

Change in ovirt-engine[master]: packaging: do not use install_without_maven
by oschreib＠redhat.com 16 Jan '13

16 Jan '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: do not use install_without_maven ...................................................................... packaging: do not use install_without_maven We can simply use: $ make MVN=true Change-Id: Idfdc06c98dc3afd4120e17dd7be380755e40b5cf Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile 1 file changed, 0 insertions(+), 4 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11088 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idfdc06c98dc3afd4120e17dd7be380755e40b5cf Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: enable auto recovery for storage domains
by yzaslavs＠redhat.com 16 Jan '13

16 Jan '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: enable auto recovery for storage domains ...................................................................... core: enable auto recovery for storage domains 1. set AutoRecoveryAllowedTypes as true for storage domains. 2. set the correct value for AutoRecoveryAllowedTypes in ConfigValues.java so that auto recovery will be enabled for hosts as configured in vdc_options table without having to reload it. Change-Id: I1ad1528e61bd4a50547b87005d5e71148cac6833 Bug-Url: https://bugzilla.redhat.com/889985 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Liron Aravot: Verified; Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10387 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ad1528e61bd4a50547b87005d5e71148cac6833 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Haim Ateya <hateya(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine-sdk-java[master]: sdk: add support for NULL params in constructor
by mpastern＠redhat.com 16 Jan '13

16 Jan '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: add support for NULL params in constructor ...................................................................... sdk: add support for NULL params in constructor Change-Id: Ie3bfcf54f5f573850faa65a86391642cf784d8c2 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/web/ConnectionsPoolBuilder.java M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/web/HttpProxyBuilder.java 2 files changed, 28 insertions(+), 17 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11093 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie3bfcf54f5f573850faa65a86391642cf784d8c2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Removing using of latch all around a code
by mkublin＠redhat.com 16 Jan '13

16 Jan '13

Michael Kublin has submitted this change and it was merged. Change subject: engine: Removing using of latch all around a code ...................................................................... engine: Removing using of latch all around a code In engine code has requirements to run couple of simentenious tasks and stuck a main thread untill they will be finished. A previous solution was to use a latch, which required to write a same code and keep some logic for increasing and reducing latch count. A new solution is to provide a generic way for such problem and to use a standard java api for running couple of parallel tasks and waits untill all of them will be finished Change-Id: I34ffc04f7abf57f3d27322a5615f89d07dee16ef Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GlusterMultipleActionsRunner.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MultipleActionsRunner.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ReconstructMasterDomainCommand.java R backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/LatchedRunnableWrapper.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/MultiThreadedDAOTest.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/SyncronizeNumberOfAsyncOperations.java D backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/thread/LatchedRunnableExecutor.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/threadpool/ThreadPoolUtil.java D backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/thread/LatchedRunnableWrapperTest.java 10 files changed, 136 insertions(+), 320 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11037 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I34ffc04f7abf57f3d27322a5615f89d07dee16ef Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0