January 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk[sdk_3.5]: sdk: Prepare for next development iteration
by juan.hernandez＠redhat.com 21 Jan '15

21 Jan '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Prepare for next development iteration ...................................................................... sdk: Prepare for next development iteration In preparation for the next development iteration this patch increases the version number and sets the RPM relase number to 0.1. Change-Id: I800082a9b2aea5231207167f0c6ffa7007e906e8 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M setup.py 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37166 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I800082a9b2aea5231207167f0c6ffa7007e906e8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk[sdk_3.5]: sdk: Release 3.5.1.0
by juan.hernandez＠redhat.com 21 Jan '15

21 Jan '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Release 3.5.1.0 ...................................................................... sdk: Release 3.5.1.0 Change-Id: If1266dd6d055ad1b4995df88f62a3c5c502d001c Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M ovirt-engine-sdk-python.spec.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37165 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1266dd6d055ad1b4995df88f62a3c5c502d001c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: removed archaic commented out code.
by tjelinek＠redhat.com 21 Jan '15

21 Jan '15

Tomas Jelinek has submitted this change and it was merged. Change subject: core: removed archaic commented out code. ...................................................................... core: removed archaic commented out code. Change-Id: I0e7a81ba3fa7e7621c50397f3fd6c01fd7f6570a Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleTreeView.java 1 file changed, 1 insertion(+), 5 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Mucha: Verified Moti Asayag: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/29901 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e7a81ba3fa7e7621c50397f3fd6c01fd7f6570a Gerrit-PatchSet: 28 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: refactor: Thou shalt not copy paste.
by tjelinek＠redhat.com 21 Jan '15

21 Jan '15

Tomas Jelinek has submitted this change and it was merged. Change subject: core: refactor: Thou shalt not copy paste. ...................................................................... core: refactor: Thou shalt not copy paste. Change-Id: I4b92b6b7f45886a1222ecb08ca895e5b7acb7076 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleTreeView.java 1 file changed, 36 insertions(+), 20 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Mucha: Verified -- To view, visit http://gerrit.ovirt.org/29900 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b92b6b7f45886a1222ecb08ca895e5b7acb7076 Gerrit-PatchSet: 28 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: refactored RoleTreeView for reababilily purposes.
by tjelinek＠redhat.com 21 Jan '15

21 Jan '15

Tomas Jelinek has submitted this change and it was merged. Change subject: core: refactored RoleTreeView for reababilily purposes. ...................................................................... core: refactored RoleTreeView for reababilily purposes. - so I've added two methods allowing removal all those array declarations. - I've fixed some code conventions violations, except for ... - I know, I'm exceeding limit of 120 chars per line, but that could be expected with identifiers so long as 85 chars. But I do believe this violation is for good thing. You can read this file now. Change-Id: I294e37d07a52c894e4293ca54f9cdb1003df8913 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleTreeView.java 1 file changed, 175 insertions(+), 234 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Mucha: Verified -- To view, visit http://gerrit.ovirt.org/29899 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I294e37d07a52c894e4293ca54f9cdb1003df8913 Gerrit-PatchSet: 27 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: cleanup in RunVmCommandBase
by ahadas＠redhat.com 21 Jan '15

21 Jan '15

Arik Hadas has submitted this change and it was merged. Change subject: core: cleanup in RunVmCommandBase ...................................................................... core: cleanup in RunVmCommandBase The part that removes hibernation volumes in RunVmCommandBase#runningSucceeded is redundant now that we store them on the active snapshot since they are removed by RunVmCommand. Thus, it is removed. Change-Id: I355cab35fd090918737f00edd356324c30ce4ca6 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java 1 file changed, 0 insertions(+), 24 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/37087 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I355cab35fd090918737f00edd356324c30ce4ca6 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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-scheduler-proxy[master]: New version
by msivak＠redhat.com 21 Jan '15

21 Jan '15

Martin Sivák has submitted this change and it was merged. Change subject: New version ...................................................................... New version Change-Id: I6a951d2df41d590e6e16dcc9d191cb9d8d007cc8 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M VERSION M ovirt-scheduler-proxy.spec.in 2 files changed, 5 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37158 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6a951d2df41d590e6e16dcc9d191cb9d8d007cc8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Support search disk by name
by juan.hernandez＠redhat.com 21 Jan '15

21 Jan '15

Juan Hernandez has submitted this change and it was merged. Change subject: core: Support search disk by name ...................................................................... core: Support search disk by name Currently we only support searching disk by "alias", not by "name". But name is used in many places as an alias for "alias", so it makes sense to allow it as well in search queries. Change-Id: I09f910f017945ceb5099cb724fb2e6432769a11c Bug-Url: https://bugzilla.redhat.com/1173196 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/DiskConditionFieldAutoCompleter.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Juan Hernandez: Verified Allon Mureinik: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/36109 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I09f910f017945ceb5099cb724fb2e6432769a11c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Add role of VM run time manager.
by mlipchuk＠redhat.com 21 Jan '15

21 Jan '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Add role of VM run time manager. ...................................................................... core: Add role of VM run time manager. Adding a new pre-defined role of VM run time operator which will act the same as VMOperator only without the permissions for create snapshot. Change-Id: Ifc0e2484aabee5779d6bfaf44e25bab0511e8e16 Bug-Url: https://bugzilla.redhat.com/1111095 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/PredefinedRoles.java A packaging/dbscripts/upgrade/03_06_0770_add_vm_run_time_permissions.sql 2 files changed, 25 insertions(+), 0 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37094 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifc0e2484aabee5779d6bfaf44e25bab0511e8e16 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: packaging: setup: fixing the check of ISO image permission
by sbonazzo＠redhat.com 21 Jan '15

21 Jan '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fixing the check of ISO image permission ...................................................................... packaging: setup: fixing the check of ISO image permission Fixing ISO image permission check against qemu user Change-Id: I3445e536f4f22b7ccdad1163b4be8afe1c8a69af Bug-Url: https://bugzilla.redhat.com/1181585 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> (cherry picked from commit 4c2ffae5bdeaf311c120711f1049f820abff2259) --- M src/plugins/ovirt-hosted-engine-setup/vm/boot_cdrom.py 1 file changed, 2 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified -- To view, visit http://gerrit.ovirt.org/36838 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3445e536f4f22b7ccdad1163b4be8afe1c8a69af Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0