September 2016 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[master]: packaging: spec: require newer ovirt-setup-lib
by sbonazzo＠redhat.com 30 Sep '16

30 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: require newer ovirt-setup-lib ...................................................................... packaging: spec: require newer ovirt-setup-lib Require newer ovirt-setup-lib for addressing BZ#1366270 Change-Id: I32568ee572191ffd367a776b75851924dc5ed88d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Simone Tiraboschi: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64992 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32568ee572191ffd367a776b75851924dc5ed88d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup 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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Add example of create new role
by omachace＠redhat.com 30 Sep '16

30 Sep '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add example of create new role ...................................................................... Add example of create new role This patch adds example of create new role. Change-Id: Ia3702f83a8c60adcc7a7077be515fae48af84d2a Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- A sdk/examples/add_role.py 1 file changed, 59 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63812 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3702f83a8c60adcc7a7077be515fae48af84d2a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Add example of assigning permission
by omachace＠redhat.com 30 Sep '16

30 Sep '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add example of assigning permission ...................................................................... Add example of assigning permission This patch adds example of assigning GlusterAdmin role to user on network. Change-Id: If10778ff21bf5cfb0f3b576f721ba6b73ce0a3f0 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- A sdk/examples/assign_permission.py 1 file changed, 64 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64436 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If10778ff21bf5cfb0f3b576f721ba6b73ce0a3f0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ovirt-setup-lib: branched 4.0
by sbonazzo＠redhat.com 30 Sep '16

30 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-setup-lib: branched 4.0 ...................................................................... ovirt-setup-lib: branched 4.0 Change-Id: If00f8ad54340530994f4d7166c2bb1b88cb98ac0 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-setup-lib/ovirt-setup-lib_build-artifacts.yaml 1 file changed, 10 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/64988 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If00f8ad54340530994f4d7166c2bb1b88cb98ac0 Gerrit-PatchSet: 1 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>

1 0

Change in ovirt-engine[master]: engine: CommandBase-isExecutedAsChildCommand
by amureini＠redhat.com 29 Sep '16

29 Sep '16

Allon Mureinik has submitted this change and it was merged. Change subject: engine: CommandBase-isExecutedAsChildCommand ...................................................................... engine: CommandBase-isExecutedAsChildCommand Enhance use of CommandBase isExecutedAsChildCommand method. Change-Id: Iac1c6ecceed39b475abc96f257647f8a2750d132 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChildCommandsCallbackBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/StopVdsCommand.java 4 files changed, 7 insertions(+), 12 deletions(-) Approvals: Allon Mureinik: Looks good to me, but someone else must approve; Passed CI tests Freddy Rolland: Verified Moti Asayag: Looks good to me, approved Liron Aravot: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/63064 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iac1c6ecceed39b475abc96f257647f8a2750d132 Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(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: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: remove TaskHandlerCommand interface
by amureini＠redhat.com 29 Sep '16

29 Sep '16

Allon Mureinik has submitted this change and it was merged. Change subject: engine: remove TaskHandlerCommand interface ...................................................................... engine: remove TaskHandlerCommand interface As part of removing SEAT framework, the TaskHandlerCommand interface is not needed. Change-Id: Idc7555a26fbfb9f3efa3ee9afe16cc8ad755edc0 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/MemoryImageRemover.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/MemoryImageRemoverFromExportDomain.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/MemoryImageRemoverOnDataDomain.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/TaskHandlerCommand.java 6 files changed, 10 insertions(+), 176 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62937 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idc7555a26fbfb9f3efa3ee9afe16cc8ad755edc0 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(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: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Remove task handlers implementation
by amureini＠redhat.com 29 Sep '16

29 Sep '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove task handlers implementation ...................................................................... core: Remove task handlers implementation Removing task handlers implementation from CommandBase. Change-Id: I103330314b80775fd0674009cb2774a4018b3037 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/repoimage/ImportRepoImageCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CoCoAsyncTaskHelper.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandAsyncTask.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/SPMAsyncTaskHandler.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CommandBaseTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionParametersBase.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/serialization/json/JsonObjectDeserializerTest.java 8 files changed, 10 insertions(+), 246 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62927 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I103330314b80775fd0674009cb2774a4018b3037 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(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: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Removed mistaken el6
by gshinar＠redhat.com 29 Sep '16

29 Sep '16

Gil Shinar has submitted this change and it was merged. Change subject: Removed mistaken el6 ...................................................................... Removed mistaken el6 I have added el6 just to test something and forgot to remove it from the yaml Change-Id: Id32bde746d24d5d0fec27bd4dde3172119d6def9 --- M jobs/confs/projects/jenkins/jenkins_standard.yaml 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Gil Shinar: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64961 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id32bde746d24d5d0fec27bd4dde3172119d6def9 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in mom[master]: Add caching for vm IO tune policies
by msivak＠redhat.com 29 Sep '16

29 Sep '16

Martin Sivák has submitted this change and it was merged. Change subject: Add caching for vm IO tune policies ...................................................................... Add caching for vm IO tune policies Instead of querying vdsm for each vm's IO tune policy seperatly this allows MOM to query vdsm once for all the needed info and cache the result for all the available vms. Change-Id: Ief6a0ee56e30c6e973321893da30f0b954cb3cfc Bug-Url: https://bugzilla.redhat.com/1366556 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/63971 Continuous-Integration: Jenkins CI Reviewed-by: Martin Sivák <msivak(a)redhat.com> --- A mom/HypervisorInterfaces/vdsmjsonrpcbulkInterface.py A mom/HypervisorInterfaces/vdsmxmlrpcbulkInterface.py 2 files changed, 108 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/63971 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief6a0ee56e30c6e973321893da30f0b954cb3cfc Gerrit-PatchSet: 7 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(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: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Fix incorrect VM status icon in NUMA menu
by awels＠redhat.com 29 Sep '16

29 Sep '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Fix incorrect VM status icon in NUMA menu ...................................................................... webadmin: Fix incorrect VM status icon in NUMA menu This patch fixes an error wherein the status icon of a VM could be out of sync with the actual status of the VM in the NUMA pinning menu. Cause: The instance of the VM model pulled from the database was being discarded and the version currently stored in memory was being used to back the VmNumaSupportModel instead. Fix: Use the model instance being returned from the database to back the VmNumaSupportModel instead of the current model stored in memory. Change-Id: Idd5ed3807d90dbd0ed6b76a31d6a1673e52eb988 Bug-Url: https://bugzilla.redhat.com/1147858 Signed-off-by: Phillip Bailey <phbailey(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/numa/VmNumaSupportModel.java 1 file changed, 1 insertion(+), 10 deletions(-) Approvals: Roman Mohr: Looks good to me, but someone else must approve Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Phillip Bailey: Verified -- To view, visit https://gerrit.ovirt.org/64042 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd5ed3807d90dbd0ed6b76a31d6a1673e52eb988 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0