January 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Add DISK_LIVE_STORAGE_MIGRATION to permit type.
by masayag＠redhat.com 26 Jan '15

26 Jan '15

Moti Asayag has submitted this change and it was merged. Change subject: restapi: Add DISK_LIVE_STORAGE_MIGRATION to permit type. ...................................................................... restapi: Add DISK_LIVE_STORAGE_MIGRATION to permit type. Adding new added permission to permit type. Change-Id: I8c114b0bb403be59772a8627efb8800a4e2600a6 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/PermitType.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Maor Lipchuk: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37296 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c114b0bb403be59772a8627efb8800a4e2600a6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Use new permission for LSM
by mlipchuk＠redhat.com 26 Jan '15

26 Jan '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Use new permission for LSM ...................................................................... core: Use new permission for LSM Add new permission for LSM to be used for all the roles which has the permissions for CONFIGURE_DISK_STORAGE. Change the permissions at LiveMigrateVmDisksCommand to be used for DISK_LIVE_STORAGE_MIGRATION on the disk, instead using the permissions for MANIPULATE_VM_SNAPSHOTS on the VM. Change-Id: I3ba7e03780ba6ef455447a4eaa34a71b1f63d8cb Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ActionGroup.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleTreeView.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java A packaging/dbscripts/upgrade/03_06_0800_add_LSM_action_group_id.sql 5 files changed, 27 insertions(+), 9 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit http://gerrit.ovirt.org/37287 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3ba7e03780ba6ef455447a4eaa34a71b1f63d8cb Gerrit-PatchSet: 4 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: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: services: Update default mem heap values
by sbonazzo＠redhat.com 26 Jan '15

26 Jan '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: services: Update default mem heap values ...................................................................... packaging: services: Update default mem heap values Engine configuration was updated to allocate between 2 and 4 GB of RAM: ENGINE_HEAP_MIN=2g ENGINE_HEAP_MAX=4g Change-Id: Ibf6177172d8a2bd02c98aa448131bbd60c793fab Signed-off-by: Lev Veyde <lveyde(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1185411 --- M packaging/services/ovirt-engine/ovirt-engine.conf.in 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve oVirt Jenkins CI Server: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/37293 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf6177172d8a2bd02c98aa448131bbd60c793fab Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(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

Change in ovirt-engine[master]: core: added default sorting of the audit log events
by tjelinek＠redhat.com 26 Jan '15

26 Jan '15

Tomas Jelinek has submitted this change and it was merged. Change subject: core: added default sorting of the audit log events ...................................................................... core: added default sorting of the audit log events If the user wanted to query the events using api using the "Filter: true", the results returned were not sorted making the ";max=N" param useless. This patch simply makes them sorted by log time returning the newer events first. Change-Id: I3d768a23c758f4d52f9f563a2e339873f451c9e4 Bug-Url: https://bugzilla.redhat.com/1180199 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M packaging/dbscripts/audit_log_sp.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified Eli Mesika: Looks good to me, but someone else must approve Juan Hernandez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36885 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d768a23c758f4d52f9f563a2e339873f451c9e4 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(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[ovirt-engine-3.5]: webadmin: Preventing moving a shareable disk to a Gluster do...
by tnisan＠redhat.com 26 Jan '15

26 Jan '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Preventing moving a shareable disk to a Gluster domain ...................................................................... webadmin: Preventing moving a shareable disk to a Gluster domain Gluster doamin does not support shareable disks- therefore UI now prevents this action. Change-Id: Ibc2ea43f2131056c6f9ea86ecfb574fe71b638be Bug-Url: https://bugzilla.redhat.com/1135771 Signed-off-by: Amit Aviram <aaviram(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/MoveOrCopyDiskModel.java 1 file changed, 21 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Amit Aviram: Verified -- To view, visit http://gerrit.ovirt.org/37278 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc2ea43f2131056c6f9ea86ecfb574fe71b638be Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Candace Sheremeta <cshereme(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: Upload RHEV WGT ISO if available
by sbonazzo＠redhat.com 26 Jan '15

26 Jan '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Upload RHEV WGT ISO if available ...................................................................... packaging: setup: Upload RHEV WGT ISO if available This patch modifies the image_upload plugin so it will upload RHEV Guest Tools ISO, if one is available on the system. Change-Id: I9235bd35cf971e91edae5b04d3b40b6f9a27acc2 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1026933 --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/system/image_upload.py 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35233 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9235bd35cf971e91edae5b04d3b40b6f9a27acc2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(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

Change in ovirt-engine[master]: packaging: setup: No default for ISO_DOMAIN nfs acl
by sbonazzo＠redhat.com 26 Jan '15

26 Jan '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: No default for ISO_DOMAIN nfs acl ...................................................................... packaging: setup: No default for ISO_DOMAIN nfs acl Change-Id: I0857fccb75b34c81c94034cfbd97d8f6f59b5a7d Bug-Url: https://bugzilla.redhat.com/1110740 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/iso_domain.py 1 file changed, 11 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve oVirt Jenkins CI Server: Looks good to me, but someone else must approve Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/37062 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0857fccb75b34c81c94034cfbd97d8f6f59b5a7d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(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

Change in ovirt-engine[master]: restapi: Fix false negative in disk tests
by juan.hernandez＠redhat.com 26 Jan '15

26 Jan '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix false negative in disk tests ...................................................................... restapi: Fix false negative in disk tests This patch fixes a false negative introduced by commit 267c358. Change-Id: Ieebf993561c4446793d4c7beec24f19555478191 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainDiskResourceTest.java 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Jakub Niedermertl: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37284 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieebf993561c4446793d4c7beec24f19555478191 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: jsonrpc: version bump
by oourfali＠redhat.com 26 Jan '15

26 Jan '15

Oved Ourfali has submitted this change and it was merged. Change subject: jsonrpc: version bump ...................................................................... jsonrpc: version bump Change-Id: I838f8e37364bd626f6e2152e7e12068193b8acd7 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> Bug-Url: https://bugzilla.redhat.com/1184454 --- M ovirt-engine.spec.in M pom.xml 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37268 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I838f8e37364bd626f6e2152e7e12068193b8acd7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk-java[master]: codegen: Extract metadata from Maven artifacts
by juan.hernandez＠redhat.com 26 Jan '15

26 Jan '15

Juan Hernandez has submitted this change and it was merged. Change subject: codegen: Extract metadata from Maven artifacts ...................................................................... codegen: Extract metadata from Maven artifacts Currently we require a live engine to generate the metadata. This increases the time required to re-generate the SDK, specially when there are more than one version that needs to be re-generated. This patch changes the code generator so that it extracts the metadata from the engine Maven artifacts, thus no live engine is required. Change-Id: I2112e499f428873ab0ae89dae14740bf9611c652 Bug-Url: https://bugzilla.redhat.com/1185853 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-java-codegen/pom.xml M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/rsdl/RsdlCodegen.java D ovirt-engine-sdk-java-codegen/src/main/resources/README R ovirt-engine-sdk-java-codegen/src/main/resources/rsdl.xml M ovirt-engine-sdk-java/README.md 5 files changed, 662 insertions(+), 750 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37283 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2112e499f428873ab0ae89dae14740bf9611c652 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master 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