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 19 Apr '16

19 Apr '16

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>

1 0

Change in ovirt-dwh[ovirt-engine-dwh-3.6]: build: ovirt-engine-dwh-3.6.6
by Shirly Radco 19 Apr '16

19 Apr '16

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>

1 0

Change in ovirt-engine[master]: core: update vm statistics through vm-manager
by ahadas＠redhat.com 19 Apr '16

19 Apr '16

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>

1 0

Change in ovirt-engine[master]: core: remove vm-statistics dao from vm-anlyzer tests
by ahadas＠redhat.com 19 Apr '16

19 Apr '16

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>

1 0

Change in ovirt-engine-sdk[master]: Makes sure SDK ignores case when handling enum types for att...
by juan.hernandez＠redhat.com 19 Apr '16

19 Apr '16

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>

1 0

Change in jenkins[master]: Adding installation of package required for livecd-creator
by eedri＠redhat.com 19 Apr '16

19 Apr '16

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>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: packaging: engine-backup: Backup ovirt-vmconsole conf and pki
by tnisan＠redhat.com 19 Apr '16

19 Apr '16

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>

1 0

Change in ovirt-engine[master]: packaging: setup: Use getDBConfig for the engine
by sbonazzo＠redhat.com 19 Apr '16

19 Apr '16

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>

1 0

Change in ovirt-release[ovirt-3.6]: packaging: spec: drop dep on main package
by sbonazzo＠redhat.com 19 Apr '16

19 Apr '16

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>

1 0

Change in ovirt-release[ovirt-4.0]: packaging: spec: drop dep on main package
by sbonazzo＠redhat.com 19 Apr '16

19 Apr '16

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>

1 0