August 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: host-deploy: remove obsolete protocol
by masayag＠redhat.com 16 Aug '15

16 Aug '15

Moti Asayag has submitted this change and it was merged. Change subject: host-deploy: remove obsolete protocol ...................................................................... host-deploy: remove obsolete protocol Change-Id: I53a974daa9257c12814ef109451722886b9e4eb6 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/RegisterServlet.java 1 file changed, 2 insertions(+), 73 deletions(-) Approvals: Douglas Schilling Landgraf: Looks good to me, but someone else must approve Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44449 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53a974daa9257c12814ef109451722886b9e4eb6 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: host-deploy: remove timestamp cache check for tarball
by masayag＠redhat.com 16 Aug '15

16 Aug '15

Moti Asayag has submitted this change and it was merged. Change subject: host-deploy: remove timestamp cache check for tarball ...................................................................... host-deploy: remove timestamp cache check for tarball when multiple packages manage the directory tree, the last file timestamp cannot be used as reference to cache integrity as an older package may be applied after newer one. we could have used hash mechanism, but then we had to read files once per hash and once per creation, so better to just create the tarball at designated interval. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1252364 Change-Id: I97fa20e0f5205ebddb6aff1cbfebf3f2879fa34e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/archivers/tar/CachedTar.java 1 file changed, 4 insertions(+), 49 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44690 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I97fa20e0f5205ebddb6aff1cbfebf3f2879fa34e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: print response of verbs that query external vms
by ahadas＠redhat.com 16 Aug '15

16 Aug '15

Arik Hadas has submitted this change and it was merged. Change subject: core: print response of verbs that query external vms ...................................................................... core: print response of verbs that query external vms Print the return value of VDSM verbs that return external VMs. For now these are query VMware and OVA file. Change-Id: Ic7020f4e948831281fafbf844206487378a6a884 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/GetOvaInfoVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/GetVmsFromExternalProviderVDSCommand.java 2 files changed, 0 insertions(+), 10 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44760 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic7020f4e948831281fafbf844206487378a6a884 Gerrit-PatchSet: 3 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: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: uutils: HttpURLConnectionBuilder: add connection timeout
by masayag＠redhat.com 16 Aug '15

16 Aug '15

Moti Asayag has submitted this change and it was merged. Change subject: uutils: HttpURLConnectionBuilder: add connection timeout ...................................................................... uutils: HttpURLConnectionBuilder: add connection timeout Change-Id: I17631642f5696f18a66d08d66b143006865fce06 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/uutils/src/main/java/org/ovirt/engine/core/uutils/net/HttpURLConnectionBuilder.java 1 file changed, 9 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44852 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17631642f5696f18a66d08d66b143006865fce06 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Added engine check-patch
by sbonazzo＠redhat.com 14 Aug '15

14 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Added engine check-patch ...................................................................... Added engine check-patch Change-Id: I77afd8b76fe4c7456ec8c7b2f86909e155a88eac Signed-off-by: David Caro <dcaroest(a)redhat.com> --- D jobs/confs/projects/ovirt-engine/ovirt-engine_build-artifacts.yaml A jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 2 files changed, 38 insertions(+), 20 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified; Ready for review -- To view, visit https://gerrit.ovirt.org/44841 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77afd8b76fe4c7456ec8c7b2f86909e155a88eac Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: frontend: Fix SerialNumberPolicyWidget layout
by tjelinek＠redhat.com 14 Aug '15

14 Aug '15

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: Fix SerialNumberPolicyWidget layout ...................................................................... frontend: Fix SerialNumberPolicyWidget layout Change-Id: Ia0e32f3b57dd09df0b6dd1977fd93d3511c39ad5 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1166056 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/SerialNumberPolicyWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/SerialNumberPolicyWidget.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml 4 files changed, 26 insertions(+), 10 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Verified -- To view, visit https://gerrit.ovirt.org/44848 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia0e32f3b57dd09df0b6dd1977fd93d3511c39ad5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: Added dummy check-patch
by piotr.kliczewski＠gmail.com 14 Aug '15

14 Aug '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: Added dummy check-patch ...................................................................... Added dummy check-patch Because not always will a job run for ovirt-engine, this ensures that at least one job will run and the CI flag will be set. The flow where this scripts comes into play, is when you submit a patch, and before jenkins had time to give a review, you submit a new patch that does no code changes (like just changing the commit message). At that point, gerrit has no previous review to carry over to the new patch and jenkins will not get triggered as it does not have any code changes, leaving that patch without CI flag at all. With this dummy script (and the related job) in that case, this job will do the dummy review and add the CI flag. Change-Id: I6d636b5538fa81adc2bc0644bd20d50e2d89f82e Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A automation/check-patch.sh 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44840 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d636b5538fa81adc2bc0644bd20d50e2d89f82e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: Added dummy check-patch
by sbonazzo＠redhat.com 14 Aug '15

14 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Added dummy check-patch ...................................................................... Added dummy check-patch Because not always will a job run for ovirt-engine, this ensures that at least one job will run and the CI flag will be set. The flow where this scripts comes into play, is when you submit a patch, and before jenkins had time to give a review, you submit a new patch that does no code changes (like just changing the commit message). At that point, gerrit has no previous review to carry over to the new patch and jenkins will not get triggered as it does not have any code changes, leaving that patch without CI flag at all. With this dummy script (and the related job) in that case, this job will do the dummy review and add the CI flag. Change-Id: I6d636b5538fa81adc2bc0644bd20d50e2d89f82e Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A automation/check-patch.sh 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44839 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d636b5538fa81adc2bc0644bd20d50e2d89f82e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-live[ovirt-live-3.6]: build: ovirt-live-3.6.0_beta2
by sbonazzo＠redhat.com 14 Aug '15

14 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-live-3.6.0_beta2 ...................................................................... build: ovirt-live-3.6.0_beta2 Change-Id: I2739e5eb65df2f6ee552d7889d1461dca322710d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44846 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2739e5eb65df2f6ee552d7889d1461dca322710d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Fixed issues with sending email. Group patches per owner whi...
by eedri＠redhat.com 14 Aug '15

14 Aug '15

Eyal Edri has submitted this change and it was merged. Change subject: Fixed issues with sending email. Group patches per owner while sending email. Add a debug option to print the gerrit command. Add command line option to give a test email along with dry-run option. Updated the gerrit commands acoording to the v2.11.2 docu ...................................................................... Fixed issues with sending email. Group patches per owner while sending email. Add a debug option to print the gerrit command. Add command line option to give a test email along with dry-run option. Updated the gerrit commands acoording to the v2.11.2 documentation. Change-Id: Ib51ac7fb29aa8f0298bfb0a82f7f5003ade8a3cf Signed-off-by: Vishnu Sreekumar <vishnu.srkmr(a)gmail.com> --- M scripts/alert_old_patches.py 1 file changed, 103 insertions(+), 60 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/43007 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib51ac7fb29aa8f0298bfb0a82f7f5003ade8a3cf Gerrit-PatchSet: 5 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Vishnu Sreekumar <vishnu.srkmr(a)gmail.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>

1 0