August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: add finish report on multi-host network commands
by masayag＠redhat.com 28 Aug '16

28 Aug '16

Moti Asayag has submitted this change and it was merged. Change subject: engine: add finish report on multi-host network commands ...................................................................... engine: add finish report on multi-host network commands The change adds reporting of finish applying network changes to the hosts that were done asynchronously. Change-Id: I93eeccdd6695ddabbcfe26517fc372134ddc0e6a Bug-Url: https://bugzilla.redhat.com/1300220 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/util/HostIdToLoggableNameFunction.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/util/ReportFailedChildHostOperationsUtil.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/ManageNetworkClustersCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/PropagateLabeledNetworksToClusterHostsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/UpdateNetworkCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandCoordinatorUtil.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/host/util/HostIdToLoggableNameFunctionTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ManageNetworkClustersParameters.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 11 files changed, 283 insertions(+), 7 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/62081 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I93eeccdd6695ddabbcfe26517fc372134ddc0e6a Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> 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-4.0]: webadmin: Make Reports-related Dashboard main tab initially ...
by tnisan＠redhat.com 28 Aug '16

28 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Make Reports-related Dashboard main tab initially hidden ...................................................................... webadmin: Make Reports-related Dashboard main tab initially hidden In 4.0, even though we don't ship Reports anymore, UI still contains code for Reports integration: (Reports related) "Dashboard" main tab will be shown on the right-hand side of tab panel for a brief moment, until the UI detects unavailability of Reports & hiding that tab. This is unrelated to the "Dashboard" main tab shown on the left-hand side of tab panel, representing the oVirt Dashboard feature introduced in 4.0. This patch simply sets the (Reports related) "Dashboard" main tab to be initially hidden, which fixes the above mentioned UX issue. Due to Reports not available in 4.0, that tab will never become visible. Change-Id: I155ac581eb71febb55df84b339a6ab46b729dde7 Bug-Url: https://bugzilla.redhat.com/1365980 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/CommonModel.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/62219 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I155ac581eb71febb55df84b339a6ab46b729dde7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: packaging: setup: Allow provisioning postgresql role
by tnisan＠redhat.com 28 Aug '16

28 Aug '16

Tal Nisan 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/62849 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40c88ad48f8f7c2b8e06802137870b0c198b5129 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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> 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-4.0]: packaging: engine-backup: Create extra users on provisioning
by tnisan＠redhat.com 28 Aug '16

28 Aug '16

Tal Nisan 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> (cherry picked from commit be0cf8f79ec8cb263e50e8c1f8f755d7e7f8ac5d) --- M packaging/bin/engine-backup.sh.in 1 file changed, 49 insertions(+), 3 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/62850 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I61c7d2fd8eee8158434fc4b9df20c965195fdb00 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in otopi[master]: otopi-config-query: fix interpreter
by didi＠redhat.com 28 Aug '16

28 Aug '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: otopi-config-query: fix interpreter ...................................................................... otopi-config-query: fix interpreter Use /usr/bin/python instead of /usr/bin/env python Fixes rpmlint error: otopi.noarch: E: wrong-script-interpreter /usr/bin/otopi-config-query /usr/bin/env python Change-Id: If8221cc066dfa2ae19d36e2ceda79705ee7a9347 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/bin/otopi-config-query 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62861 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8221cc066dfa2ae19d36e2ceda79705ee7a9347 Gerrit-PatchSet: 1 Gerrit-Project: otopi 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: remove check-merged from 3.6.9
by eedri＠redhat.com 28 Aug '16

28 Aug '16

Eyal Edri has submitted this change and it was merged. Change subject: remove check-merged from 3.6.9 ...................................................................... remove check-merged from 3.6.9 We already run build artifacts for 3.6.9 and also check-merged for 3.6, this job has little use and not worth the resources. Change-Id: Ifbc5a164e82e7e4d0fffe410f18065e0ca3b0144 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 0 insertions(+), 6 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62877 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifbc5a164e82e7e4d0fffe410f18065e0ca3b0144 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

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