September 2015 - Engine-commits - oVirt List Archives

Change in ovirt-reports[master]: reports: fixed issues with network interfaces chart in br2a
by ydary＠redhat.com 02 Sep '15

02 Sep '15

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

1 0

Change in ovirt-engine[master]: core: Fix affinity group test which relies on null
by rgolan＠redhat.com 02 Sep '15

02 Sep '15

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

1 0

Change in jenkins[master]: Added openstack-juno repo for vdsm master el7 rpm sanity job.
by eedri＠redhat.com 02 Sep '15

02 Sep '15

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>

1 0

Change in ovirt-engine[master]: core: Change random data generation in SSCEDaoTest
by tnisan＠redhat.com 02 Sep '15

02 Sep '15

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

1 0

Change in ovirt-engine[master]: core: Implement toString() in StorageServerConnectionExtension
by tnisan＠redhat.com 02 Sep '15

02 Sep '15

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

1 0

Change in ovirt-hosted-engine-ha[master]: states, broker: Check for already starting VMs on other hosts
by msivak＠redhat.com 02 Sep '15

02 Sep '15

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

1 0

Change in jenkins[master]: Adding 3.6 branch jobs for ovirt-node
by tlitovsk＠redhat.com 02 Sep '15

02 Sep '15

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>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: restapi: Treat template watchdogs as template devices
by tnisan＠redhat.com 02 Sep '15

02 Sep '15

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

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Allow user to config CertificationValidityCheckTimeInH...
by tnisan＠redhat.com 02 Sep '15

02 Sep '15

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

1 0

Change in ovirt-engine[master]: webadmin: adding sync all networks button
by alkaplan＠redhat.com 02 Sep '15

02 Sep '15

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

1 0