- Engine-commits - oVirt List Archives

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

Change in ovirt-engine[master]: restapi: Support skip LUN status check
by juan.hernandez＠redhat.com 19 Apr '16

19 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Support skip LUN status check ...................................................................... restapi: Support skip LUN status check Adding support for skipping the LUN status check in the REST API. Checking the status of the LUN is an heavy weight operation and this data is not always needed by the user. The default is `true` for backward compatibility. The parameter `report_status` is available both on getting the list of a host storages or a specific host storage: GET /hosts/{host:id}/storage GET /hosts/{host:id}/storage/{storage:id} For example, to skip the LUN status check, set `report_status` to false: (The `status` field of the LUN will not be present in the response) GET /hosts/{host:id}/storage/{storage:id};report_status=false <host_storage id="360014051136c20574f743bdbd28177fd"> <logical_units> <logical_unit id="360014051136c20574f743bdbd28177fd"> <lun_mapping>0</lun_mapping> <paths>1</paths> <product_id>lun0</product_id> <serial>SLIO-ORG_lun0_1136c205-74f7-43bd-bd28-177fd5ce6993</serial> <size>10737418240</size> <vendor_id>LIO-ORG</vendor_id> <volume_group_id>O9Du7I-RahN-ECe1-dZ1w-nh0b-64io-MNzIBZ</volume_group_id> </logical_unit> </logical_units> <type>iscsi</type> <host id="8bb5ade5-e988-4000-8b93-dbfc6717fe50"/> </host_storage> Change-Id: Ia772a0e7d9a67c8364e05c9f36a71a9111d1fdde Bug-Url: https://bugzilla.redhat.com/1308350 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/restapi/README.adoc M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostStorageResource.java 3 files changed, 86 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests Freddy Rolland: Verified -- To view, visit https://gerrit.ovirt.org/56301 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia772a0e7d9a67c8364e05c9f36a71a9111d1fdde Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ovirt-engine_upgrade_to_master: add missing key
by sbonazzo＠redhat.com 19 Apr '16

19 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine_upgrade_to_master: add missing key ...................................................................... ovirt-engine_upgrade_to_master: add missing key Add: OVESETUP_DWH_CORE/enable=bool:False DWH is now mandatory for the engine. For jenkins purposes, it's enough if we disable it. Change-Id: I6872c588c7aacfa30a4409639759b8006ab61457 Bug-Url: https://bugzilla.redhat.com/1318665 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M jobs/ovirt-engine_upgrade_to_master/setup.file.otopi 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Passed CI tests; Ready for merge Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/56321 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6872c588c7aacfa30a4409639759b8006ab61457 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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: do not test individual locking of pool, do test generi...
by alkaplan＠redhat.com 19 Apr '16

19 Apr '16

Alona Kaplan has submitted this change and it was merged. Change subject: core: do not test individual locking of pool, do test generic behavior ...................................................................... core: do not test individual locking of pool, do test generic behavior Change-Id: I926f10fe6fc3070c126b9e8f395a32ccb7575b69 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/macpool/MacPoolPerDc.java D backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/macpool/MacPoolLockingProxyTest.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/lock/LockedObjectFactory.java A backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/lock/LockedObjectFactoryTest.java 4 files changed, 209 insertions(+), 256 deletions(-) Approvals: Martin Mucha: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/53132 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I926f10fe6fc3070c126b9e8f395a32ccb7575b69 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0