Change in jenkins[master]: ovirt-reports: keep only 3.6 builds
by sbonazzo＠redhat.com
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>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-3.6]: Adding dao tests to std ci
by tnisan＠redhat.com
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>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0]: engine: add isMacInRange to MacPool
by tnisan＠redhat.com
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>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0]: engine: report VMs with external MAC on VM import
by tnisan＠redhat.com
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>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0]: engine: Enable subscribing to external MAC warning
by tnisan＠redhat.com
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>
8 years, 4 months
Change in jenkins[master]: ovirt-vmconsole: move from fc22 to fc24
by sbonazzo＠redhat.com
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>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0]: core: prevent floating vms in unknown status
by tnisan＠redhat.com
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>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0]: core: switch multiple vms to unknown in one update
by tnisan＠redhat.com
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>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0]: packaging: setup: Verify Postgresql server version
by tnisan＠redhat.com
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>
8 years, 4 months
Change in jenkins[master]: add 4.0 hosted engine tests
by eedri＠redhat.com
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>
8 years, 4 months