August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: engine : Async tasks should handle exception in creating task
by piotr.kliczewski＠gmail.com 12 Aug '16

12 Aug '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: engine : Async tasks should handle exception in creating task ...................................................................... engine : Async tasks should handle exception in creating task When async tasks have multiple child commands and one of them fails to be submitted to vdsm, the AsyncTaskManager should be able to keep track of the root command and invoke endWithFailure. Change-Id: Ic51a7d94b7c593a178222fee2e0cf01eaa7cfc45 Bug-Url: https://bugzilla.redhat.com/1360373 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/SPMAsyncTask.java 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62264 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic51a7d94b7c593a178222fee2e0cf01eaa7cfc45 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-4.0]: build: ovirt-live 4.0.2
by sbonazzo＠redhat.com 12 Aug '16

12 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-live 4.0.2 ...................................................................... build: ovirt-live 4.0.2 Change-Id: I86c4ddb0904a018128f745fb34bab490b218a572 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62269 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86c4ddb0904a018128f745fb34bab490b218a572 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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]: webadmin: enlarge the create snapshot message dialog
by tjelinek＠redhat.com 12 Aug '16

12 Aug '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: enlarge the create snapshot message dialog ...................................................................... webadmin: enlarge the create snapshot message dialog Enlarge the size of the message Panel inside the snapshot creation dialog in order to avoid message truncation. Change-Id: I8e4716171b39c99774ae36d2eb51cb8d8583cb43 Bug-Url: https://bugzilla.redhat.com/1360914 Signed-off-by: Sharon Gratch <sgratch(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmSnapshotCreatePopupWidget.ui.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Sharon Gratch: Verified -- To view, visit https://gerrit.ovirt.org/62218 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e4716171b39c99774ae36d2eb51cb8d8583cb43 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: el7: move glusterfs to 3.8
by sbonazzo＠redhat.com 11 Aug '16

11 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: el7: move glusterfs to 3.8 ...................................................................... el7: move glusterfs to 3.8 Moving glusterfs to 3.8 since it's the default for Fedora 24 and CentOS Storage SIG. Change-Id: I3746f619d6c2dba20e9a0b8fd5f7fa233b2386d9 Bug-Url: https://bugzilla.redhat.com/1366118 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/check-patch.repos M configure.ac M ovirt-el7-deps.repo.in M ovirt-release-master.spec.in 4 files changed, 9 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62254 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3746f619d6c2dba20e9a0b8fd5f7fa233b2386d9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[master]: repos: move to GlusterFS 3.8
by sbonazzo＠redhat.com 11 Aug '16

11 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: repos: move to GlusterFS 3.8 ...................................................................... repos: move to GlusterFS 3.8 Moving glusterfs to 3.8 since it's the default for Fedora 24 and CentOS Storage SIG. Change-Id: I6d8b6bf5ea94ca81be33465ba214c8ece0f3c366 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62252 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d8b6bf5ea94ca81be33465ba214c8ece0f3c366 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ioprocess: explicitly set job name for build_artifacts
by sbonazzo＠redhat.com 11 Aug '16

11 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ioprocess: explicitly set job name for build_artifacts ...................................................................... ioprocess: explicitly set job name for build_artifacts explicitly set job name for build_artifacts dropped in the ppc64le patch, should re-enable deploy to experimental job. Change-Id: I4f0f545a8691357d419c2c32300d66d432646c56 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ioprocess/ioprocess.yaml 1 file changed, 4 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/62256 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f0f545a8691357d419c2c32300d66d432646c56 Gerrit-PatchSet: 2 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 jenkins[master]: qemu: move to gluster 3.8
by eedri＠redhat.com 11 Aug '16

11 Aug '16

Eyal Edri has submitted this change and it was merged. Change subject: qemu: move to gluster 3.8 ...................................................................... qemu: move to gluster 3.8 Moving glusterfs to 3.8 since it's the default for Fedora 24 and CentOS Storage SIG. Issue: https://ovirt-jira.atlassian.net/browse/OVIRT-680 Change-Id: I7d1c9e7306df5cc7bc5571a95906828232529593 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/qemu/qemu_create-rpms.yaml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for review Eyal Edri: Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62251 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d1c9e7306df5cc7bc5571a95906828232529593 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-live[master]: fedora: moving to ganeric branding
by sbonazzo＠redhat.com 11 Aug '16

11 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: fedora: moving to ganeric branding ...................................................................... fedora: moving to ganeric branding moving to generic branding dropping fedora branding Change-Id: I107fc7b35f50aa8ce5276377b46fecbedfbc685f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M fedora-24/kickstart/ovirt-live-ovirt-custom.ks.in 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62215 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I107fc7b35f50aa8ce5276377b46fecbedfbc685f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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-live[master]: fedora: fixed make clean
by sbonazzo＠redhat.com 11 Aug '16

11 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: fedora: fixed make clean ...................................................................... fedora: fixed make clean fixed make clean dropping fedora ks symlinks Change-Id: I2b979d71e447480856f8d3a4fc4a53bfff3f3e85 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M fedora-24/Makefile 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62214 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b979d71e447480856f8d3a4fc4a53bfff3f3e85 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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]: engine: Init task handlers after the constructor
by piotr.kliczewski＠gmail.com 11 Aug '16

11 Aug '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: engine: Init task handlers after the constructor ...................................................................... engine: Init task handlers after the constructor In order to make sure that any data preparation done in the init method of a command will be performed before the tasks handlers are initiated, the call to initTaskHandlers method is moved to postConstruct instead of inside the constructor. Change-Id: Ib65f84ae8fdeb4dcb4034ace74558d07d34c99f8 Bug-Url: https://bugzilla.redhat.com/1365393 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Freddy Rolland: Verified -- To view, visit https://gerrit.ovirt.org/62243 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib65f84ae8fdeb4dcb4034ace74558d07d34c99f8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0