- Engine-commits - oVirt List Archives

Change in ovirt-release[master]: node: Pull updates from the regular ovirt repositories
by fabiand＠redhat.com 27 May '16

27 May '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Pull updates from the regular ovirt repositories ...................................................................... node: Pull updates from the regular ovirt repositories Previously jenkins repositories got injected. Change-Id: I127b3e0dc7ece602a29e0116f01570d930371fbf Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58173 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I127b3e0dc7ece602a29e0116f01570d930371fbf Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Add imgbase-motd.service
by fabiand＠redhat.com 27 May '16

27 May '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Add imgbase-motd.service ...................................................................... node: Add imgbase-motd.service To generate the motd. Change-Id: Id38bc18a3e20e9ef18a5ba4ad577547c500d87ac Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58109 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id38bc18a3e20e9ef18a5ba4ad577547c500d87ac Gerrit-PatchSet: 3 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Use sshd.service
by fabiand＠redhat.com 27 May '16

27 May '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Use sshd.service ...................................................................... node: Use sshd.service Don't use socket, see bug. Change-Id: I8bb6bc6050a93dc407e40842fe6674c7fee69b17 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1327332 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58108 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8bb6bc6050a93dc407e40842fe6674c7fee69b17 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: sla: Mark CPU_OVERCOMMIT_DURATION as editable in balancing u...
by rgolan＠redhat.com 26 May '16

26 May '16

Roy Golan has submitted this change and it was merged. Change subject: sla: Mark CPU_OVERCOMMIT_DURATION as editable in balancing units ...................................................................... sla: Mark CPU_OVERCOMMIT_DURATION as editable in balancing units This patch re-adds the CPU_OVERCOMMIT_DURATION field to the scheduling unit annotations to make it visible in the UI and editable in the scheduling policy dialogs. Change-Id: I215cbc9bef44771259c5747a24b01321a3b0e005 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1339640 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/EvenDistributionBalancePolicyUnit.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PowerSavingBalancePolicyUnit.java 2 files changed, 21 insertions(+), 18 deletions(-) Approvals: Martin Sivák: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved Jenny Tokar: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/58064 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I215cbc9bef44771259c5747a24b01321a3b0e005 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[master]: Add hostname in ovirt-ha-agent emails
by msivak＠redhat.com 26 May '16

26 May '16

Martin Sivák has submitted this change and it was merged. Change subject: Add hostname in ovirt-ha-agent emails ...................................................................... Add hostname in ovirt-ha-agent emails Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1324673 Change-Id: I6489ea2e6911360fa19915b347f9a2deffb107f8 Signed-off-by: mshira <mshira(a)redhat.com> --- M ovirt_hosted_engine_ha/broker/notifications/state_transition.txt 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Sivák: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/57495 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6489ea2e6911360fa19915b347f9a2deffb107f8 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Maximov Maximov <mshira(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Maximov Maximov <mshira(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Stanislav Mikhalevich <msio57(a)gmail.com> Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: userportal: New VM dialog offers each VM template twice
by tnisan＠redhat.com 26 May '16

26 May '16

Tal Nisan has submitted this change and it was merged. Change subject: userportal: New VM dialog offers each VM template twice ...................................................................... userportal: New VM dialog offers each VM template twice The decision to show also the "latest" templates has to be done according to the fact if the VM is stateless or not. This has been implemented and used in webadmin but forgotten to be used it for userportal. Change-Id: Iffb7f3d172cf769f9bef78a4dd067b30e952282b Bug-Url: https://bugzilla.redhat.com/1293154 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UserPortalNewVmModelBehavior.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Martin Betak: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58133 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iffb7f3d172cf769f9bef78a4dd067b30e952282b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: userportal: GetAllMigrationPolicies user query
by tnisan＠redhat.com 26 May '16

26 May '16

Tal Nisan has submitted this change and it was merged. Change subject: userportal: GetAllMigrationPolicies user query ...................................................................... userportal: GetAllMigrationPolicies user query It is inited in user poral and needed during creating a new VM. Without this it is not possible to create a new VM in user portal. Change-Id: Idd1522ef6fa79301ca00c41e38d373f061c02ac9 Bug-Url: https://bugzilla.redhat.com/1339668 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58132 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd1522ef6fa79301ca00c41e38d373f061c02ac9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: GetAllAttachableDisksForVmQuery's DbFacade
by amureini＠redhat.com 26 May '16

26 May '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetAllAttachableDisksForVmQuery's DbFacade ...................................................................... core: GetAllAttachableDisksForVmQuery's DbFacade Use getDbFacade() instead of DbFacade.getInstance() to make writing tests easier. Change-Id: Id471eafd0dd2bd1378d041b877c5dca6cefb679e Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/GetAllAttachableDisksForVmQuery.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/58139 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id471eafd0dd2bd1378d041b877c5dca6cefb679e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(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-engine[master]: core: GetAllAttachableDisksForVmQuery branching
by amureini＠redhat.com 26 May '16

26 May '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetAllAttachableDisksForVmQuery branching ...................................................................... core: GetAllAttachableDisksForVmQuery branching Removed a useless if from GetAllAttachableDisksForVmQuery#executeQueryCommand. Since entering and not entering the if has the same effect, it's useless, and can safely be removed. This patch fixes Coverity issue 1355578. Change-Id: If6afa979b5fb4375bfa344ee8711b1a155e2d848 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/GetAllAttachableDisksForVmQuery.java 1 file changed, 2 insertions(+), 11 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/58138 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If6afa979b5fb4375bfa344ee8711b1a155e2d848 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(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-engine[master]: Fix a policy unit db upgrade script according to oVirt style...
by rgolan＠redhat.com 26 May '16

26 May '16

Roy Golan has submitted this change and it was merged. Change subject: Fix a policy unit db upgrade script according to oVirt style rules ...................................................................... Fix a policy unit db upgrade script according to oVirt style rules Change-Id: I86f05058abc7fa5c591f1cecb68981277705e85e Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M packaging/dbscripts/upgrade/04_00_0150_remove_internal_policy_units.sql 1 file changed, 18 insertions(+), 10 deletions(-) Approvals: Martin Sivák: Verified Eli Mesika: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51361 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86f05058abc7fa5c591f1cecb68981277705e85e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0