- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: core: limit disk alias and description for disk metadata
by tnisan＠redhat.com 09 Sep '15

09 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: limit disk alias and description for disk metadata ...................................................................... core: limit disk alias and description for disk metadata On block storage, the disk metadata is limited to 512 bytes. Since we store its alias and description in the metadata, we should limit their size or else they will be truncated by vdsm and we won't be able to read them. This patch limits the value of the metadata description field size to a maximum of 210 bytes. We do it also for file storage in order to prevent a truncation of the description field when moving a disk from file to block storage. Change-Id: I74d04291bbaacb5e642d8658ab752b1a8d32c0ac Bug-Url: https://bugzilla.redhat.com/1258547 Related-To: https://bugzilla.redhat.com/1249130 Related-To: https://bugzilla.redhat.com/1219680 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MetadataDiskDescriptionHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/MetadataDiskDescriptionHandlerTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 6 files changed, 218 insertions(+), 18 deletions(-) Approvals: Tal Nisan: Looks good to me, approved; Passed CI tests Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/45899 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74d04291bbaacb5e642d8658ab752b1a8d32c0ac Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(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: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: implement MetadataDiskDescriptionHandler as a singleton
by tnisan＠redhat.com 09 Sep '15

09 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: implement MetadataDiskDescriptionHandler as a singleton ...................................................................... core: implement MetadataDiskDescriptionHandler as a singleton This patch is re-implementing MetadataDiskDescriptionHandler to be a singleton so that it would be easier to test it. Change-Id: I2832b4803d9a2b96a0e63973e0571c1b46892d3e Related-To: https://bugzilla.redhat.com/1249130 Related-To: https://bugzilla.redhat.com/1219680 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetUnregisteredDiskQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MetadataDiskDescriptionHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/MetadataDiskDescriptionHandlerTest.java 5 files changed, 15 insertions(+), 7 deletions(-) Approvals: Tal Nisan: Looks good to me, approved; Passed CI tests Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/45898 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2832b4803d9a2b96a0e63973e0571c1b46892d3e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: create MetadataDiskDescriptionHandler
by tnisan＠redhat.com 09 Sep '15

09 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: create MetadataDiskDescriptionHandler ...................................................................... core: create MetadataDiskDescriptionHandler This patch moves the logic regarding reading and writing the disk description in its metadata to a separate class. Change-Id: I1fb296980a737dfdd279d73615c4943d84de588b Related-To: https://bugzilla.redhat.com/1249130 Related-To: https://bugzilla.redhat.com/1219680 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetUnregisteredDiskQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MetadataDiskDescriptionHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ImagesHandlerTest.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/MetadataDiskDescriptionHandlerTest.java 7 files changed, 89 insertions(+), 59 deletions(-) Approvals: Tal Nisan: Looks good to me, approved; Passed CI tests Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/45897 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1fb296980a737dfdd279d73615c4943d84de588b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: improve disk description encode/decode API
by tnisan＠redhat.com 09 Sep '15

09 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: improve disk description encode/decode API ...................................................................... core: improve disk description encode/decode API This patch changes the methods that encode and decode the disk description to json to get the disk itself instead of its values. Change-Id: I49aaaf07fe5be36bd9708c1860db7417b8e0efa0 Related-To: https://bugzilla.redhat.com/1249130 Related-To: https://bugzilla.redhat.com/1219680 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddImageFromScratchCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateImageTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetUnregisteredDiskQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ImagesHandlerTest.java 7 files changed, 31 insertions(+), 27 deletions(-) Approvals: Tal Nisan: Looks good to me, approved; Passed CI tests Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/45896 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I49aaaf07fe5be36bd9708c1860db7417b8e0efa0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: remove duplicate code from tests in ImagesHandlerTest
by tnisan＠redhat.com 09 Sep '15

09 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: remove duplicate code from tests in ImagesHandlerTest ...................................................................... core: remove duplicate code from tests in ImagesHandlerTest Change-Id: If0fbb1d7c5e72103ba6df8869bdc2c2415fca180 Related-To: https://bugzilla.redhat.com/1249130 Related-To: https://bugzilla.redhat.com/1219680 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ImagesHandlerTest.java 1 file changed, 11 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/45895 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If0fbb1d7c5e72103ba6df8869bdc2c2415fca180 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup:avoid asking cluster name on automatic depl...
by sbonazzo＠redhat.com 09 Sep '15

09 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup:avoid asking cluster name on automatic deployment ...................................................................... packaging: setup:avoid asking cluster name on automatic deployment If the user selected to automatically execute engine-setup on the appliace for sure we have just the Default cluster so asking is just a waste of time. Bug-Url: https://bugzilla.redhat.com/1260161 Change-Id: Ib0fa01648069ad356a427b8517060a69ee64868b Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/vm/cloud_init.py 1 file changed, 9 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44733 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib0fa01648069ad356a427b8517060a69ee64868b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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

1 0

Change in jenkins[master]: some more fixes to bash script
by eedri＠redhat.com 08 Sep '15

08 Sep '15

Eyal Edri has submitted this change and it was merged. Change subject: some more fixes to bash script ...................................................................... some more fixes to bash script Change-Id: I80895ff67f749905b5b8b8a7c3f1d589c30d1366 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/yaml/jobs/system/system_gerrit-alert-old-patches.yaml 1 file changed, 7 insertions(+), 4 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/45884 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I80895ff67f749905b5b8b8a7c3f1d589c30d1366 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI

1 0

Change in ovirt-engine[ovirt-engine-3.6]: scheduler: Refactor AffinityRulesEnforcementManager
by tnisan＠redhat.com 08 Sep '15

08 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: scheduler: Refactor AffinityRulesEnforcementManager ...................................................................... scheduler: Refactor AffinityRulesEnforcementManager * Fix check if a VM is still migrating. * Trigger one migration per cluster instead of one migration per datacenter. * Split unit tests in AffinityRulesEnforcementManager tests and AffinityRulesEnforcementPerCluster tests. * Remove old tests and added more new ones. * Remove not fully implemented features: - lower wakeup frequency when everything is satisfied - temporary suspension of clusters when too many migration failures occur They will be added later. Bug-Url: https://bugzilla.redhat.com/1251742 Change-Id: I8921baa3b5335103d8dc0cd31eed094eb025cba1 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManager.java R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcer.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/MigrationEntryDS.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcerTest.java 5 files changed, 329 insertions(+), 574 deletions(-) Approvals: Tal Nisan: Passed CI tests Roman Mohr: Verified Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45844 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8921baa3b5335103d8dc0cd31eed094eb025cba1 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Add possibility to query for VMs with specific status
by tnisan＠redhat.com 08 Sep '15

08 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Add possibility to query for VMs with specific status ...................................................................... core: Add possibility to query for VMs with specific status Add a DAO method which allows to query for any VM within a specific status. For instance query for all VMs which are currently migrating Bug-Url: https://bugzilla.redhat.com/1251742 Change-Id: Ief02ac7e5891c556665cd526a975a850634f738a Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- 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 M packaging/dbscripts/vms_sp.sql 4 files changed, 43 insertions(+), 2 deletions(-) Approvals: Tal Nisan: Passed CI tests Roman Mohr: Verified Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45872 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief02ac7e5891c556665cd526a975a850634f738a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: scheduler: Formatter treatment for AffinityRulesEnforcementM...
by tnisan＠redhat.com 08 Sep '15

08 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: scheduler: Formatter treatment for AffinityRulesEnforcementManager ...................................................................... scheduler: Formatter treatment for AffinityRulesEnforcementManager Bug-Url: https://bugzilla.redhat.com/1251742 Change-Id: I9829971bdd387a83b526c5b7306fdba25d03a3e1 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManager.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcementPerCluster.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 3 files changed, 54 insertions(+), 58 deletions(-) Approvals: Tal Nisan: Passed CI tests Roman Mohr: Verified Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45843 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9829971bdd387a83b526c5b7306fdba25d03a3e1 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0