- Engine-commits - oVirt List Archives

Change in jenkins[master]: limit ppc64le builds for master only
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: limit ppc64le builds for master only ...................................................................... limit ppc64le builds for master only We need to let the ppc64le run for awhile on master and test them before backporting to 4.0 if at all. Change-Id: Ic6572e5ccacb1ef449d20c95f86f623a1e3a45fc Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/vdsm/vdsm_standard.yaml 1 file changed, 0 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62801 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic6572e5ccacb1ef449d20c95f86f623a1e3a45fc Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: engine-backup: Create extra users on provisioning
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: engine-backup: Create extra users on provisioning ...................................................................... packaging: engine-backup: Create extra users on provisioning If passing --provision-*, and the dump file containts extra GRANT statements, create these users before restore. Change-Id: I61c7d2fd8eee8158434fc4b9df20c965195fdb00 Bug-Url: https://bugzilla.redhat.com/1369757 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/bin/engine-backup.sh.in 1 file changed, 49 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved; Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/62844 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I61c7d2fd8eee8158434fc4b9df20c965195fdb00 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(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[master]: packaging: setup: Allow provisioning postgresql role
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Allow provisioning postgresql role ...................................................................... packaging: setup: Allow provisioning postgresql role ... without provisioning also a database. Change-Id: I40c88ad48f8f7c2b8e06802137870b0c198b5129 Bug-Url: https://bugzilla.redhat.com/1369757 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine_common/postgres.py M packaging/setup/plugins/ovirt-engine-provisiondb/ovirt-engine-common/provisioning/database.py 2 files changed, 136 insertions(+), 36 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/62823 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40c88ad48f8f7c2b8e06802137870b0c198b5129 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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[master]: packaging: spec: require newest otopi
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: require newest otopi ...................................................................... packaging: spec: require newest otopi Require otopi >= 1.6.0 which is targeted to 4.1. It includes a new feature which allow to get coverage reports when executing the setup. In order to enable the feature you'll need to provide a configuration file like: 8<-------------------------------------- [run] branch = True [report] exclude_lines = # Have to re-enable the standard pragma pragma: no cover # Don't complain about missing debug-only code: def __repr__ if self\.debug # Don't complain if tests don't hit defensive assertion code: raise AssertionError raise NotImplementedError # Don't complain if non-runnable code isn't run: if 0: if __name__ == .__main__.: ignore_errors = True 8<------------------------------------ Be sure to have python-coverage and libselinux-python installed on your system and then execute: export OTOPI_COVERAGE=1 export COVERAGE_PROCESS_START="automation/coverage.rc" engine-setup In order to generate the report: mkdir coverage_html_report coverage html -d coverage_html_report Change-Id: I41df99d82723e998eda9084b93063cd842d8f065 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.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/62871 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41df99d82723e998eda9084b93063cd842d8f065 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine 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-engine[master]: automation: add isort check
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: add isort check ...................................................................... automation: add isort check Added isort check for python code Issue: https://ovirt-jira.atlassian.net/browse/OVIRT-702 Change-Id: Icc4227e66641934153873c28cf53519ae3fce8d9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A .isort.cfg M Makefile A automation/check-patch.packages.fc24 M build/python-check.sh.in 4 files changed, 39 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62866 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc4227e66641934153873c28cf53519ae3fce8d9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine 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[master]: python: cleanup import lines using isort
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: python: cleanup import lines using isort ...................................................................... python: cleanup import lines using isort Ran again isort for passing automation checks introduced in next commit. Change-Id: I9db9cf798706e8fd364ecbf8470234078ef38a77 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.py M packaging/services/ovirt-engine/ovirt-engine.py M packaging/services/ovirt-fence-kdump-listener/listener.py M packaging/services/ovirt-fence-kdump-listener/ovirt-fence-kdump-listener.py M packaging/services/ovirt-websocket-proxy/ovirt-websocket-proxy.py M packaging/setup/plugins/ovirt-engine-common/base/network/process_firewalld_services.py 6 files changed, 11 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62865 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9db9cf798706e8fd364ecbf8470234078ef38a77 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine 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 jenkins[master]: ovirt-engine: added fc24 on master check patch
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine: added fc24 on master check patch ...................................................................... ovirt-engine: added fc24 on master check patch Added fc24 check patch to master since isort is not available in el7 Change-Id: I7481d37a4de7574491e68e411a9aa8aac358e3cd Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 9 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/62868 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7481d37a4de7574491e68e411a9aa8aac358e3cd Gerrit-PatchSet: 1 Gerrit-Project: jenkins 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 jenkins[master]: ovirt-engine: drop 4.0.2 jobs
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine: drop 4.0.2 jobs ...................................................................... ovirt-engine: drop 4.0.2 jobs 4.0.2 has been released, dropping jobs Change-Id: Ia211337654a796e0497954aa856ce6913bfa41d8 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62867 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia211337654a796e0497954aa856ce6913bfa41d8 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: automation: dropped el6 configs
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: dropped el6 configs ...................................................................... automation: dropped el6 configs el6 is not supported anymore since 4.0. Dropping el6 configuration. Change-Id: I5f326c79a9b4b4d3250cc0b3718508c330f059ea Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- D automation/check-patch.packages.el6 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62864 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5f326c79a9b4b4d3250cc0b3718508c330f059ea Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(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-engine-sdk[sdk_4.0]: README: add missing quote.
by omachace＠redhat.com 26 Aug '16

26 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: README: add missing quote. ...................................................................... README: add missing quote. Change-Id: I71e2c07e4c42eaa8e890793a37d8f904d92a42d2 Signed-off-by: Amit Aviram <aaviram(a)redhat.com> (cherry picked from commit edcc8191e227efebf274b6c155ff6f628f3eab1f) --- M sdk/README.adoc 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62847 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I71e2c07e4c42eaa8e890793a37d8f904d92a42d2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0