July 2016 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-2.0]: build: ovirt-hosted-engine-setup-2.0.1.2
by stirabos＠redhat.com 26 Jul '16

26 Jul '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-setup-2.0.1.2 ...................................................................... build: ovirt-hosted-engine-setup-2.0.1.2 Change-Id: Ib71829d1b7f1675006a4b8ec0622ce00c8feae9a Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-hosted-engine-setup.spec.in 3 files changed, 7 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61405 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib71829d1b7f1675006a4b8ec0622ce00c8feae9a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.0 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: minor refactoring in VmCommand
by ahadas＠redhat.com 26 Jul '16

26 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: minor refactoring in VmCommand ...................................................................... core: minor refactoring in VmCommand 1. Remove VmCommand#vmDao that is not used. 2. Modify incorrect log printing Change-Id: I21fdeb2365b770d9915c2a6c97f26b0b421ea167 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java 1 file changed, 1 insertion(+), 5 deletions(-) Approvals: Jenkins CI: Passed CI tests Arik Hadas: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61364 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I21fdeb2365b770d9915c2a6c97f26b0b421ea167 Gerrit-PatchSet: 4 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: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ovirt-engine-extension-logger-log4j: drop fedora on 3.6
by sbonazzo＠redhat.com 26 Jul '16

26 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine-extension-logger-log4j: drop fedora on 3.6 ...................................................................... ovirt-engine-extension-logger-log4j: drop fedora on 3.6 Change-Id: Ic637155011a0ff3bde88b6b7542e07be82db6b3b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-extension-logger-log4j/ovirt-engine-extension-logger-log4j_create-rpms.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61371 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic637155011a0ff3bde88b6b7542e07be82db6b3b Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(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[ovirt-engine-4.0]: restapi: Remove `networkfilters` link from V3
by oourfali＠redhat.com 26 Jul '16

26 Jul '16

Oved Ourfali has submitted this change and it was merged. Change subject: restapi: Remove `networkfilters` link from V3 ...................................................................... restapi: Remove `networkfilters` link from V3 Currently when version 3 of the API is requested the root resource contains a link to the `networkfilters` collection that was added in version 4 of the API. But following this link produces an empty result, because there is no such collection in version 3 of the API. This patch removes the link from version 3 of the API. Change-Id: I58a03e31ff7a86477198b039eb685a201eac5ad1 Bug-Url: https://bugzilla.redhat.com/1359643 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 9f1bfbe55ad58b4c5658be2a6d794259286a1be2) --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3ApiOutAdapter.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61391 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I58a03e31ff7a86477198b039eb685a201eac5ad1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: restapi: Add V3 payload adapter
by oourfali＠redhat.com 26 Jul '16

26 Jul '16

Oved Ourfali has submitted this change and it was merged. Change subject: restapi: Add V3 payload adapter ...................................................................... restapi: Add V3 payload adapter This patch adds the missing V3 adapters for the "payloads" object. Change-Id: Id267d3f580a63de9f1f5abcf9c577f5cbb98679e Bug-Url: https://bugzilla.redhat.com/1359885 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 95f5a6b3ae48392376171c710a040b48e0f53370) --- A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3PayloadsInAdapter.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3PayloadsOutAdapter.java 2 files changed, 64 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61390 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id267d3f580a63de9f1f5abcf9c577f5cbb98679e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: Fix log message for CPU and memory balancing
by oourfali＠redhat.com 26 Jul '16

26 Jul '16

Oved Ourfali has submitted this change and it was merged. Change subject: core: Fix log message for CPU and memory balancing ...................................................................... core: Fix log message for CPU and memory balancing When CPU and memory balancing policy cannot be applied on a cluster using the primary or secondary methods - the warning log messages states that all hosts are over utilized which is not the case in some implementations such as power saving balancing. Log messages where changed to be more clear. Change-Id: Ib776cdbd90d8ee9023955cf64616125ac008a52a Bug-Url: https://bugzilla.redhat.com/1353059 Signed-off-by: yquinn <yquinn(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/CpuAndMemoryBalancingPolicyUnit.java 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved Andrej Krejcir: Looks good to me, but someone else must approve Yanir Quinn: Verified -- To view, visit https://gerrit.ovirt.org/60445 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib776cdbd90d8ee9023955cf64616125ac008a52a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yanir Quinn <yquinn(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: Oved Ourfali <oourfali(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 tool-tip for disk hot-plug buttons
by derez＠redhat.com 26 Jul '16

26 Jul '16

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: fix tool-tip for disk hot-plug buttons ...................................................................... webadmin: fix tool-tip for disk hot-plug buttons When disk hot-plug is not supported Activate/Deactivate buttons should display an appropriate tool-tip on hover. Hence, fixing tool-tip by examining support according to selected disks' interfaces. Change-Id: I5ab8b0c721bff17bbd422ff9b042804cadf0eb30 Bug-Url: https://bugzilla.redhat.com/1359632 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/VmDiskListModelTable.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmDiskListModel.java 2 files changed, 4 insertions(+), 29 deletions(-) Approvals: Tomas Jelinek: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Daniel Erez: Verified Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61294 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ab8b0c721bff17bbd422ff9b042804cadf0eb30 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-2.0]: upgrade: migrate file based lockspace and metadata
by stirabos＠redhat.com 26 Jul '16

26 Jul '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: upgrade: migrate file based lockspace and metadata ...................................................................... upgrade: migrate file based lockspace and metadata On oVirt 3.4, metadata and lockspace area were based on files directly saved in the NFS storage. This would not work with iSCSI and so with 3.5 we moved to regular VDSM volumes also on NFS. This is a procedure to upgrade system started with 3.4 to use VDSM volumes. The upgrade procedure will scan for the metadata and the lockspace area, if they aren't on the storage domain it will trigger the volume creation. Volume creation will also remove the previous file and it will replace it with a symlink pointing to the new volume. Upon restart, all the hosts will point to the new volume since they'll simply consume the symlink. Change-Id: If64a1325dc4187b61bff9eb1d2763995c8387e46 Bug-Url: https://bugzilla.redhat.com/1313917 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M po/POTFILES.in M src/ovirt_hosted_engine_setup/constants.py M src/plugins/gr-he-common/Makefile.am M src/plugins/gr-he-common/core/misc.py R src/plugins/gr-he-common/sanlock/Makefile.am R src/plugins/gr-he-common/sanlock/__init__.py R src/plugins/gr-he-common/sanlock/lockspace.py M src/plugins/gr-he-setup/Makefile.am M src/plugins/gr-he-setup/engine/add_host.py M src/plugins/gr-he-upgradeappliance/core/misc.py 11 files changed, 231 insertions(+), 14 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61345 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If64a1325dc4187b61bff9eb1d2763995c8387e46 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.0 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: removed redundant source column header
by derez＠redhat.com 26 Jul '16

26 Jul '16

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: removed redundant source column header ...................................................................... webadmin: removed redundant source column header Removed redundant 'Source' column header from disks table in VM dialog (Resource Allocation tab) and template dialog. Change-Id: Ia6c9ea2891637430cc9f300eb2b28bf0986275c0 Bug-Url: https://bugzilla.redhat.com/1358432 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/DisksAllocationModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageRegisterDiskImageListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/PoolModelBehaviorBase.java 3 files changed, 1 insertion(+), 8 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61333 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6c9ea2891637430cc9f300eb2b28bf0986275c0 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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[ovirt-engine-4.0.2]: engine: ConnectStorageServer on attach FC SD
by piotr.kliczewski＠gmail.com 26 Jul '16

26 Jul '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: engine: ConnectStorageServer on attach FC SD ...................................................................... engine: ConnectStorageServer on attach FC SD After creating a new storage domain, we will attach the SD to a pool. As part of the AttachStorageDomainToPoolCommand, we connect all the hosts to the storage server. In case of Fiber Channel, the connectStorageServer command was ignored since the SD was not connected to any pool. Change-Id: I9aedd2cd177a0697c8096f025d0fe636ac04557f Bug-Url: https://bugzilla.redhat.com/1353430 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/connection/FCPStorageHelper.java 1 file changed, 1 insertion(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Freddy Rolland: Verified -- To view, visit https://gerrit.ovirt.org/61392 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9aedd2cd177a0697c8096f025d0fe636ac04557f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.2 Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0