Change in ovirt-host-deploy[master]: Revert "vdsm: Configure metrics"
by Code Review
>From Sandro Bonazzola <sbonazzo(a)redhat.com>:
Sandro Bonazzola has submitted this change and it was merged.
Change subject: Revert "vdsm: Configure metrics"
......................................................................
Revert "vdsm: Configure metrics"
This reverts commit 09ad206f33872964b8e2115e4aca0326dbb5d040.
Not needed, vdsm already sends metrics by default:
https://gerrit.ovirt.org/66257
Change-Id: Id5293e016c1626505b1c00c8fd1c585b9ac34504
Bug-Url: https://bugzilla.redhat.com/1414277
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M src/ovirt_host_deploy/constants.py
M src/plugins/ovirt-host-deploy/vdsm/Makefile.am
M src/plugins/ovirt-host-deploy/vdsm/__init__.py
D src/plugins/ovirt-host-deploy/vdsm/metrics.py
4 files changed, 0 insertions(+), 107 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Shirly Radco: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Yedidyah Bar David: Verified
--
To view, visit https://gerrit.ovirt.org/70797
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id5293e016c1626505b1c00c8fd1c585b9ac34504
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-host-deploy
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: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 11 months
Change in ovirt-host-deploy[master]: vdsm: Configure metrics
by Code Review
>From Sandro Bonazzola <sbonazzo(a)redhat.com>:
Sandro Bonazzola has submitted this change and it was merged.
Change subject: vdsm: Configure metrics
......................................................................
vdsm: Configure metrics
Write to statsd by default.
Change-Id: I476ac8757b9853a74a2195ad12e10391e331206a
Bug-Url: https://bugzilla.redhat.com/1414277
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M src/ovirt_host_deploy/constants.py
M src/plugins/ovirt-host-deploy/vdsm/Makefile.am
M src/plugins/ovirt-host-deploy/vdsm/__init__.py
A src/plugins/ovirt-host-deploy/vdsm/metrics.py
4 files changed, 107 insertions(+), 0 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/70713
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I476ac8757b9853a74a2195ad12e10391e331206a
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-host-deploy
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>
7 years, 11 months
Change in ovirt-engine[master]: packaging: Configure hosts for metrics: Fixes
by Code Review
>From Sandro Bonazzola <sbonazzo(a)redhat.com>:
Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: Configure hosts for metrics: Fixes
......................................................................
packaging: Configure hosts for metrics: Fixes
Allow collectd to listen on statsd port in selinux
Restart collectd after changing its conf
Change-Id: Ia665d1b127da5cec58d2b156954748eda0244fa6
Bug-Url: https://bugzilla.redhat.com/1405813
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
A packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/meta/main.yml
M packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/tasks/main.yml
A packaging/setup/ansible/roles/ovirt_collectd/read_statsd/meta/main.yml
M packaging/setup/ansible/roles/ovirt_collectd/read_statsd/tasks/main.yml
A packaging/setup/ansible/roles/ovirt_collectd/write_http/meta/main.yml
M packaging/setup/ansible/roles/ovirt_collectd/write_http/tasks/main.yml
6 files changed, 18 insertions(+), 9 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/70722
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia665d1b127da5cec58d2b156954748eda0244fa6
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: Rafael Martins <rmartins(a)redhat.com>
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>
7 years, 11 months
Change in ovirt-engine[master]: userportal, webadmin: Replace info icon
by Code Review
>From Alexander Wels <awels(a)redhat.com>:
Alexander Wels has submitted this change and it was merged.
Change subject: userportal,webadmin: Replace info icon
......................................................................
userportal,webadmin: Replace info icon
- Replaced dark blue ? icon with patternfly i icon and color.
- note this doesn't replace the ? on the dialog header.
Change-Id: I4ef4ccd1a65b431722300b45bf00c77a68094b78
Signed-off-by: Alexander Wels <awels(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/dialog/InfoIcon.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/dialog/TooltippedIcon.java
A frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/css/InfoIcon.css
3 files changed, 51 insertions(+), 7 deletions(-)
Approvals:
Alexander Wels: Verified
Jenkins CI: Passed CI tests
Greg Sheremeta: Verified; Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/67959
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4ef4ccd1a65b431722300b45bf00c77a68094b78
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <awels(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: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 11 months
Change in ovirt-log-collector[ovirt-log-collector-4.1]: build: post ovirt-log-collector-4.1.0-1
by Code Review
>From Rafael Martins <rmartins(a)redhat.com>:
Rafael Martins has submitted this change and it was merged.
Change subject: build: post ovirt-log-collector-4.1.0-1
......................................................................
build: post ovirt-log-collector-4.1.0-1
Change-Id: I0acaede25a6577156b6eba831879af84d8a89f86
Signed-off-by: Rafael Martins <rmartins(a)redhat.com>
---
M configure.ac
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Rafael Martins: Verified; Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/70764
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0acaede25a6577156b6eba831879af84d8a89f86
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-log-collector
Gerrit-Branch: ovirt-log-collector-4.1
Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 11 months
Change in ovirt-log-collector[ovirt-log-collector-4.1]: build: ovirt-log-collector-4.1.0-1
by Code Review
>From Rafael Martins <rmartins(a)redhat.com>:
Rafael Martins has submitted this change and it was merged.
Change subject: build: ovirt-log-collector-4.1.0-1
......................................................................
build: ovirt-log-collector-4.1.0-1
Change-Id: Ifdbbcd6a00f412f8c2687dd1d180673e99eb44d0
Signed-off-by: Rafael Martins <rmartins(a)redhat.com>
---
M configure.ac
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Rafael Martins: Verified; Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/70763
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifdbbcd6a00f412f8c2687dd1d180673e99eb44d0
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-log-collector
Gerrit-Branch: ovirt-log-collector-4.1
Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 11 months
Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: console: getting displayPort from VDSM
by Code Review
>From Simone Tiraboschi <stirabos(a)redhat.com>:
Simone Tiraboschi has submitted this change and it was merged.
Change subject: console: getting displayPort from VDSM
......................................................................
console: getting displayPort from VDSM
getting displayPort from VDSM for vnc and spice consoles.
For spice, extract the host certificate subject directly
from the cert.
Change-Id: I2c5495ad1d8b599a0435998b0952d94221fb0ba9
Bug-Url: https://bugzilla.redhat.com/1398447
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M src/bin/hosted-engine.in
1 file changed, 10 insertions(+), 5 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Simone Tiraboschi: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/70687
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2c5495ad1d8b599a0435998b0952d94221fb0ba9
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: ovirt-hosted-engine-setup-1.3
Gerrit-Owner: Simone Tiraboschi <stirabos(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: gerrit-hooks <automation(a)ovirt.org>
7 years, 11 months
Change in ovirt-engine[master]: webadmin: quota edit dialog
by Code Review
>From Alexander Wels <awels(a)redhat.com>:
Alexander Wels has submitted this change and it was merged.
Change subject: webadmin: quota edit dialog
......................................................................
webadmin: quota edit dialog
- Updated quota edit dialogs to be boostrap grid
based instead of tables.
Change-Id: I5afaf90adf4855a14e0e6843e464999940d32a58
Signed-off-by: Alexander Wels <awels(a)redhat.com>
---
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/quota/EditQuotaClusterPopupView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/quota/EditQuotaClusterPopupView.ui.xml
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/quota/EditQuotaStoragePopupView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/quota/EditQuotaStoragePopupView.ui.xml
4 files changed, 95 insertions(+), 147 deletions(-)
Approvals:
Alexander Wels: Verified
Jenkins CI: Passed CI tests
Greg Sheremeta: Looks good to me, approved
Vojtech Szocs: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/70057
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5afaf90adf4855a14e0e6843e464999940d32a58
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 11 months
Change in ovirt-release[ovirt-4.1]: build: ovirt-release41-4.1.0-0.8.rc1
by Code Review
>From Sandro Bonazzola <sbonazzo(a)redhat.com>:
Sandro Bonazzola has submitted this change and it was merged.
Change subject: build: ovirt-release41-4.1.0-0.8.rc1
......................................................................
build: ovirt-release41-4.1.0-0.8.rc1
Change-Id: Ibe1ccfbf44c199d0fe0c73e9d4e86ec94c40c42f
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M automation/build-artifacts.sh
M automation/check-patch.sh
M configure.ac
M ovirt-release41.spec.in
4 files changed, 7 insertions(+), 6 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/70754
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibe1ccfbf44c199d0fe0c73e9d4e86ec94c40c42f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-release
Gerrit-Branch: ovirt-4.1
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>
7 years, 11 months
Change in jenkins[master]: Added jobs for ovirt-{image, iso}-uploader in ovirt 4.1
by Code Review
>From Sandro Bonazzola <sbonazzo(a)redhat.com>:
Sandro Bonazzola has submitted this change and it was merged.
Change subject: Added jobs for ovirt-{image,iso}-uploader in ovirt 4.1
......................................................................
Added jobs for ovirt-{image,iso}-uploader in ovirt 4.1
Please note that as these tools are deprecated, we are building them
based on the 4.0 branches during the transition to engine built-in tools
Change-Id: I888cd17d85a90a47b58dc14216f8d974a3cd8fe4
Signed-off-by: Rafael Martins <rmartins(a)redhat.com>
---
M jobs/confs/projects/ovirt-image-uploader/ovirt-image-uploader_standard.yaml
M jobs/confs/projects/ovirt-iso-uploader/ovirt-iso-uploader_standard.yaml
2 files changed, 12 insertions(+), 3 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved; Ready for merge
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/70753
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I888cd17d85a90a47b58dc14216f8d974a3cd8fe4
Gerrit-PatchSet: 2
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years, 11 months