August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: StepDao - adding updateStepProgress()
by laravot＠redhat.com 23 Aug '16

23 Aug '16

Liron Aravot has submitted this change and it was merged. Change subject: core: StepDao - adding updateStepProgress() ...................................................................... core: StepDao - adding updateStepProgress() This patch adds the updateStepProgress() method to update a Step progress by its id, avoiding the need to query the step, update it and call the dao update method. Change-Id: Ia309abf1a8f28a7502d353fbfd8fb26593f5c241 Bug-Url: https://bugzilla.redhat.com/529086 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StepDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StepDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StepDaoTest.java M packaging/dbscripts/job_sp.sql 4 files changed, 40 insertions(+), 3 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved Liron Aravot: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62132 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia309abf1a8f28a7502d353fbfd8fb26593f5c241 Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(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: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: adding entity progress support/DiskImage usage
by laravot＠redhat.com 23 Aug '16

23 Aug '16

Liron Aravot has submitted this change and it was merged. Change subject: core: adding entity progress support/DiskImage usage ...................................................................... core: adding entity progress support/DiskImage usage This patch adds support to calculate the progress of operations done on different entities by the steps related to the entity and the steps entity weight. Additionally, this patch contains usage of the progress report on image disks. Change-Id: I55083d6700ff79613fae55eaee28ce4738dca330 Bug-Url: https://bugzilla.redhat.com/529086 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ImagesActionsParametersBase.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ProgressEntity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage/BaseDisk.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DiskImageDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/FixturesTool.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StepDaoTest.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M packaging/dbscripts/create_views.sql 8 files changed, 106 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Allon Mureinik: Looks good to me, approved Liron Aravot: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61861 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55083d6700ff79613fae55eaee28ce4738dca330 Gerrit-PatchSet: 16 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.6]: build: post ovirt-image-uploader-3.6.1
by rmartins＠redhat.com 23 Aug '16

23 Aug '16

Rafael Martins has submitted this change and it was merged. Change subject: build: post ovirt-image-uploader-3.6.1 ...................................................................... build: post ovirt-image-uploader-3.6.1 Change-Id: I9d758dceb57b70656769544a832113e2509826a4 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-image-uploader.spec.in 3 files changed, 9 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62701 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d758dceb57b70656769544a832113e2509826a4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.6]: build: ovirt-image-uploader-3.6.1
by rmartins＠redhat.com 23 Aug '16

23 Aug '16

Rafael Martins has submitted this change and it was merged. Change subject: build: ovirt-image-uploader-3.6.1 ...................................................................... build: ovirt-image-uploader-3.6.1 Change-Id: I6acd03b27115b51f4349434304f306f032dc6b69 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-image-uploader.spec.in 3 files changed, 7 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62700 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6acd03b27115b51f4349434304f306f032dc6b69 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: enforce bricks are not from same server
by sabose＠redhat.com 23 Aug '16

23 Aug '16

Sahina Bose has submitted this change and it was merged. Change subject: engine: enforce bricks are not from same server ...................................................................... engine: enforce bricks are not from same server Enforce bricks in a replica set are not from same server for HC clusters. Change-Id: Ic3c5b27c11cec328d9d972e6f4fff4d4a6fe8070 Bug-Url: https://bugzilla.redhat.com/1182369 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/AddBricksToGlusterVolumeCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/CreateGlusterVolumeCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/AddBricksToGlusterVolumeCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/EngineMessage.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 9 files changed, 150 insertions(+), 28 deletions(-) Approvals: Jenkins CI: Passed CI tests Sahina Bose: Verified; Looks good to me, approved Ramesh N: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59498 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic3c5b27c11cec328d9d972e6f4fff4d4a6fe8070 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(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]: packaging: enforcing log dir permission
by sbonazzo＠redhat.com 23 Aug '16

23 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: enforcing log dir permission ...................................................................... packaging: enforcing log dir permission enforcing log dir permission Change-Id: I2cb442765a6ad71063b98c9b5f73031772ee7680 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62682 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2cb442765a6ad71063b98c9b5f73031772ee7680 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>

1 0

Change in jenkins[master]: adding 3.6.9 to standard ci checks
by sbonazzo＠redhat.com 23 Aug '16

23 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: adding 3.6.9 to standard ci checks ...................................................................... adding 3.6.9 to standard ci checks Change-Id: I193049941f89e820986ff6ec7e4f467fa400cb9a Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 10 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/62688 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I193049941f89e820986ff6ec7e4f467fa400cb9a Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(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[master]: core: Fix null cpu_profile_id in the DB
by dfediuck＠redhat.com 22 Aug '16

22 Aug '16

Doron Fediuck has submitted this change and it was merged. Change subject: core: Fix null cpu_profile_id in the DB ...................................................................... core: Fix null cpu_profile_id in the DB Every VM should have a valid CPU profile. Change-Id: Iad6281415ab0e9637bc12ecc0b9a9cfe8529f93e Bug-Url: https://bugzilla.redhat.com/1367025 Signed-off-by: Andrej Krejcir <akrejcir(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVMClusterCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVmClusterValidator.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/CpuProfileHelper.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/FixturesTool.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDaoTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmStaticDaoTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmTemplateDaoTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/profiles/CpuProfileDaoTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/qos/CpuQosDaoTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml A packaging/dbscripts/upgrade/04_01_0260_fix_null_cpu_profile_id.sql 12 files changed, 191 insertions(+), 50 deletions(-) Approvals: Martin Sivák: Verified; Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Doron Fediuck: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62502 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad6281415ab0e9637bc12ecc0b9a9cfe8529f93e Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(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: 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 gerrit-admin[master]: Update bz.py and hook-dispatcher files
by eedri＠redhat.com 22 Aug '16

22 Aug '16

Eyal Edri has submitted this change and it was merged. Change subject: Update bz.py and hook-dispatcher files ...................................................................... Update bz.py and hook-dispatcher files Following to adding the missing links and files in gerrit hooks directory The bz.py file inside lib directory and hook-dispatcher need to be updated as well (sync to the current files) Change-Id: I675e7dce51d498bf9b260243094ff6ca5476b7be Signed-off-by: Shlomi Ben-David <sbendavi(a)redhat.com> --- M hooks/hook-dispatcher M hooks/lib/bz.py 2 files changed, 37 insertions(+), 32 deletions(-) Approvals: Shlomo Ben David: Verified Eyal Edri: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62681 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I675e7dce51d498bf9b260243094ff6ca5476b7be Gerrit-PatchSet: 2 Gerrit-Project: gerrit-admin Gerrit-Branch: master Gerrit-Owner: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: enforcing log dir permission
by stirabos＠redhat.com 22 Aug '16

22 Aug '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: packaging: enforcing log dir permission ...................................................................... packaging: enforcing log dir permission enforcing log dir permission Change-Id: I2cb442765a6ad71063b98c9b5f73031772ee7680 Bug-Url: https://bugzilla.redhat.com/1369099 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/62679 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2cb442765a6ad71063b98c9b5f73031772ee7680 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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