February 2016 - Engine-commits - oVirt List Archives

Change in otopi[master]: packaging: spec: drop EL6 support
by didi＠redhat.com 17 Feb '16

17 Feb '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: packaging: spec: drop EL6 support ...................................................................... packaging: spec: drop EL6 support EL6 is not supported anymore for 4.0 Change-Id: Id788802d8916047a5f2706e048a8efa8dc8f3dc5 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M otopi.spec.in 1 file changed, 3 insertions(+), 66 deletions(-) Approvals: Sandro Bonazzola: Verified Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/53474 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id788802d8916047a5f2706e048a8efa8dc8f3dc5 Gerrit-PatchSet: 2 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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-engine[master]: core: Added cpu profiles permissions propagation
by rgolan＠redhat.com 17 Feb '16

17 Feb '16

Roy Golan has submitted this change and it was merged. Change subject: core: Added cpu profiles permissions propagation ...................................................................... core: Added cpu profiles permissions propagation Cpu Profile permissions will now propagate from parent objects of Cpu Profile. For example, If you have CpuProfileOperator permissions on a cluster that has Cpu Profile 'A' then you will automatically have CpuProfileOperator on 'A' and all the other Cpu Profiles on that cluster. The list of objects that Cpu Profile permissions progpagates from is: System -> Storage Domain -> Cluster -> Cpu Profile. This also solves the problem that admin user (That has SuperUser permissions on System) couldn't run any virtual machines because it needed to have specific permissions for a CpuProfile on it's cluster. Change-Id: Iaf5c142f8e563377cc262bf36ea5a44060b90d81 Bug-Url: https://bugzilla.redhat.com/1143869 Signed-off-by: Tomer Saban <tsaban(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/CpuProfileHelper.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/GetCpuProfilesByClusterIdQuery.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/profiles/CpuProfileDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/profiles/CpuProfileDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/profiles/CpuProfileDaoTest.java M packaging/dbscripts/cpu_profiles_sp.sql M packaging/dbscripts/create_views.sql 7 files changed, 66 insertions(+), 32 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Tomer Saban: Verified Allon Mureinik: Looks good to me, but someone else must approve Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/52727 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf5c142f8e563377cc262bf36ea5a44060b90d81 Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Enable fencing of previous host which HE VM was runnin...
by oourfali＠redhat.com 17 Feb '16

17 Feb '16

Oved Ourfali has submitted this change and it was merged. Change subject: core: Enable fencing of previous host which HE VM was running on ...................................................................... core: Enable fencing of previous host which HE VM was running on If host (for example hostA), which hosted engine VM was running on, crashes, then hosted engine VM is restarted on another host (hostB). When engine starts up on hostB, hostA is detected as non responsive, but it's not fenced automatically because fencing is disabled for 5 minutes after engine startup (config value DisableFenceAtStartupInSec). This causes issues: 1. HA VMs executed on hostA are not restarted on different host 2. If hostA was SPM, new SPM cannot be selected This patch enables fencing of hostA even during DisableFenceAtStartupInSec timeout to prevent above issues. Change-Id: Iff4368d02d3cbb71dd3cff9c2fc62586fa560a14 Bug-Url: https://bugzilla.redhat.com/1266099 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostedengine/PreviousHostedEngineHost.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/VdsNotRespondingTreatmentCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/BaseDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDaoTest.java A packaging/dbscripts/upgrade/03_06_2030-add-index-on-origin-in-vm-static.sql M packaging/dbscripts/vms_sp.sql 9 files changed, 141 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/53658 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff4368d02d3cbb71dd3cff9c2fc62586fa560a14 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Enable fencing of previous host which HE VM was runnin...
by mperina＠redhat.com 17 Feb '16

17 Feb '16

Martin Peřina has submitted this change and it was merged. Change subject: core: Enable fencing of previous host which HE VM was running on ...................................................................... core: Enable fencing of previous host which HE VM was running on If host (for example hostA), which hosted engine VM was running on, crashes, then hosted engine VM is restarted on another host (hostB). When engine starts up on hostB, hostA is detected as non responsive, but it's not fenced automatically because fencing is disabled for 5 minutes after engine startup (config value DisableFenceAtStartupInSec). This causes issues: 1. HA VMs executed on hostA are not restarted on different host 2. If hostA was SPM, new SPM cannot be selected This patch enables fencing of hostA even during DisableFenceAtStartupInSec timeout to prevent above issues. Change-Id: Iff4368d02d3cbb71dd3cff9c2fc62586fa560a14 Bug-Url: https://bugzilla.redhat.com/1266099 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostedengine/PreviousHostedEngineHost.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/VdsNotRespondingTreatmentCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDaoTest.java A packaging/dbscripts/upgrade/04_00_0250-add-index-on-origin-in-vm-static.sql M packaging/dbscripts/vms_sp.sql 8 files changed, 126 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Arik Hadas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/53436 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff4368d02d3cbb71dd3cff9c2fc62586fa560a14 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(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: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: backend: override default job property in ImportRepoImageCom...
by amureini＠redhat.com 17 Feb '16

17 Feb '16

Allon Mureinik has submitted this change and it was merged. Change subject: backend: override default job property in ImportRepoImageCommand ...................................................................... backend: override default job property in ImportRepoImageCommand This patch overrides the "storage" job property in ImportRepoImageCommand to be the target storage domain's name. Change-Id: Ie48f14e2cf34b8fcae409c76b95cc6d4ae01fd96 Bug-Url: https://bugzilla.redhat.com/1308642 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/repoimage/ImportRepoImageCommand.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/53649 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie48f14e2cf34b8fcae409c76b95cc6d4ae01fd96 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-3.6]: ovirt-live-3.6.3_rc3
by sbonazzo＠redhat.com 17 Feb '16

17 Feb '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-live-3.6.3_rc3 ...................................................................... ovirt-live-3.6.3_rc3 Change-Id: Icec9676e270b5624d99c446c5ebc612486665fb2 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/53643 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icec9676e270b5624d99c446c5ebc612486665fb2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 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[master]: core: update vm-jobs only in stats monitoring cycles
by ahadas＠redhat.com 17 Feb '16

17 Feb '16

Arik Hadas has submitted this change and it was merged. Change subject: core: update vm-jobs only in stats monitoring cycles ...................................................................... core: update vm-jobs only in stats monitoring cycles VM-jobs are reported by VDSM only as part of the VM statistics so there is no reason to call VmAnalyzer#updateVmJobs in monitoring cycles that do not update statistics. Change-Id: I30980920554fdd40fe8e117569372c9bfca269ae Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java 1 file changed, 1 insertion(+), 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/53585 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I30980920554fdd40fe8e117569372c9bfca269ae 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-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: build: ovirt-hosted-engine-ha-1.2.10
by sbonazzo＠redhat.com 17 Feb '16

17 Feb '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-1.2.10 ...................................................................... build: ovirt-hosted-engine-ha-1.2.10 Change-Id: I22818236aad030090b1b4902f1cfaa91ed58ca8b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 8 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified Martin Sivák: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/53631 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I22818236aad030090b1b4902f1cfaa91ed58ca8b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: storage: calling getStorageDomainStats to refresh symlinks
by sbonazzo＠redhat.com 17 Feb '16

17 Feb '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: storage: calling getStorageDomainStats to refresh symlinks ...................................................................... storage: calling getStorageDomainStats to refresh symlinks The symlinks under /rhev/data-center/ are not persisted on RHEV-H and so they have to be recreated on each reboot. getStorageDomainStats has the side effect of doing that. Change-Id: Ia83ccd345cdd29b973980497a088bcde3886ebba Bug-Url: https://bugzilla.redhat.com/1309211 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py 1 file changed, 21 insertions(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/53602 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia83ccd345cdd29b973980497a088bcde3886ebba Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: add event notification for number of LVs
by amureini＠redhat.com 17 Feb '16

17 Feb '16

Allon Mureinik has submitted this change and it was merged. Change subject: engine: add event notification for number of LVs ...................................................................... engine: add event notification for number of LVs This patch adds the ability to subscribe to the event of exceeding the threshold of the recommended number of LVs in a storage domain. Change-Id: I7e18299d8e9377d9a843dcb778f91f52e13a6b21 Bug-Url: https://bugzilla.redhat.com/1275182 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcEventNotificationUtils.java 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 3 files changed, 5 insertions(+), 0 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved; Passed CI tests Freddy Rolland: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, but someone else must approve Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/53583 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e18299d8e9377d9a843dcb778f91f52e13a6b21 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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: gerrit-hooks <automation(a)ovirt.org>

1 0