- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Move RemoveDiskSnapshots to CoCo
by laravot＠redhat.com 28 Sep '16

28 Sep '16

Liron Aravot has submitted this change and it was merged. Change subject: engine: Move RemoveDiskSnapshots to CoCo ...................................................................... engine: Move RemoveDiskSnapshots to CoCo Move disks snapshot cold merge from taskhandlers to CoCo. Change-Id: I53d5423b42fe1dba5a1e3813b8409cd63c655748 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RemoveDiskSnapshotTaskHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RemoveDiskSnapshotsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RemoveSnapshotSingleDiskCommand.java 4 files changed, 96 insertions(+), 279 deletions(-) Approvals: Freddy Rolland: Verified Liron Aravot: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62232 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53d5423b42fe1dba5a1e3813b8409cd63c655748 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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 jenkins[master]: Fixes ovirt-engine-extension-aaa-misc experimental flow
by eedri＠redhat.com 28 Sep '16

28 Sep '16

Eyal Edri has submitted this change and it was merged. Change subject: Fixes ovirt-engine-extension-aaa-misc experimental flow ...................................................................... Fixes ovirt-engine-extension-aaa-misc experimental flow It was missing reprovision param Change-Id: Ia40e0fcc9e31f6325c84b3734737f2f7d7cfa78e Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-extension-aaa-misc/ovirt-engine-extension-aaa-misc_create-rpms.yaml 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Looks good to me; Ready for merge Gil Shinar: Verified; Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64914 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia40e0fcc9e31f6325c84b3734737f2f7d7cfa78e Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Fix ovirt-engine-wildfly experimental flow
by eedri＠redhat.com 28 Sep '16

28 Sep '16

Eyal Edri has submitted this change and it was merged. Change subject: Fix ovirt-engine-wildfly experimental flow ...................................................................... Fix ovirt-engine-wildfly experimental flow It was missing reprovision parameter and was configured not to be part of deploy packages Change-Id: I1daf53fb63b78aeb1ffe5590a8a20de51db57f80 Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-wildfly-overlay/ovirt-engine-wildfly-overlay.yaml M jobs/confs/projects/ovirt-engine-wildfly/ovirt-engine-wildfly.yaml M jobs/confs/yaml/templates/create-rpms_mock.yaml 3 files changed, 8 insertions(+), 2 deletions(-) Approvals: Eyal Edri: Looks good to me; Passed CI tests; Ready for merge Gil Shinar: Verified -- To view, visit https://gerrit.ovirt.org/64920 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1daf53fb63b78aeb1ffe5590a8a20de51db57f80 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Evgheni Dereveanchin <ederevea(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Pavel Zhukov <pavel(a)zhukoff.net> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Fixed experimental flow for nsis-simple-service-plugin
by eedri＠redhat.com 28 Sep '16

28 Sep '16

Eyal Edri has submitted this change and it was merged. Change subject: Fixed experimental flow for nsis-simple-service-plugin ...................................................................... Fixed experimental flow for nsis-simple-service-plugin Create rpms of nsis-simple-service-plugin weren't executing experimental jobs. They were missing some parameters for that to work properly Change-Id: I617f156a76da7d0f4b63f79eae024698e2f52cd5 Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M jobs/confs/yaml/templates/create-rpms_mock.yaml 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Eyal Edri: Looks good to me; Ready for merge Gil Shinar: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64525 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I617f156a76da7d0f4b63f79eae024698e2f52cd5 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in gerrit-admin[master]: Simplify get_bug_url function
by sbendavi＠redhat.com 28 Sep '16

28 Sep '16

Shlomo Ben David has submitted this change and it was merged. Change subject: Simplify get_bug_url function ...................................................................... Simplify get_bug_url function It uses 'bug-url.*\d\b' regex as a search pattern in the commit message. If it exist, it will return a list of all bug-urls. Change-Id: I208a468adc9370789616366f250e2e49ce9e472f Signed-off-by: Shlomi Ben-David <sbendavi(a)redhat.com> --- M hooks/lib/bz.py 1 file changed, 5 insertions(+), 7 deletions(-) Approvals: Shlomo Ben David: Verified; Looks good to me, approved Gil Shinar: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64510 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I208a468adc9370789616366f250e2e49ce9e472f Gerrit-PatchSet: 1 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: Gil Shinar <gshinar(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-engine[master]: webadmin, userportal: Improve UiCommonEditorDriver cleanup() ...
by awels＠redhat.com 28 Sep '16

28 Sep '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin,userportal: Improve UiCommonEditorDriver cleanup() impl. ...................................................................... webadmin,userportal: Improve UiCommonEditorDriver cleanup() impl. Make sure that each call like getEditor().myField.cleanup(); is wrapped in if (getEditor().myField != null) block to avoid potential NPEs. Change-Id: I57f433ee4fbe2f89e1d1d083c84d01655b570a5e Bug-Url: https://bugzilla.redhat.com/1368101 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/binding/UiCommonEditorDriverGenerator.java 1 file changed, 10 insertions(+), 2 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/64535 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I57f433ee4fbe2f89e1d1d083c84d01655b570a5e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Add imgbase-clean-grub to the default services
by sbonazzo＠redhat.com 28 Sep '16

28 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Add imgbase-clean-grub to the default services ...................................................................... node: Add imgbase-clean-grub to the default services Make sure it starts at boot time. Change-Id: I4cafce1261b411f6bf4d132713ba455faaabadfb Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1346872 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M ovirt-release-master.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Ryan Barry: Verified -- To view, visit https://gerrit.ovirt.org/63507 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4cafce1261b411f6bf4d132713ba455faaabadfb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.5]: build: post ovirt-host-deploy-1.5.3
by sbonazzo＠redhat.com 28 Sep '16

28 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-host-deploy-1.5.3 ...................................................................... build: post ovirt-host-deploy-1.5.3 Change-Id: I260b349fbd51fcb9946d2aadf958f81eb9fcdd6e Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ChangeLog M automation/build-artifacts.sh M configure.ac 3 files changed, 8 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64923 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I260b349fbd51fcb9946d2aadf958f81eb9fcdd6e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.5 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-host-deploy[ovirt-host-deploy-1.5]: build: ovirt-host-deploy-1.5.3
by sbonazzo＠redhat.com 28 Sep '16

28 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-host-deploy-1.5.3 ...................................................................... build: ovirt-host-deploy-1.5.3 Change-Id: Ifcf7ea804b21fd8b0b26b28235a399dd090cc43e Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ChangeLog M automation/build-artifacts.sh M configure.ac M ovirt-host-deploy.spec.in 4 files changed, 10 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64922 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifcf7ea804b21fd8b0b26b28235a399dd090cc43e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.5 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]: core: use ligher db queries when querying host certificate s...
by ahadas＠redhat.com 28 Sep '16

28 Sep '16

Arik Hadas has submitted this change and it was merged. Change subject: core: use ligher db queries when querying host certificate subject ...................................................................... core: use ligher db queries when querying host certificate subject We can query only the dynamic part of a VM in order to retrieve the host the VM is running on and only the static part of a host in order to retrieve its certificate subject. Change-Id: Ife0875bebce5daa22dc26abb8268677e41884861 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVdsCertificateSubjectByVmIdQuery.java 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/64909 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife0875bebce5daa22dc26abb8268677e41884861 Gerrit-PatchSet: 1 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: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0