December 2015 - Engine-commits - oVirt List Archives

Change in ovirt-live[master]: packaging: fix nfs server issue for CentOS
by sbonazzo＠redhat.com 24 Dec '15

24 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: fix nfs server issue for CentOS ...................................................................... packaging: fix nfs server issue for CentOS It seems that on recent versions of CentOS the installation fails, due to the nfs server that fails to start properly. This patch adds code to fix this issue. Change-Id: I5c45ef11a0c2003b260d5b6c77051805c9bbe77f Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/engine-setup-yad.sh 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51043 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5c45ef11a0c2003b260d5b6c77051805c9bbe77f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: engine : Importing an image from glance never finishes
by tnisan＠redhat.com 24 Dec '15

24 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: engine : Importing an image from glance never finishes ...................................................................... engine : Importing an image from glance never finishes Importing an image from glance leaves the disk locked after image has been downloaded Change-Id: I43fa65de4443ad234a23ac493e9f1e49c355b71a Bug-Url: https://bugzilla.redhat.com/1293644 Bug-Url: https://bugzilla.redhat.com/1275629 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CoCoAsyncTaskHelper.java 1 file changed, 12 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Passed CI tests Daniel Erez: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50988 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43fa65de4443ad234a23ac493e9f1e49c355b71a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(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[ovirt-engine-3.6]: core: fix creation of stateless snapshot
by tnisan＠redhat.com 24 Dec '15

24 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: core: fix creation of stateless snapshot ...................................................................... core: fix creation of stateless snapshot Fix a regression that broke the creation of stateless snapshot. Change-Id: I250caff93d77f8e542cac13fd120dbce8ded24ba Bug-Url: https://bugzilla.redhat.com/1287771 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Maor Lipchuk: Looks good to me, approved Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/51030 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I250caff93d77f8e542cac13fd120dbce8ded24ba Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(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[ovirt-engine-3.6]: engine : After an command is finished tasks are not cleared ...
by tnisan＠redhat.com 24 Dec '15

24 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: engine : After an command is finished tasks are not cleared and stay in executing status ...................................................................... engine : After an command is finished tasks are not cleared and stay in executing status Tasks are not being updated to FINISHED status after completion execution. Change-Id: I37bf0926432b4ea592c47ec9e61c093d7ecbd55f Bug-Url: https://bugzilla.redhat.com/1293644 Bug-Url: https://bugzilla.redhat.com/1273447 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CoCoAsyncTaskHelper.java 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Passed CI tests Daniel Erez: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50987 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I37bf0926432b4ea592c47ec9e61c093d7ecbd55f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(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[ovirt-engine-3.6]: userportal: Fix exception in resources VM table
by tnisan＠redhat.com 24 Dec '15

24 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: userportal: Fix exception in resources VM table ...................................................................... userportal: Fix exception in resources VM table - Fixed issue where the ID was not being passed to the columns - Fixed issue where getContainerElement was returning the wrong object causing some NPEs. Change-Id: I403f139c9814e6b2bf61f684285d2b6d5cb88380 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1291156 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/widget/resources/VmTable.java 1 file changed, 1 insertion(+), 7 deletions(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/51032 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I403f139c9814e6b2bf61f684285d2b6d5cb88380 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: remove redundant semi-colon
by masayag＠redhat.com 24 Dec '15

24 Dec '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: remove redundant semi-colon ...................................................................... engine: remove redundant semi-colon Change-Id: I1c05ad0d9a9ff31391e1cff3113d54b10254a4f5 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/51020 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c05ad0d9a9ff31391e1cff3113d54b10254a4f5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: engine : Can't remove vm template - the disks are removed an...
by tnisan＠redhat.com 23 Dec '15

23 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: engine : Can't remove vm template - the disks are removed and the template stays locked ...................................................................... engine : Can't remove vm template - the disks are removed and the template stays locked The patch Ib3261800f383a6e3973acab4bd1 caused regression in commands that ran Async Tasks. This patch reverts the changes in CommandBase related to AsyncTasks and implements a simpler solution using existing shouldBeEndedByParent flag in VdcActionParametersBase. Change-Id: I41f318c8fbef94d3d125d2ca80d687cb27195667 Bug-Url: https://bugzilla.redhat.com/1293644 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CoCoAsyncTaskHelper.java 2 files changed, 16 insertions(+), 81 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50911 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41f318c8fbef94d3d125d2ca80d687cb27195667 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 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: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(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-1.3]: build: post ovirt-hosted-engine-setup-1.3.2.1
by sbonazzo＠redhat.com 23 Dec '15

23 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-hosted-engine-setup-1.3.2.1 ...................................................................... build: post ovirt-hosted-engine-setup-1.3.2.1 Change-Id: I977caa419d67b1629181e8aa956e4c75b9dfcf06 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in 2 files changed, 9 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51031 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I977caa419d67b1629181e8aa956e4c75b9dfcf06 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 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-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: build: ovirt-hosted-engine-setup-1.3.2.1
by sbonazzo＠redhat.com 23 Dec '15

23 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-setup-1.3.2.1 ...................................................................... build: ovirt-hosted-engine-setup-1.3.2.1 Change-Id: I2e706dd1e19a347977b106fe9e6f7a29efc57dc4 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in 2 files changed, 8 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51016 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e706dd1e19a347977b106fe9e6f7a29efc57dc4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 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]: userportal: Fix exception in resources VM table
by awels＠redhat.com 23 Dec '15

23 Dec '15

Alexander Wels has submitted this change and it was merged. Change subject: userportal: Fix exception in resources VM table ...................................................................... userportal: Fix exception in resources VM table - Fixed issue where the ID was not being passed to the columns - Fixed issue where getContainerElement was returning the wrong object causing some NPEs. Change-Id: I403f139c9814e6b2bf61f684285d2b6d5cb88380 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1291156 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/widget/resources/VmTable.java 1 file changed, 1 insertion(+), 7 deletions(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50895 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I403f139c9814e6b2bf61f684285d2b6d5cb88380 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0