Change in ovirt-engine[master]: engine: add finish report on multi-host network commands
by masayag＠redhat.com
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>
8 years, 2 months
Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Make Reports-related Dashboard main tab initially ...
by tnisan＠redhat.com
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>
8 years, 2 months
Change in ovirt-engine[ovirt-engine-4.0]: packaging: setup: Allow provisioning postgresql role
by tnisan＠redhat.com
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>
8 years, 2 months
Change in ovirt-engine[ovirt-engine-4.0]: packaging: engine-backup: Create extra users on provisioning
by tnisan＠redhat.com
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>
8 years, 2 months
Change in otopi[master]: otopi-config-query: fix interpreter
by didi＠redhat.com
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>
8 years, 2 months
Change in jenkins[master]: remove check-merged from 3.6.9
by eedri＠redhat.com
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>
8 years, 2 months
Change in jenkins[master]: limit ppc64le builds for master only
by sbonazzo＠redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: packaging: engine-backup: Create extra users on provisioning
by sbonazzo＠redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: packaging: setup: Allow provisioning postgresql role
by sbonazzo＠redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: packaging: spec: require newest otopi
by sbonazzo＠redhat.com
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>
8 years, 3 months