- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: tools: added migration policies to engine-config
by tjelinek＠redhat.com 10 May '16

10 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: tools: added migration policies to engine-config ...................................................................... tools: added migration policies to engine-config Added a possibility to configure the migration policies using engine config. The dependency to jackson is there to make sure the provided policy is valid and also to give an example policy. Change-Id: I2f84034348b0af963cc63dd1cb2470d9b45c28c8 Bug-Url: https://bugzilla.redhat.com/1252426 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- A backend/manager/tools/src/main/java/org/ovirt/engine/core/config/entity/helper/MigrationPoliciesValueHelper.java M backend/manager/tools/src/main/modules/org/ovirt/engine/core/tools/main/module.xml A backend/manager/tools/src/test/java/org/ovirt/engine/core/config/entity/helper/MigrationPoliciesValueHelperTest.java M packaging/etc/engine-config/engine-config.properties 4 files changed, 86 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57305 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2f84034348b0af963cc63dd1cb2470d9b45c28c8 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in jenkins[master]: std ci: increase limit of parallel runs to 6
by eedri＠redhat.com 10 May '16

10 May '16

Eyal Edri has submitted this change and it was merged. Change subject: std ci: increase limit of parallel runs to 6 ...................................................................... std ci: increase limit of parallel runs to 6 - so far we let only 3 instances of a job to run in parallel on different slaves. This creates a status of idle slaves for no apparent reason since there is no real problem in utilizing all slaves to run more instances of the same job. This patch increase the amount to 6 (double), but we might increase it even more if we see the need and we have the resources for it. Change-Id: I755d1351ac6a7d579fc8e455fa4453283f35e9c4 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/yaml/templates/standard-stage.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eyal Edri: Verified; Ready for merge Jenkins CI: Passed CI tests David Caro: Looks good to me -- To view, visit https://gerrit.ovirt.org/57306 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I755d1351ac6a7d579fc8e455fa4453283f35e9c4 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: rename: Fix SSO client callback prefix on engine ...
by sbonazzo＠redhat.com 10 May '16

10 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: rename: Fix SSO client callback prefix on engine rename ...................................................................... packaging: rename: Fix SSO client callback prefix on engine rename Fix SSO client callback prefix after ovirt engine rename. ovirt-engine-rename needs to update the database to change the client callback prefix for SSO with the new engine fqdn. Change-Id: Ib8411eb1a2a188991aeea0f09472093aac716945 Bug-Url: https://bugzilla.redhat.com/1330168 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M packaging/dbscripts/sso_clients_sp.sql M packaging/setup/plugins/ovirt-engine-rename/ovirt-engine/__init__.py A packaging/setup/plugins/ovirt-engine-rename/ovirt-engine/sso.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/sso.py 4 files changed, 91 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Ravi Nori: Verified Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/56845 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib8411eb1a2a188991aeea0f09472093aac716945 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[master]: packaging: setup: Allow choosing scale
by Shirly Radco 10 May '16

10 May '16

Shirly Radco has submitted this change and it was merged. Change subject: packaging: setup: Allow choosing scale ...................................................................... packaging: setup: Allow choosing scale Change-Id: I8e399cd3a9d9e2730b1e2e5f750308b8ac5eccb6 Bug-Url: https://bugzilla.redhat.com/1328805 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/dwh/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/core/__init__.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/core/scale.py 3 files changed, 165 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/57238 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e399cd3a9d9e2730b1e2e5f750308b8ac5eccb6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[master]: packaging: setup: Copy db creds to engine
by Shirly Radco 10 May '16

10 May '16

Shirly Radco has submitted this change and it was merged. Change subject: packaging: setup: Copy db creds to engine ...................................................................... packaging: setup: Copy db creds to engine Bug-Url: https://bugzilla.redhat.com/1318665 Change-Id: Ie4a4c7e08015f5c6f60bb9e39724f46274671754 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/dwh/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/core/__init__.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/core/config.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/core/remote_engine.py 4 files changed, 223 insertions(+), 90 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/56465 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie4a4c7e08015f5c6f60bb9e39724f46274671754 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: prevent host in maintenance while vm runs on it in db
by ahadas＠redhat.com 10 May '16

10 May '16

Arik Hadas has submitted this change and it was merged. Change subject: core: prevent host in maintenance while vm runs on it in db ...................................................................... core: prevent host in maintenance while vm runs on it in db Before switching a host to maintenance, we set its status to prepare-to- maintenance and migrate the VMs that run on it. Only when no VMs run on the host, we switch it to maintenance. The problem is that we count on the number of VMs that are reported by the host. It is racy because we might get this report from VDSM before the Down event arrived or after the Down event has been ignored. This patch refine the check that is done before switching a host to maintenance to check that no VM is running on the host in the database as well. We check the database only if VDSM reports that no VM runs on it so it should be fine performance wise. Change-Id: Ia953d71bb0b036ed0e1e65527c650e955a031fa1 Bug-Url: https://bugzilla.redhat.com/1325938 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDynamicDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDynamicDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDynamicDaoTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/MonitoringStrategyFactory.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VirtMonitoringStrategy.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/MultipleServicesMonitoringStrategyTest.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VirtMonitoringStrategyTest.java M packaging/dbscripts/vms_sp.sql 8 files changed, 66 insertions(+), 6 deletions(-) Approvals: Jenkins CI: Passed CI tests Arik Hadas: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57230 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia953d71bb0b036ed0e1e65527c650e955a031fa1 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: appliance: auto-accepting defaults on unattended engine-setup
by stirabos＠redhat.com 10 May '16

10 May '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: appliance: auto-accepting defaults on unattended engine-setup ...................................................................... appliance: auto-accepting defaults on unattended engine-setup Since https://gerrit.ovirt.org/55479 otopi provide a mechanism to auto-accept the default value of a question if present. Triggering it on unattended engine-setup execution on the engine appliance. Change-Id: If94911b2484e87a362ac4dc0f874c3b2137ea191 Bug-Url: https://bugzilla.redhat.com/1332927 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/vm/cloud_init.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57236 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If94911b2484e87a362ac4dc0f874c3b2137ea191 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Add support to use system wide CA certificate store
by omachace＠redhat.com 10 May '16

10 May '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add support to use system wide CA certificate store ...................................................................... Add support to use system wide CA certificate store This patch add support to use system wide CA certificate store. In case user don't specify `ca_file` parameter, SDK will use system CA store. Change-Id: I12f0ecd724631d4e449fc49d00a382764d0b417d Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1326386 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M sdk/README.adoc M sdk/lib/ovirtsdk4/__init__.py M sdk/tests/test_connection_create.py 3 files changed, 4 insertions(+), 15 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57291 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I12f0ecd724631d4e449fc49d00a382764d0b417d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: make MonitoringStrategyFactory singleton
by ahadas＠redhat.com 10 May '16

10 May '16

Arik Hadas has submitted this change and it was merged. Change subject: core: make MonitoringStrategyFactory singleton ...................................................................... core: make MonitoringStrategyFactory singleton MonitoringStrategyFactory used to have static fields/utility methods. This patch changes it to be Singleton so it would be possible to inject DAOs for it. Change-Id: I010c45436aa9dc8b0bf82d6a56a59f99e86c94cd Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/VdsNotRespondingTreatmentCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.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/monitoring/MonitoringStrategyFactory.java 4 files changed, 52 insertions(+), 22 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/57229 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I010c45436aa9dc8b0bf82d6a56a59f99e86c94cd Gerrit-PatchSet: 4 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: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: aligned the "no migration policy" with the latest VDSM...
by tjelinek＠redhat.com 10 May '16

10 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: core: aligned the "no migration policy" with the latest VDSM setting ...................................................................... core: aligned the "no migration policy" with the latest VDSM setting The vdsm's configuration is now that 2 parallel migrations are allowed. This patch alignes the "no migration policy" with this setting. Change-Id: I9a8a6c99b0ca9709186ddd3454e8dc7535789d17 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/migration/NoMigrationPolicy.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57261 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9a8a6c99b0ca9709186ddd3454e8dc7535789d17 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0