Change in ovirt-engine[master]: host-deploy: remove obsolete protocol
by masayag＠redhat.com
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
9 years, 1 month
Change in ovirt-engine[master]: host-deploy: remove timestamp cache check for tarball
by masayag＠redhat.com
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
9 years, 1 month
Change in ovirt-engine[master]: core: print response of verbs that query external vms
by ahadas＠redhat.com
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
9 years, 1 month
Change in ovirt-engine[master]: uutils: HttpURLConnectionBuilder: add connection timeout
by masayag＠redhat.com
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
9 years, 1 month
Change in jenkins[master]: Added engine check-patch
by sbonazzo＠redhat.com
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>
9 years, 1 month
Change in ovirt-engine[master]: frontend: Fix SerialNumberPolicyWidget layout
by tjelinek＠redhat.com
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
9 years, 1 month
Change in ovirt-engine[ovirt-engine-3.6]: Added dummy check-patch
by piotr.kliczewski＠gmail.com
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>
9 years, 1 month
Change in ovirt-engine[master]: Added dummy check-patch
by sbonazzo＠redhat.com
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
9 years, 1 month
Change in ovirt-live[ovirt-live-3.6]: build: ovirt-live-3.6.0_beta2
by sbonazzo＠redhat.com
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
9 years, 1 month
Change in jenkins[master]: Fixed issues with sending email. Group patches per owner whi...
by eedri＠redhat.com
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>
9 years, 1 month