Change in ovirt-reports[master]: reports: fixed issues with network interfaces chart in br2a
by ydary@redhat.com
Yaniv Dary has submitted this change and it was merged.
Change subject: reports: fixed issues with network interfaces chart in br2a
......................................................................
reports: fixed issues with network interfaces chart in br2a
Also fixed disk\nics charts in br10a
Change-Id: Iad4661c419a3247396ac490599107dc3d220bcba
Signed-off-by: Yaniv Dary <ydary(a)redhat.com>
Bug-Url: https://bugzilla.redhat.com/1209999
---
M packaging/ovirt-reports/resources/Reports/Trend/Hosts/single_host_resource_br2a_files/single_host_resource_br2a_jrxml.data
M packaging/ovirt-reports/resources/Reports/Trend/Virtual_machines/single_vm_resources_over_time_BR10A_files/single_vm_resources_over_time_BR10A_jrxml.data
2 files changed, 74 insertions(+), 46 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Yaniv Dary: Verified; Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/45609
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iad4661c419a3247396ac490599107dc3d220bcba
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 8 months
Change in ovirt-engine[master]: core: Fix affinity group test which relies on null
by rgolan@redhat.com
Roy Golan has submitted this change and it was merged.
Change subject: core: Fix affinity group test which relies on null
......................................................................
core: Fix affinity group test which relies on null
Because AffinityGroups always returns a list and never null the test
should check if the list is empty instead of null.
Change-Id: I7ee327ad10b597568c05f06ba4b7a6a73eb48104
Bug-Url: https://bugzilla.redhat.com/1251742
Signed-off-by: Roman Mohr <rmohr(a)redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AffinityGroupDaoTest.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Roman Mohr: Verified
Jenkins CI: Passed CI tests
Roy Golan: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/45647
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7ee327ad10b597568c05f06ba4b7a6a73eb48104
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 8 months
Change in jenkins[master]: Added openstack-juno repo for vdsm master el7 rpm sanity job.
by eedri@redhat.com
Eyal Edri has submitted this change and it was merged.
Change subject: Added openstack-juno repo for vdsm master el7 rpm sanity job.
......................................................................
Added openstack-juno repo for vdsm master el7 rpm sanity job.
This repo addition was requested by phoracek since it contains
openvswitch that is needed by new vdsm openvswitch hook.
Change-Id: I6ef11043fb931797be748e4eb839eba35348126b
Signed-off-by: Anton Marchukov <amarchuk(a)redhat.com>
---
M jobs/confs/projects/vdsm/vdsm_install-rpm-sanity.yaml
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me
Eyal Edri: Ready for merge
Anton Marchukov: Verified; Ready for review
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/45633
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6ef11043fb931797be748e4eb839eba35348126b
Gerrit-PatchSet: 2
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Anton Marchukov <amarchuk(a)redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
9 years, 8 months
Change in ovirt-engine[master]: core: Change random data generation in SSCEDaoTest
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: core: Change random data generation in SSCEDaoTest
......................................................................
core: Change random data generation in SSCEDaoTest
Change random data generation StorageServerConnectionExtensionDaoTest to
use only alphanumeric characters
Change-Id: I7b76e13ec1d0d84426b8de45857840fb1a2b0dea
Bug-Url: https://bugzilla.redhat.com/1239266
Signed-off-by: Tal Nisan <tnisan(a)redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageServerConnectionExtensionDaoTest.java
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
Tal Nisan: Verified; Passed CI tests
Allon Mureinik: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/45579
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7b76e13ec1d0d84426b8de45857840fb1a2b0dea
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 8 months
Change in ovirt-engine[master]: core: Implement toString() in StorageServerConnectionExtension
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: core: Implement toString() in StorageServerConnectionExtension
......................................................................
core: Implement toString() in StorageServerConnectionExtension
Change-Id: Idc4060042b6847366cc764158a92b0ca79e46ce3
Bug-Url: https://bugzilla.redhat.com/1239266
Signed-off-by: Tal Nisan <tnisan(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage/StorageServerConnectionExtension.java
1 file changed, 10 insertions(+), 0 deletions(-)
Approvals:
Tal Nisan: Verified; Passed CI tests
Allon Mureinik: Looks good to me, approved
Objections:
Jenkins CI: Failed CI tests
--
To view, visit https://gerrit.ovirt.org/45578
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idc4060042b6847366cc764158a92b0ca79e46ce3
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 8 months
Change in ovirt-hosted-engine-ha[master]: states, broker: Check for already starting VMs on other hosts
by msivak@redhat.com
Martin Sivák has submitted this change and it was merged.
Change subject: states, broker: Check for already starting VMs on other hosts
......................................................................
states, broker: Check for already starting VMs on other hosts
When two or more hosts try to take over when one host goes down, only
one of them can start the hosted engine VM. The others will fail to
acquire the storage lock.
The health monitor now detects when this is the case, so that the host,
which could not acquire the lock, can remove the VM which it could not
start and go back to the down state.
Change-Id: Ia5d321039801a9868f2b1232ce0acfdfe756b477
Bug-Url: https://bugzilla.redhat.com/1207634
Signed-off-by: Roman Mohr <rmohr(a)redhat.com>
---
M ovirt_hosted_engine_ha/agent/states.py
M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py
M ovirt_hosted_engine_ha/lib/engine.py
M ovirt_hosted_engine_ha/lib/util.py
4 files changed, 25 insertions(+), 4 deletions(-)
Approvals:
Martin Sivák: Looks good to me, approved
Roman Mohr: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/45592
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia5d321039801a9868f2b1232ce0acfdfe756b477
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 8 months
Change in jenkins[master]: Adding 3.6 branch jobs for ovirt-node
by tlitovsk@redhat.com
Anatoly Litovsky has submitted this change and it was merged.
Change subject: Adding 3.6 branch jobs for ovirt-node
......................................................................
Adding 3.6 branch jobs for ovirt-node
Change-Id: Ie4de3bfa3844fa3eee87d894a6b96b3322962791
Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com>
---
M jobs/confs/projects/ovirt-node/ovirt-node-job.yaml
1 file changed, 9 insertions(+), 3 deletions(-)
Approvals:
Fabian Deutsch: Looks good to me
Jenkins CI: Passed CI tests
Anatoly Litovsky: Verified; Passed CI tests; Ready for merge
--
To view, visit https://gerrit.ovirt.org/45643
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie4de3bfa3844fa3eee87d894a6b96b3322962791
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: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
9 years, 8 months
Change in ovirt-engine[ovirt-engine-3.6]: restapi: Treat template watchdogs as template devices
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: restapi: Treat template watchdogs as template devices
......................................................................
restapi: Treat template watchdogs as template devices
When support for WildFly was intruduced a series of changes were needed
to move the DELETE methods from collections to resources. The change to
the watchdogs resources introduced a regression in the treatment of
template watchdogs: they are treated as VM resources instead of template
resources. This patch fixes that issue.
Change-Id: I8c8dbad721446e71bd6dc135af37ad42fd8518bf
Bug-Url: https://bugzilla.redhat.com/1258224
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
(cherry picked from commit 550d2a9eb73ab6fa5fd4239ff43a9a6e9eb253cf)
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendWatchdogsResource.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/45619
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8c8dbad721446e71bd6dc135af37ad42fd8518bf
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Juan Hernandez <juan.hernandez(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: automation(a)ovirt.org
9 years, 8 months
Change in ovirt-engine[ovirt-engine-3.6]: core: Allow user to config CertificationValidityCheckTimeInH...
by tnisan@redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: core: Allow user to config CertificationValidityCheckTimeInHours
......................................................................
core: Allow user to config CertificationValidityCheckTimeInHours
Change-Id: Ifa356d10dc70b86781a451bfdad6dae6dd1ebcc9
Bug-Url: https://bugzilla.redhat.com/1257905
Signed-off-by: Moti Asayag <masayag(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CertificationValidityChecker.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
M packaging/etc/engine-config/engine-config.properties
4 files changed, 11 insertions(+), 2 deletions(-)
Approvals:
Eli Mesika: Looks good to me, approved
Jenkins CI: Passed CI tests
Moti Asayag: Verified
--
To view, visit https://gerrit.ovirt.org/45625
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifa356d10dc70b86781a451bfdad6dae6dd1ebcc9
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 8 months
Change in ovirt-engine[master]: webadmin: adding sync all networks button
by alkaplan@redhat.com
Alona Kaplan has submitted this change and it was merged.
Change subject: webadmin: adding sync all networks button
......................................................................
webadmin: adding sync all networks button
Adding new button to SubTabHostInterface. The
button will allow to sync all host's networks at
once by calling SyncAllHostNetworks Vdc command
Change-Id: Ic8ade04016cfe0cec9cb8190637dac7983ec262a
Bug-Url: https://bugzilla.redhat.com/1194297
Signed-off-by: Eliraz Levi <elevi(a)redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/help/HelpTag.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostInterfaceListModel.java
M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/host/SubTabHostInterfaceView.java
5 files changed, 83 insertions(+), 9 deletions(-)
Approvals:
Eliraz Levi: Verified
Alona Kaplan: Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/42024
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic8ade04016cfe0cec9cb8190637dac7983ec262a
Gerrit-PatchSet: 19
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 8 months