January 2017 - Engine-commits - oVirt List Archives

Change in ovirt-host-deploy[master]: Revert "vdsm: Configure metrics"
by Code Review 19 Jan '17

19 Jan '17

>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>

1 0

Change in ovirt-host-deploy[master]: vdsm: Configure metrics
by Code Review 19 Jan '17

19 Jan '17

>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>

1 0

Change in ovirt-engine[master]: packaging: Configure hosts for metrics: Fixes
by Code Review 19 Jan '17

19 Jan '17

>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>

1 0

Change in ovirt-engine[master]: userportal, webadmin: Replace info icon
by Code Review 18 Jan '17

18 Jan '17

>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>

1 0

Change in ovirt-log-collector[ovirt-log-collector-4.1]: build: post ovirt-log-collector-4.1.0-1
by Code Review 18 Jan '17

18 Jan '17

>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>

1 0

Change in ovirt-log-collector[ovirt-log-collector-4.1]: build: ovirt-log-collector-4.1.0-1
by Code Review 18 Jan '17

18 Jan '17

>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>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: console: getting displayPort from VDSM
by Code Review 18 Jan '17

18 Jan '17

>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>

1 0

Change in ovirt-engine[master]: webadmin: quota edit dialog
by Code Review 18 Jan '17

18 Jan '17

>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>

1 0

Change in ovirt-release[ovirt-4.1]: build: ovirt-release41-4.1.0-0.8.rc1
by Code Review 18 Jan '17

18 Jan '17

>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>

1 0

Change in jenkins[master]: Added jobs for ovirt-{image, iso}-uploader in ovirt 4.1
by Code Review 18 Jan '17

18 Jan '17

>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>

1 0