- Engine-commits - oVirt List Archives

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

adding Freeipa to Ovirtengine
by Khalid ES SRIJI 15 Dec '14

15 Dec '14

Hello , i have 4 servers in my work one for ipa . one for the hyper one for ovirt engine one is a file server and i use an external DNS server well the problem is that the ovirt engine server can not authenticat to ipa server when a want to add the Freeipa server to the the ovirt engine by the command "engine-manage-domains add --provider=ipa --domain=mydomaine.com --user=admin" it shows the following message "Error: exception message: Aucun chemin d'accÃ¨s pour atteindre l'hÃ´te cible Failure while testing domain fso.ump.ma. Details: Kerberos error. Please check log for further details." i have 2 users in Freeipa.All servers can ping to each other. I tried all the solutions on the mail list and the internet and it did not work . hope there is a solution for this problem . thnks.

2 1

Change in ovirt-engine[master]: webadmin: Start/stop progress on async operations
by lvernia＠redhat.com 14 Dec '14

14 Dec '14

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Start/stop progress on async operations ...................................................................... webadmin: Start/stop progress on async operations Implemented some infrastructure to automatically trigger loading animation whenever an async operation is invoked, and cancel the animation whenever the operations returns. This only happens if upon initialization of the operation, the source model is "properly" passed. For a single query, the model should be passed as the target of its AsyncQuery callback. Otherwise (multiple queries, single/multiple actions), the model should be passed as the "state" argument. Also, this only happens in case progress wasn't manually set; if it had been, then this would override the automatic infrastructure altogether. Change-Id: I24ed80a5453668b37f488e0226f7bdfbf36e503d Bug-Url: https://bugzilla.redhat.com/1167327 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/frontend/exclude-filters.xml M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java A frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/communication/AsyncOperationComplete.java A frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/communication/AsyncOperationStarted.java M frontend/webadmin/modules/frontend/src/test/java/org/ovirt/engine/ui/frontend/FrontendActionTest.java M frontend/webadmin/modules/frontend/src/test/java/org/ovirt/engine/ui/frontend/FrontendTest.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/AbstractModelBoundPopupPresenterWidget.java 7 files changed, 147 insertions(+), 16 deletions(-) Approvals: Lior Vernia: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35964 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24ed80a5453668b37f488e0226f7bdfbf36e503d Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(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[master]: webadmin: Change query context from String to Object
by lvernia＠redhat.com 14 Dec '14

14 Dec '14

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Change query context from String to Object ...................................................................... webadmin: Change query context from String to Object There's no reason to limit the context of a query to be represented by a String. Also, changed the argument's name from "context" to "state", to comply with the more widely-used argument related to action running. Change-Id: Ib5a0e51fdf3eeb092d41a4724be89f432d975a83 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java 1 file changed, 12 insertions(+), 12 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/35963 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib5a0e51fdf3eeb092d41a4724be89f432d975a83 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(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-dwh[master]: packaging: setup: Use dwh_history_timekeeping from engine
by sradco＠redhat.com 14 Dec '14

14 Dec '14

Shirly Radco has submitted this change and it was merged. Change subject: packaging: setup: Use dwh_history_timekeeping from engine ...................................................................... packaging: setup: Use dwh_history_timekeeping from engine This module was at first in dwh. Then we decided the engine needs it too and copied it there. This change removes our copy and uses the one in the engine. Change-Id: I74b156b1941a7d48181b58ea6962fd02ab646fc8 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- D packaging/setup/ovirt_engine_setup/dwh/engine_db_timekeeping.py M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/core/single_etl.py 3 files changed, 4 insertions(+), 76 deletions(-) Approvals: Shirly Radco: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/36068 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74b156b1941a7d48181b58ea6962fd02ab646fc8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(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]: aaa: Changing RootDSE parsing for IPA to include defaultNami...
by oourfali＠redhat.com 14 Dec '14

14 Dec '14

Oved Ourfali has submitted this change and it was merged. Change subject: aaa: Changing RootDSE parsing for IPA to include defaultNamingContext ...................................................................... aaa: Changing RootDSE parsing for IPA to include defaultNamingContext As written in the bug, both freeIPA 3 and 4 have defaultNamingContext Change-Id: Iaba699c5cc40adcac793155724be7cbdfe3254df Topic: AAA Bug-Url: https://bugzilla.redhat.com/1171395 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/IPARootDSE.java M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/IPARootDSEAttributes.java M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/IPARootDSEContextMapper.java 3 files changed, 5 insertions(+), 5 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36140 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaba699c5cc40adcac793155724be7cbdfe3254df Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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: fix interfaces initialization for VM object
by ahadas＠redhat.com 14 Dec '14

14 Dec '14

Arik Hadas has submitted this change and it was merged. Change subject: core: fix interfaces initialization for VM object ...................................................................... core: fix interfaces initialization for VM object In the constructor of VM object we always set the interfaces to empty list. It is redundant as we get VmStatic object which is initialized with empty list. The problem with the redundant setting is that even if passing VmStatic instance initialized with interfaces, we were override it. Thus, this setting is removed. Change-Id: I4537f48ecfd7f3a42e96e345c24f085105361e02 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/36107 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4537f48ecfd7f3a42e96e345c24f085105361e02 Gerrit-PatchSet: 3 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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

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

14 Dec '14

Doron Fediuck 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: I4b1546c2885146dcd1623b6dd62e7b029f5047c4 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/36070 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b1546c2885146dcd1623b6dd62e7b029f5047c4 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master 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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: packaging: rename: change reports fqdn on engine conf
by sradco＠redhat.com 14 Dec '14

14 Dec '14

Shirly Radco has submitted this change and it was merged. Change subject: packaging: rename: change reports fqdn on engine conf ...................................................................... packaging: rename: change reports fqdn on engine conf Change-Id: I241417a9c0660e7798c1cf4eaac2fa192a0ffd32 Bug-Url: https://bugzilla.redhat.com/1170229 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- A packaging/setup/plugins/ovirt-engine-rename/ovirt-engine-reports/__init__.py A packaging/setup/plugins/ovirt-engine-rename/ovirt-engine-reports/engine.py 2 files changed, 147 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/36056 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I241417a9c0660e7798c1cf4eaac2fa192a0ffd32 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0