April 2016 - Engine-commits - Ovirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: userportal: use the default host that was set in vm in template

by tnisan＠redhat.com

Tal Nisan has submitted this change and it was merged. Change subject: userportal: use the default host that was set in vm in template ...................................................................... userportal: use the default host that was set in vm in template When creating a template from a vm that has a default host specified the host was not passed to the created template. The fix was in two layers: 1. UI - UserPortalListModel - populate the default host in the newly created template. 2. Backend - AddVmTemplateCommand - changed the permission check so that the user will be able to create a template from a vm with default host. Added tests. Change-Id: I65c7735e59b2056c341b35eaa9d6b840dc6608d7 Bug-Url: https://bugzilla.redhat.com/1284472 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> (cherry picked from commit d2f5165) --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmTemplateCommandTest.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java 3 files changed, 53 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/56250 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65c7735e59b2056c341b35eaa9d6b840dc6608d7 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

Change in ovirt-dwh[ovirt-engine-dwh-3.6]: build: ovirt-engine-dwh-3.6.6

by Shirly Radco

Shirly Radco has submitted this change and it was merged. Change subject: build: ovirt-engine-dwh-3.6.6 ...................................................................... build: ovirt-engine-dwh-3.6.6 updated release notes Change-Id: I7a454f0e74559e3b8981c283d459cb5b8d6b6f52 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M ovirt-engine-dwh.spec.in M version.mak 2 files changed, 9 insertions(+), 3 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56339 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7a454f0e74559e3b8981c283d459cb5b8d6b6f52 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-3.6 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

Change in ovirt-engine[master]: core: update vm statistics through vm-manager

by ahadas＠redhat.com

Arik Hadas has submitted this change and it was merged. Change subject: core: update vm statistics through vm-manager ...................................................................... core: update vm statistics through vm-manager Changed places that updated VmStatistics directly using the DAO with calls to VmManager, to enable caching of the statistics in the VmManager instance. Change-Id: I827fc2929082467a52139e51c226fa3249e093b2 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- 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/SetVmStatusVDSCommand.java 2 files changed, 2 insertions(+), 10 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/56319 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I827fc2929082467a52139e51c226fa3249e093b2 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: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

Change in ovirt-engine[master]: core: remove vm-statistics dao from vm-anlyzer tests

by ahadas＠redhat.com

Arik Hadas has submitted this change and it was merged. Change subject: core: remove vm-statistics dao from vm-anlyzer tests ...................................................................... core: remove vm-statistics dao from vm-anlyzer tests VmStatisticsDao is not being used in the tests of VmAnalyzer, thus it is removed from VmAnalyzerTest. Change-Id: Ic7a7d680f54df4142008a32b0bad80f13e89b5d4 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerTest.java 1 file changed, 0 insertions(+), 9 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/56318 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic7a7d680f54df4142008a32b0bad80f13e89b5d4 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: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

Change in ovirt-engine-sdk[master]: Makes sure SDK ignores case when handling enum types for att...

by juan.hernandez＠redhat.com

Juan Hernandez has submitted this change and it was merged. Change subject: Makes sure SDK ignores case when handling enum types for attributes ...................................................................... Makes sure SDK ignores case when handling enum types for attributes Change-Id: I1b7ec6d4e0727c2aaba39ba24c52c7b62a7621a3 Signed-off-by: karimb <karimboumedhel(a)gmail.com> --- M generator/src/main/java/org/ovirt/sdk/python/ReadersGenerator.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Karim Boumedhel: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56226 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b7ec6d4e0727c2aaba39ba24c52c7b62a7621a3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Karim Boumedhel <karimboumedhel(a)gmail.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Karim Boumedhel <karimboumedhel(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

Change in jenkins[master]: Adding installation of package required for livecd-creator

by eedri＠redhat.com

Eyal Edri has submitted this change and it was merged. Change subject: Adding installation of package required for livecd-creator ...................................................................... Adding installation of package required for livecd-creator it seems sssd-client is required by livecd-creator Due to the fact the livecd-creator is mostly unmantained Adding the package externally Change-Id: Iec4f7ff359267f022b1e83048b1c4987e1c8e717 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M jobs/confs/shell-scripts/build-local-ovirt-node-iso.sh 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Looks good to me Jenkins CI: Passed CI tests Ryan Barry: Looks good to me Anatoly Litovsky: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/56304 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iec4f7ff359267f022b1e83048b1c4987e1c8e717 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.6]: packaging: engine-backup: Backup ovirt-vmconsole conf and pki

by tnisan＠redhat.com

Tal Nisan has submitted this change and it was merged. Change subject: packaging: engine-backup: Backup ovirt-vmconsole conf and pki ...................................................................... packaging: engine-backup: Backup ovirt-vmconsole conf and pki Change-Id: I8512db986297fb463b2643602a430a2d654a1774 Bug-Url: https://bugzilla.redhat.com/1328036 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit f9a20d402204b8e1a1e3a6c089230623faaa7d2d) --- M packaging/bin/engine-backup.sh.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/56276 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8512db986297fb463b2643602a430a2d654a1774 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

Change in ovirt-engine[master]: packaging: setup: Use getDBConfig for the engine

by sbonazzo＠redhat.com

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Use getDBConfig for the engine ...................................................................... packaging: setup: Use getDBConfig for the engine Change-Id: I8a391b5286093cb53e49ba499c3f41c224e6af1e Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/database.py 1 file changed, 5 insertions(+), 28 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/56298 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a391b5286093cb53e49ba499c3f41c224e6af1e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine 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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

Change in ovirt-release[ovirt-3.6]: packaging: spec: drop dep on main package

by sbonazzo＠redhat.com

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: drop dep on main package ...................................................................... packaging: spec: drop dep on main package Dropping dependency on main package allowing to install -snapshot subpackage alone. Change-Id: Ib4046b5f27252f30a3bebaa462e7358fbb8326c0 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-release36.spec.in 1 file changed, 22 insertions(+), 3 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56294 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib4046b5f27252f30a3bebaa462e7358fbb8326c0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

Change in ovirt-release[ovirt-4.0]: packaging: spec: drop dep on main package

by sbonazzo＠redhat.com

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: drop dep on main package ...................................................................... packaging: spec: drop dep on main package Dropping dependency on main package allowing to install -snapshot subpackage alone. Change-Id: Ib4046b5f27252f30a3bebaa462e7358fbb8326c0 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 16 insertions(+), 3 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56293 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib4046b5f27252f30a3bebaa462e7358fbb8326c0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

8 years, 8 months

1
0
0 / 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

Engine-commits April 2016