July 2016 - Engine-commits - oVirt List Archives

Change in jenkins[master]: ovirt-reports: keep only 3.6 builds
by sbonazzo＠redhat.com 14 Jul '16

14 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-reports: keep only 3.6 builds ...................................................................... ovirt-reports: keep only 3.6 builds ovirt-reports is not used anymore since 4.0. Keeping it on 3.6 for el6 and el7 only. Change-Id: I8d73bccff1a85652ebfaa0d31bb95b12b904134c Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-reports/ovirt-reports_create-rpms.yaml 1 file changed, 0 insertions(+), 23 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60715 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d73bccff1a85652ebfaa0d31bb95b12b904134c Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: Adding dao tests to std ci
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: Adding dao tests to std ci ...................................................................... Adding dao tests to std ci Basically this patch is all about running dao tests inside a mock. For this, a postgresql DB should be installed and run inside the mock. Added postgres packages to .packages files Created a new script, dao-tests.sh. This script is being called only if files that are relevant to DB had been changed. The script will pick a port randomly and will update pg_hba.conf file to allow DB access. All the rest is the same as how the tests were running when not in std ci. Change-Id: I19252210d4692987fa82b323ff2f71543404ea46 Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M automation/check-merged.packages M automation/check-merged.packages.el6 M automation/check-merged.sh M automation/check-patch.packages M automation/check-patch.packages.el6 M automation/check-patch.sh A automation/dao-tests.sh 7 files changed, 118 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Barak Korren: Looks good to me, but someone else must approve Gil Shinar: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60578 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I19252210d4692987fa82b323ff2f71543404ea46 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: engine: add isMacInRange to MacPool
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: engine: add isMacInRange to MacPool ...................................................................... engine: add isMacInRange to MacPool Change-Id: Ib03ee34496fe897a2981cc8e7ac9ddc1226c1174 Bug-Url: https://bugzilla.redhat.com/1209795 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/macpool/DelegatingMacPoolDecorator.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/macpool/MacPool.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/macpool/MacPoolUsingRanges.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/macpool/MacsStorage.java 4 files changed, 15 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/60632 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib03ee34496fe897a2981cc8e7ac9ddc1226c1174 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: engine: report VMs with external MAC on VM import
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: engine: report VMs with external MAC on VM import ...................................................................... engine: report VMs with external MAC on VM import Change-Id: Id90802b81018194523fccdffe0eff5b764e902a2 Bug-Url: https://bugzilla.redhat.com/1209795 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandBase.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/ExternalVmMacsFinder.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/exportimport/ImportVMFromConfigurationCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandTest.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/vm/ExternalVmMacsFinderTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java 6 files changed, 197 insertions(+), 17 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/60633 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id90802b81018194523fccdffe0eff5b764e902a2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: engine: Enable subscribing to external MAC warning
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: engine: Enable subscribing to external MAC warning ...................................................................... engine: Enable subscribing to external MAC warning Change-Id: I2d9a251a977400464dd5420db130a96f0449fe0b Bug-Url: https://bugzilla.redhat.com/1209795 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcEventNotificationUtils.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties A packaging/dbscripts/upgrade/04_00_0830_external_mac_events.sql 5 files changed, 13 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/60634 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d9a251a977400464dd5420db130a96f0449fe0b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ovirt-vmconsole: move from fc22 to fc24
by sbonazzo＠redhat.com 14 Jul '16

14 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-vmconsole: move from fc22 to fc24 ...................................................................... ovirt-vmconsole: move from fc22 to fc24 - drop fc22 support since gone EOL - move master to fc24 Change-Id: Ib41a630a429e6ac34069e1f6c0d821959bce3790 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-vmconsole/ovirt-vmconsole_create-rpms.yaml 1 file changed, 5 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60712 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib41a630a429e6ac34069e1f6c0d821959bce3790 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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[ovirt-engine-4.0]: core: prevent floating vms in unknown status
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: core: prevent floating vms in unknown status ...................................................................... core: prevent floating vms in unknown status On non-responsive treatment, update VMs to unknown status only if they run on some host. This check is done in the database level, with the protection of the database lock. This way, if the monitoring has just updated the VM to down state we will not end up with a VM with unknown status and run_on_vds = NULL. This is not the ideal solution but it seems to cover the most problematic scenario and yet simple. Change-Id: I48c205e9228a32822148332af13c89e51cc80afa Bug-Url: https://bugzilla.redhat.com/1354494 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M packaging/dbscripts/vms_sp.sql 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/60653 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I48c205e9228a32822148332af13c89e51cc80afa Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: switch multiple vms to unknown in one update
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: core: switch multiple vms to unknown in one update ...................................................................... core: switch multiple vms to unknown in one update Update multiple VMs to unknown status in a single db access. Change-Id: I5a02dfc98da704616f0e00840195e6252fe62609 Bug-Url: https://bugzilla.redhat.com/1354494 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/VdsManager.java M packaging/dbscripts/vms_sp.sql 5 files changed, 55 insertions(+), 7 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/60652 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a02dfc98da704616f0e00840195e6252fe62609 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: packaging: setup: Verify Postgresql server version
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: setup: Verify Postgresql server version ...................................................................... packaging: setup: Verify Postgresql server version Verify that server and client versions are equal. Change-Id: Id316925392c712a4e5471d3d9c6625f4b10194c3 Bug-Url: https://bugzilla.redhat.com/1331168 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 27ef7ea7adba5a001e7e9bf167b4e1634adb3ba6) --- M packaging/setup/ovirt_engine_setup/engine_common/database.py M packaging/setup/ovirt_engine_setup/engine_common/postgres.py 2 files changed, 30 insertions(+), 4 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/60705 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id316925392c712a4e5471d3d9c6625f4b10194c3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.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 jenkins[master]: add 4.0 hosted engine tests
by eedri＠redhat.com 14 Jul '16

14 Jul '16

Eyal Edri has submitted this change and it was merged. Change subject: add 4.0 hosted engine tests ...................................................................... add 4.0 hosted engine tests Change-Id: I46558fe0868861c571fb97c9f0658eea3a00cfe8 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/ovirt/system-tests.yaml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Lev Veyde: Looks good to me Eyal Edri: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60703 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I46558fe0868861c571fb97c9f0658eea3a00cfe8 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0