- Engine-commits - oVirt List Archives

Change in jenkins[master]: Add jobs for stable branches of API model and metamodel
by sbonazzo＠redhat.com 07 Jul '16

07 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Add jobs for stable branches of API model and metamodel ...................................................................... Add jobs for stable branches of API model and metamodel This patch adds jobs to build artifacts and check patches for the stable branches of the API model and metamodel projects. Change-Id: I4a8056ad9a015480c198929bdadc04ad5ffa204c Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-api-metamodel/ovirt-engine-api-metamodel_build.yaml M jobs/confs/projects/ovirt-engine-api-metamodel/ovirt-engine-api-metamodel_checks.yaml M jobs/confs/projects/ovirt-engine-api-model/ovirt-engine-api-model_build.yaml M jobs/confs/projects/ovirt-engine-api-model/ovirt-engine-api-model_checks.yaml 4 files changed, 8 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Juan Hernandez: Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60319 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a8056ad9a015480c198929bdadc04ad5ffa204c Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Add/update VM Pool commands refactoring
by ahadas＠redhat.com 07 Jul '16

07 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: Add/update VM Pool commands refactoring ...................................................................... core: Add/update VM Pool commands refactoring Minor refactoring of add/update VM Pool commands: * Several fields and methods were renamed to more understandable names; * Access to several methods was restricted; * poolId is no more passed as argument; * getQuotaStorageConsumptionParameters() method now uses streams; * All conditions were moved from updatePoolVms() to isUpdateVmRequired(); * Unused diskSize field was removed from command parameters. Change-Id: I78167b8fb59faabe7217406f30e716c3050e91ea Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmPoolCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmPoolCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmPoolCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommandTestAbstract.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVmPoolCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmPoolWithVmsParameters.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolsResource.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java 10 files changed, 93 insertions(+), 108 deletions(-) Approvals: Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/59937 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I78167b8fb59faabe7217406f30e716c3050e91ea Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Shmuel Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0.1]: webadmin: Sync file size and uploaded image's actual size.
by tnisan＠redhat.com 07 Jul '16

07 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Sync file size and uploaded image's actual size. ...................................................................... webadmin: Sync file size and uploaded image's actual size. When uploading an image, the actual image's size needs to be set, so the lv that is created in the storage will allocte enough size. currently, it is not being set, and the created LV is limited to 1GB. Change-Id: I486089be67439d950fad655f2a639d915d2fce4c Bug-Url: https://bugzilla.redhat.com/1353196 Signed-off-by: Amit Aviram <aaviram(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/UploadImageModel.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60309 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I486089be67439d950fad655f2a639d915d2fce4c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.1 Gerrit-Owner: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Sync file size and uploaded image's actual size.
by tnisan＠redhat.com 07 Jul '16

07 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Sync file size and uploaded image's actual size. ...................................................................... webadmin: Sync file size and uploaded image's actual size. When uploading an image, the actual image's size needs to be set, so the lv that is created in the storage will allocte enough size. currently, it is not being set, and the created LV is limited to 1GB. Change-Id: I486089be67439d950fad655f2a639d915d2fce4c Bug-Url: https://bugzilla.redhat.com/1353196 Signed-off-by: Amit Aviram <aaviram(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/UploadImageModel.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Amit Aviram: Verified -- To view, visit https://gerrit.ovirt.org/60308 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I486089be67439d950fad655f2a639d915d2fce4c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Sync file size and uploaded image's actual size.
by tnisan＠redhat.com 07 Jul '16

07 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Sync file size and uploaded image's actual size. ...................................................................... webadmin: Sync file size and uploaded image's actual size. When uploading an image, the actual image's size needs to be set, so the lv that is created in the storage will allocte enough size. currently, it is not being set, and the created LV is limited to 1GB. Change-Id: I486089be67439d950fad655f2a639d915d2fce4c Bug-Url: https://bugzilla.redhat.com/1353196 Signed-off-by: Amit Aviram <aaviram(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/UploadImageModel.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Amit Aviram: Verified Allon Mureinik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/60270 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I486089be67439d950fad655f2a639d915d2fce4c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: CreateBrickCommandTest type inference
by amureini＠redhat.com 07 Jul '16

07 Jul '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: CreateBrickCommandTest type inference ...................................................................... core: CreateBrickCommandTest type inference Change-Id: I8135ec86c6f29e2e867e59f331a6f595934bde0f Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/CreateBrickCommandTest.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/60304 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8135ec86c6f29e2e867e59f331a6f595934bde0f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: CreateBrickCommandTest collections
by amureini＠redhat.com 07 Jul '16

07 Jul '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: CreateBrickCommandTest collections ...................................................................... core: CreateBrickCommandTest collections Used Collections#singletonList instead of Arrays#asList for single element "constant" collections to gain a [minor] performance improvement. Change-Id: I38d164d6b621fbfa461d24509d83f9a1f07fe9c9 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/CreateBrickCommandTest.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/60305 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38d164d6b621fbfa461d24509d83f9a1f07fe9c9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(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: rework the upgrade tool
by stirabos＠redhat.com 07 Jul '16

07 Jul '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: upgrade: rework the upgrade tool ...................................................................... upgrade: rework the upgrade tool We cannot upgrade switching to a new disk, so we identified a different flow creating a backup of the current disk and deploying in place the new appliance. Depends-on: https://gerrit.ovirt.org/60054 Change-Id: I409d9143642601b28f54109864fe4b298be9ca38 Bug-Url: https://bugzilla.redhat.com/1319457 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M man/hosted-engine.8 M man/ovirt-hosted-engine-setup.8 M po/POTFILES.in M po/pt_BR.po M src/bin/hosted-engine.in M src/bin/ovirt-hosted-engine-setup M src/ovirt_hosted_engine_setup/constants.py M src/ovirt_hosted_engine_setup/tasks.py M src/ovirt_hosted_engine_setup/util.py M src/plugins/Makefile.am M src/plugins/gr-he-common/core/misc.py M src/plugins/gr-he-common/core/remote_answerfile.py M src/plugins/gr-he-common/engine/ca.py M src/plugins/gr-he-common/engine/health.py M src/plugins/gr-he-common/vm/boot_disk.py M src/plugins/gr-he-common/vm/cloud_init.py M src/plugins/gr-he-common/vm/misc.py A src/plugins/gr-he-rollbackupgrade/Makefile.am A src/plugins/gr-he-rollbackupgrade/core/Makefile.am A src/plugins/gr-he-rollbackupgrade/core/__init__.py A src/plugins/gr-he-rollbackupgrade/core/misc.py M src/plugins/gr-he-upgradeappliance/core/misc.py M src/plugins/gr-he-upgradeappliance/engine/Makefile.am M src/plugins/gr-he-upgradeappliance/engine/__init__.py D src/plugins/gr-he-upgradeappliance/engine/add_vm_disk.py A src/plugins/gr-he-upgradeappliance/engine/misc.py M src/plugins/gr-he-upgradeappliance/vm/configurevm.py M src/plugins/gr-he-upgradeappliance/vm/runvm.py 29 files changed, 1,328 insertions(+), 531 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60277 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I409d9143642601b28f54109864fe4b298be9ca38 Gerrit-PatchSet: 2 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 jenkins[master]: mom: refresh supported distributions
by sbonazzo＠redhat.com 07 Jul '16

07 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: mom: refresh supported distributions ...................................................................... mom: refresh supported distributions - remove EOL fedora 22 - replace fedora 23 with new released fedora 24 for master - remove el6 since vdsm is not supported there since 3.6 Change-Id: I6df7452b8acb5d8784be3b1217524ad89dc67617 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/mom/mom_create-rpms.yaml 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for merge Eyal Edri: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60274 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6df7452b8acb5d8784be3b1217524ad89dc67617 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-hosted-engine-setup[master]: upgrade: rework the upgrade tool
by stirabos＠redhat.com 06 Jul '16

06 Jul '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: upgrade: rework the upgrade tool ...................................................................... upgrade: rework the upgrade tool We cannot upgrade switching to a new disk, so we identified a different flow creating a backup of the current disk and deploying in place the new appliance. Depends-on: https://gerrit.ovirt.org/60054 Change-Id: I409d9143642601b28f54109864fe4b298be9ca38 Bug-Url: https://bugzilla.redhat.com/1319457 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M man/hosted-engine.8 M man/ovirt-hosted-engine-setup.8 M po/POTFILES.in M po/pt_BR.po M src/bin/hosted-engine.in M src/bin/ovirt-hosted-engine-setup M src/ovirt_hosted_engine_setup/constants.py M src/ovirt_hosted_engine_setup/tasks.py M src/ovirt_hosted_engine_setup/util.py M src/plugins/Makefile.am M src/plugins/gr-he-common/core/misc.py M src/plugins/gr-he-common/core/remote_answerfile.py M src/plugins/gr-he-common/engine/ca.py M src/plugins/gr-he-common/engine/health.py M src/plugins/gr-he-common/vm/boot_disk.py M src/plugins/gr-he-common/vm/cloud_init.py M src/plugins/gr-he-common/vm/misc.py A src/plugins/gr-he-rollbackupgrade/Makefile.am A src/plugins/gr-he-rollbackupgrade/core/Makefile.am A src/plugins/gr-he-rollbackupgrade/core/__init__.py A src/plugins/gr-he-rollbackupgrade/core/misc.py M src/plugins/gr-he-upgradeappliance/core/misc.py M src/plugins/gr-he-upgradeappliance/engine/Makefile.am M src/plugins/gr-he-upgradeappliance/engine/__init__.py D src/plugins/gr-he-upgradeappliance/engine/add_vm_disk.py A src/plugins/gr-he-upgradeappliance/engine/misc.py M src/plugins/gr-he-upgradeappliance/vm/configurevm.py M src/plugins/gr-he-upgradeappliance/vm/runvm.py 29 files changed, 1,328 insertions(+), 531 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59946 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I409d9143642601b28f54109864fe4b298be9ca38 Gerrit-PatchSet: 15 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: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(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: gerrit-hooks <automation(a)ovirt.org>

1 0