December 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.5]: jsonrpc: version bump
by oourfali＠redhat.com 15 Dec '14

15 Dec '14

Oved Ourfali has submitted this change and it was merged. Change subject: jsonrpc: version bump ...................................................................... jsonrpc: version bump Changing the ordering to mitigate rebase conflicts. Change-Id: Ib22057574a52527a2a3f205dd58e1303a044ec23 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> Bug-Url: https://bugzilla.redhat.com/1172757 --- M ovirt-engine.spec.in M pom.xml 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Piotr Kliczewski: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36173 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib22057574a52527a2a3f205dd58e1303a044ec23 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-ha[master]: don't use vdsm backend when there is not valid vol or imgUUID
by msivak＠redhat.com 15 Dec '14

15 Dec '14

Martin Sivák has submitted this change and it was merged. Change subject: don't use vdsm backend when there is not valid vol or imgUUID ...................................................................... don't use vdsm backend when there is not valid vol or imgUUID Change-Id: Idcd181d260f4f20676a4124b899575b42935753c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1171452 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/lib/storage_backends.py 2 files changed, 3 insertions(+), 1 deletion(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/36051 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idcd181d260f4f20676a4124b899575b42935753c Gerrit-PatchSet: 7 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Artyom Lukianov <alukiano(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: Revert "core: Set emulated machines after capabilities"
by ahadas＠redhat.com 15 Dec '14

15 Dec '14

Arik Hadas has submitted this change and it was merged. Change subject: Revert "core: Set emulated machines after capabilities" ...................................................................... Revert "core: Set emulated machines after capabilities" This reverts commit 9bf94d4a779719c4286f97a5f98166a7961866bf since host cannot be added to empty cluster with this change. Change-Id: I08a2350a83feba4f050e312bf23925ff56d9ec11 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VirtMonitoringStrategy.java 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/36171 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I08a2350a83feba4f050e312bf23925ff56d9ec11 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vdelima(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: userportal, webadmin: fixed tabindexes for login pages
by awels＠redhat.com 15 Dec '14

15 Dec '14

Alexander Wels has submitted this change and it was merged. Change subject: userportal, webadmin: fixed tabindexes for login pages ...................................................................... userportal, webadmin: fixed tabindexes for login pages Since PatternFly adoption, tabbing through fields on the login pages was erratic. Fixed to be in the proper, predictable order. As part of fixing this, changed UiCommonEditorVisitor to not set tabindex on an editor if it's already set (i.e. don't override a manually-set tabindex). Change-Id: I434eb12b5efe003c0d1c139ad94d4d352668f403 Bug-Url: https://bugzilla.redhat.com/1154051 Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/editor/UiCommonEditorVisitor.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/PatternflyUiCommandButton.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/login/view/LoginFormView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/login/view/LoginFormView.ui.xml 4 files changed, 22 insertions(+), 15 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Greg Sheremeta: Verified -- To view, visit http://gerrit.ovirt.org/34738 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I434eb12b5efe003c0d1c139ad94d4d352668f403 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: Make the number of concurrent vds and irs connections ...
by oourfali＠redhat.com 15 Dec '14

15 Dec '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Make the number of concurrent vds and irs connections configurable ...................................................................... core: Make the number of concurrent vds and irs connections configurable Allow tuning of apache HttpClients MaxConnectionsPerHost and MaxTotalConnections values, to increase command throughput. - Add VdsMaxConnectionsPerHost and IrsMaxConnectionsPerHost config values. Each value is used for specific type of the connection, with default 2, which is the value in use without tuning - Add MaxTotalConnections config value which determines maxiumum number of allowed connections, with default 20, which is the value in use without tuning Change-Id: Id6a7e2bc0f6865e41dd85ab1fd188625cab22ddf Bug-Url: https://bugzilla.redhat.com/1168538 Signed-off-by: Tim Speetjens <tim.speetjens(a)redhat.com> Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- 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/jsonrpc/TransportFactory.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/xmlrpc/XmlRpcUtils.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 4 files changed, 49 insertions(+), 8 deletions(-) Approvals: Piotr Kliczewski: Verified Tim Speetjens: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36161 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6a7e2bc0f6865e41dd85ab1fd188625cab22ddf Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tim Speetjens <tim.speetjens(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Make the number of concurrent vds and irs connections ...
by oourfali＠redhat.com 15 Dec '14

15 Dec '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Make the number of concurrent vds and irs connections configurable ...................................................................... core: Make the number of concurrent vds and irs connections configurable Allow tuning of apache HttpClients MaxConnectionsPerHost and MaxTotalConnections values, to increase command throughput. - Add VdsMaxConnectionsPerHost and IrsMaxConnectionsPerHost config values. Each value is used for specific type of the connection, with default 2, which is the value in use without tuning - Add MaxTotalConnections config value which determines maxiumum number of allowed connections, with default 20, which is the value in use without tuning Change-Id: Id6a7e2bc0f6865e41dd85ab1fd188625cab22ddf Bug-Url: https://bugzilla.redhat.com/1168538 Signed-off-by: Tim Speetjens <tim.speetjens(a)redhat.com> Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- 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/TransportFactory.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/xmlrpc/XmlRpcUtils.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 4 files changed, 49 insertions(+), 8 deletions(-) Approvals: Piotr Kliczewski: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35717 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6a7e2bc0f6865e41dd85ab1fd188625cab22ddf Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tim Speetjens <tim.speetjens(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Pablo Iranzo Gómez <Pablo.Iranzo(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tim Speetjens <tim.speetjens(a)redhat.com> Gerrit-Reviewer: Tomáš Došek <tdosek(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: don't migrate HE VM using engine when moving host to m...
by tnisan＠redhat.com 15 Dec '14

15 Dec '14

Tal Nisan has submitted this change and it was merged. Change subject: core: don't migrate HE VM using engine when moving host to maintenance ...................................................................... core: don't migrate HE VM using engine when moving host to maintenance Change-Id: I499380807e3f4d0c882771c7f22e8233e29aba93 Bug-Url: https://bugzilla.redhat.com/1172701 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MaintenanceVdsCommand.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Jiří Moskovčák: Verified Doron Fediuck: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36071 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I499380807e3f4d0c882771c7f22e8233e29aba93 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: fix external events injection
by oourfali＠redhat.com 15 Dec '14

15 Dec '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: fix external events injection ...................................................................... core: fix external events injection This patch fixes a regression that cause external events injection to fail MLA validation. The problem was in the DB squashing of 3.2 done in commit 11bc072021a28560261dc2601f8fd106c9c322e6 Change-Id: I8703aa99b8f630e99110de8fc523a55c63fd10c1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1173947 Signed-off-by: emesika <emesika(a)redhat.com> --- A packaging/dbscripts/upgrade/03_05_1190_fix_external_events_role_groups.sql 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36154 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8703aa99b8f630e99110de8fc523a55c63fd10c1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: fix external events injection
by oourfali＠redhat.com 15 Dec '14

15 Dec '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: fix external events injection ...................................................................... core: fix external events injection This patch fixes a regression that cause external events injection to fail MLA validation. The problem was in the DB squashing of 3.2 done in commit 11bc072021a28560261dc2601f8fd106c9c322e6 Change-Id: I8703aa99b8f630e99110de8fc523a55c63fd10c1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1173947 Signed-off-by: emesika <emesika(a)redhat.com> --- A packaging/dbscripts/upgrade/03_06_0600_fix_external_events_role_groups.sql 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36149 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8703aa99b8f630e99110de8fc523a55c63fd10c1 Gerrit-PatchSet: 1 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: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Set emulated machines after capabilities
by ahadas＠redhat.com 15 Dec '14

15 Dec '14

Arik Hadas has submitted this change and it was merged. Change subject: core: Set emulated machines after capabilities ...................................................................... core: Set emulated machines after capabilities After incorrectly adding a ppc64 host in an empty x86-64 cluster and removing it, the cluster cannot accept new x86-64 hosts, accusing incompatibility between the emulatedMachines property of the cluster and the hosts being added. To fix the issue, the place where emulated machines is set checks if the host is operational before being executed. Change-Id: Ibc25acf8e9914e382a5ebf66f9c5d895ed4dab2a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1061320 Signed-off-by: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Signed-off-by: Vitor de Lima <vdelima(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VirtMonitoringStrategy.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Vitor de Lima: Verified Arik Hadas: Looks good to me, approved Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25699 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc25acf8e9914e382a5ebf66f9c5d895ed4dab2a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vdelima(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0