November 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: services: notifier: use new health uri
by emesika＠redhat.com 05 Nov '13

05 Nov '13

Eli Mesika has submitted this change and it was merged. Change subject: services: notifier: use new health uri ...................................................................... services: notifier: use new health uri no idea why notifier does not use API. URI of health servlet was modified, we need to use the new one. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=961677 Change-Id: I5747c1137cd8d0b4c5e89f82d772526103ce8dc5 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/EngineLocalConfig.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/EngineMonitorService.java 2 files changed, 7 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20791 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5747c1137cd8d0b4c5e89f82d772526103ce8dc5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: Use the live-data metadata field when showing host status
by sbonazzo＠redhat.com 05 Nov '13

05 Nov '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: Use the live-data metadata field when showing host status ...................................................................... Use the live-data metadata field when showing host status Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1026351 Change-Id: I56319093035554b1ead8e738856f1f8786e3a6b3 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M src/ovirt_hosted_engine_setup/vm_status.py 1 file changed, 14 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20862 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56319093035554b1ead8e738856f1f8786e3a6b3 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: Add live-data key to parsed stats to mark live/stale data st...
by msivak＠redhat.com 05 Nov '13

05 Nov '13

Martin Sivák has submitted this change and it was merged. Change subject: Add live-data key to parsed stats to mark live/stale data status ...................................................................... Add live-data key to parsed stats to mark live/stale data status Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1026351 Change-Id: I32e1b5cbaeb075b1e543f9beaba3734f812ccf58 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M ovirt_hosted_engine_ha/env/constants.py.in M ovirt_hosted_engine_ha/lib/metadata.py 2 files changed, 6 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20860 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32e1b5cbaeb075b1e543f9beaba3734f812ccf58 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: fix issue with search on OSs
by iheim＠redhat.com 05 Nov '13

05 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: core: fix issue with search on OSs ...................................................................... core: fix issue with search on OSs * fix the search validation of template * on UI load - set the search dependencies once the cache population is done - prevents null list of OSs * make the auto-completion case insensitive * show the full list when the input is blank Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1003804 Change-Id: I9434189173939ba595862a40483d8da5107228b2 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/OsValueAutoCompleter.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VmTemplateConditionFieldAutoCompleter.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/BaseApplicationInit.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java 4 files changed, 15 insertions(+), 11 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20743 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9434189173939ba595862a40483d8da5107228b2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Detect host in idle prep for maintenance and re-migrate
by rgolan＠redhat.com 05 Nov '13

05 Nov '13

Roy Golan has submitted this change and it was merged. Change subject: core: Detect host in idle prep for maintenance and re-migrate ...................................................................... core: Detect host in idle prep for maintenance and re-migrate Host going to Maintenance can have ongoing incoming migration before it changes its status to Preparing for maintainance. This host will migrate all its VMs except those incoming, which are not he's still. This will result the host staying in PreparingForMaint till manual intervention. The solution has 3 parts: 1. cancel all incoming migrations 2. make cancel migration command trigger a rerun of the migration automatically 3. detect if host is idle long time in PreparingForMaint and migrate VMs out Bug-Url: https://bugzilla.redhat.com/966503 Change-Id: I0e18bcfbc566b6fd92d276c5c739ba607e2a53f0 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CancelMigrateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MaintenanceNumberOfVdssCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/IVdsEventListener.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/CancelMigrateVDSCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/CancelMigrationVDSParameters.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/dbscripts/vms_sp.sql M packaging/etc/engine-config/engine-config.properties 13 files changed, 100 insertions(+), 19 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17942 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e18bcfbc566b6fd92d276c5c739ba607e2a53f0 Gerrit-PatchSet: 25 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: tools: send notifications according to order of creation.
by masayag＠redhat.com 05 Nov '13

05 Nov '13

Moti Asayag has submitted this change and it was merged. Change subject: tools: send notifications according to order of creation. ...................................................................... tools: send notifications according to order of creation. Change-Id: Ifcaab815ad880269d57d3faa1f8eb67be78d4e2d Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1020793 Signed-off-by: Mooli Tayer <mtayer(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/NotificationService.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20870 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifcaab815ad880269d57d3faa1f8eb67be78d4e2d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: userportal: fix debug URI in pom.xml
by derez＠redhat.com 04 Nov '13

04 Nov '13

Daniel Erez has submitted this change and it was merged. Change subject: userportal: fix debug URI in pom.xml ...................................................................... userportal: fix debug URI in pom.xml Following product URIs rearranging, modified userportal debug URI accordingly (used for GWT devmode plugin). Change-Id: If394f00ed0da7fe717af8e375fb750b972807bbd Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/userportal-gwtp/pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20820 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If394f00ed0da7fe717af8e375fb750b972807bbd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: fix debug URI in pom.xml
by derez＠redhat.com 04 Nov '13

04 Nov '13

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: fix debug URI in pom.xml ...................................................................... webadmin: fix debug URI in pom.xml Following product URIs rearranging, modified webadmin debug URI accordingly (used for GWT devmode plugin). Change-Id: Idb2e53c97d9afbfa2681975f7fc7f9ed3248c73f Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/webadmin/pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20819 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idb2e53c97d9afbfa2681975f7fc7f9ed3248c73f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine-cli[cli_3.3]: cli: update on vm --os-boot does not work as expected(#1012985)
by mpastern＠redhat.com 04 Nov '13

04 Nov '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: update on vm --os-boot does not work as expected(#1012985) ...................................................................... cli: update on vm --os-boot does not work as expected(#1012985) Updating vm --os-bbot does not update the boot order Fixed by constructing object using factory and set parameters to it instead of fetching it from server and updating it. Change-Id: I34a910a7f1ecae5385e2a6dcd2297a65f419bc9c Bug-Url: https://bugzilla.redhat.com/1012985 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M src/ovirtcli/command/update.py 1 file changed, 9 insertions(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20880 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I34a910a7f1ecae5385e2a6dcd2297a65f419bc9c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.3 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: tools: Fixes adding superuser permissions
by iheim＠redhat.com 04 Nov '13

04 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: tools: Fixes adding superuser permissions ...................................................................... tools: Fixes adding superuser permissions Fixes adding superuser permissions when adding or editing a domain. Change-Id: Iac3e1792e6c8a106612195c824a777b01d35851d Bug-Url: https://bugzilla.redhat.com/1020172 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java 1 file changed, 16 insertions(+), 30 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iac3e1792e6c8a106612195c824a777b01d35851d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0