Change in ovirt-dwh[ovirt-engine-dwh-3.5]: history: fixed bug in users details sync
by sradco＠redhat.com
Shirly Radco has submitted this change and it was merged.
Change subject: history: fixed bug in users details sync
......................................................................
history: fixed bug in users details sync
Change-Id: I5fd093fde6a659994abab68db82a3c0328670529
Bug-Url:https://bugzilla.redhat.com/1189000
Signed-off-by: Shirly Radco <sradco(a)redhat.com>
---
M etl_export/src/ovirt_engine_dwh/configurationsync_3_5/ConfigurationSync.java
M tos_project/process/ConfigurationSync_3.5.item
M tos_project/process/ConfigurationSync_3.5.properties
3 files changed, 276 insertions(+), 296 deletions(-)
Approvals:
Shirly Radco: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/37516
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5fd093fde6a659994abab68db82a3c0328670529
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
Gerrit-Branch: ovirt-engine-dwh-3.5
Gerrit-Owner: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 11 months
Change in ovirt-dwh[master]: history: fixed bug in users details sync
by sradco＠redhat.com
Shirly Radco has submitted this change and it was merged.
Change subject: history: fixed bug in users details sync
......................................................................
history: fixed bug in users details sync
Change-Id: I5fd093fde6a659994abab68db82a3c0328670529
Bug-Url:https://bugzilla.redhat.com/1189000
Signed-off-by: Shirly Radco <sradco(a)redhat.com>
---
M etl_export/src/ovirt_engine_dwh/configurationsync_3_6/ConfigurationSync.java
M tos_project/process/ConfigurationSync_3.6.item
M tos_project/process/ConfigurationSync_3.6.properties
3 files changed, 571 insertions(+), 1,423 deletions(-)
Approvals:
Shirly Radco: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/37511
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5fd093fde6a659994abab68db82a3c0328670529
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
9 years, 11 months
Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: vdsm: use vdscli instead of vdsClient for storage operations
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: vdsm: use vdscli instead of vdsClient for storage operations
......................................................................
vdsm: use vdscli instead of vdsClient for storage operations
Dropping vdsClient usage in favor of vdscli API for storage
operations. vdsClient includes a non modificable 60 secs timeout
with could not be enough on long storage operations.
Change-Id: Iec7db5ccf55752266fa668a8b1e8f914cd632521
Bug-Url: https://bugzilla.redhat.com/1190207
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M src/ovirt_hosted_engine_setup/constants.py
M src/ovirt_hosted_engine_setup/mixins.py
M src/ovirt_hosted_engine_setup/tasks.py
M src/plugins/ovirt-hosted-engine-setup/storage/iscsi.py
M src/plugins/ovirt-hosted-engine-setup/storage/storage.py
M src/plugins/ovirt-hosted-engine-setup/system/vdsmenv.py
M src/plugins/ovirt-hosted-engine-setup/vm/boot_disk.py
M src/plugins/ovirt-hosted-engine-setup/vm/configurevm.py
M src/plugins/ovirt-hosted-engine-setup/vm/image.py
9 files changed, 94 insertions(+), 78 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Simone Tiraboschi: Verified
--
To view, visit http://gerrit.ovirt.org/37559
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iec7db5ccf55752266fa668a8b1e8f914cd632521
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: ovirt-hosted-engine-setup-1.2
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
9 years, 11 months
Change in ovirt-live[master]: Add CentOS 7 build
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: Add CentOS 7 build
......................................................................
Add CentOS 7 build
Moved centos to centos-6 and added centos-7
introducing CentOS 7 build with TintCore 6.
Bug-Url: https://bugzilla.redhat.com/1189842
Change-Id: I8a2d90850cf7e4e00511fb506ebce49d5f61564b
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
R centos-6/.gitignore
R centos-6/Makefile
R centos-6/kickstart/ovirt-live-base.ks
R centos-6/kickstart/ovirt-live-gnome.ks
R centos-6/kickstart/ovirt-live-ovirt-custom.ks.in
R centos-6/oVirtLiveFiles/root/etc/gdm/custom.conf
R centos-6/oVirtLiveFiles/root/etc/ovirt-host-deploy.conf.d/50-vdsm-conf-fake-nic.conf
R centos-6/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-dummy0
R centos-6/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-eth0
R centos-6/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-ovirtmgmt
R centos-6/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-wlan0
R centos-6/oVirtLiveFiles/root/etc/sysconfig/network-scripts/route-ovirtmgmt
R centos-6/oVirtLiveFiles/root/etc/sysconfig/network-scripts/rule-ovirtmgmt
R centos-6/oVirtLiveFiles/root/etc/xdg/autostart/engine-setup.desktop
R centos-6/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/engine-setup-yad.sh
R centos-6/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-icon-128.png
R centos-6/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-setup.png
R centos-6/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-usb-stick.png
R centos-6/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-usb-stick.svg
R centos-6/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-wallpaper-16:10.jpg
R centos-6/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-wallpaper-16:9.jpg
R centos-6/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-wallpaper-4:3.jpg
R centos-6/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/ovirt-answer
R centos-6/oVirtLiveFiles/root/usr/share/applications/ovirt.desktop
R centos-6/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.desktop.lockdown.gschema.override
R centos-6/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.desktop.screensaver.gschema.override
R centos-6/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.settings-daemon.plugins.updates.gschema.override
R centos-6/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.shell.gschema.override
R centos-6/oVirtLiveFiles/root/usr/share/ovirt-engine/setup/plugins/ovirt-engine-setup/olive/__init__.py
R centos-6/oVirtLiveFiles/root/usr/share/ovirt-engine/setup/plugins/ovirt-engine-setup/olive/core.py
R centos-6/oVirtLiveFiles/root/usr/share/ovirt-engine/setup/plugins/ovirt-engine-setup/olive/oliveconst.py
C centos-7/.gitignore
A centos-7/Makefile
A centos-7/kickstart/centos-7-livecd.cfg
A centos-7/kickstart/ovirt-live-ovirt-custom.ks.in
C centos-7/oVirtLiveFiles/root/etc/gdm/custom.conf
C centos-7/oVirtLiveFiles/root/etc/ovirt-host-deploy.conf.d/50-vdsm-conf-fake-nic.conf
C centos-7/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-dummy0
C centos-7/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-eth0
C centos-7/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-ovirtmgmt
C centos-7/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-wlan0
C centos-7/oVirtLiveFiles/root/etc/sysconfig/network-scripts/route-ovirtmgmt
C centos-7/oVirtLiveFiles/root/etc/sysconfig/network-scripts/rule-ovirtmgmt
C centos-7/oVirtLiveFiles/root/etc/xdg/autostart/engine-setup.desktop
C centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/engine-setup-yad.sh
C centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-icon-128.png
C centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-setup.png
C centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-usb-stick.png
C centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-usb-stick.svg
C centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-wallpaper-16:10.jpg
C centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-wallpaper-16:9.jpg
C centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/images/ovirt-wallpaper-4:3.jpg
C centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/ovirt-answer
C centos-7/oVirtLiveFiles/root/usr/share/applications/ovirt.desktop
C centos-7/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.desktop.lockdown.gschema.override
C centos-7/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.desktop.screensaver.gschema.override
C centos-7/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.settings-daemon.plugins.updates.gschema.override
C centos-7/oVirtLiveFiles/root/usr/share/glib-2.0/schemas/org.gnome.shell.gschema.override
C centos-7/oVirtLiveFiles/root/usr/share/ovirt-engine/setup/plugins/ovirt-engine-setup/olive/__init__.py
C centos-7/oVirtLiveFiles/root/usr/share/ovirt-engine/setup/plugins/ovirt-engine-setup/olive/core.py
C centos-7/oVirtLiveFiles/root/usr/share/ovirt-engine/setup/plugins/ovirt-engine-setup/olive/oliveconst.py
61 files changed, 1,538 insertions(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/37564
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8a2d90850cf7e4e00511fb506ebce49d5f61564b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-live
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
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: automation(a)ovirt.org
9 years, 11 months
Change in ovirt-hosted-engine-setup[master]: vdsm: use vdscli instead of vdsClient
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: vdsm: use vdscli instead of vdsClient
......................................................................
vdsm: use vdscli instead of vdsClient
Dropping vdsClient usage in favor of
vdscli API.
Using vdscli allows to achieve a better error handling
and to have more control on what's going on.
Bug-Url: https://bugzilla.redhat.com/1144334
Bug-Url: https://bugzilla.redhat.com/1101553
Bug-Url: https://bugzilla.redhat.com/1142098
Related-To: https://bugzilla.redhat.com/1005923
Related-To: https://bugzilla.redhat.com/1150427
Change-Id: I0f17abcc4ec83a69832bdb5a986136831504da2c
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M src/ovirt_hosted_engine_setup/constants.py
M src/ovirt_hosted_engine_setup/mixins.py
M src/ovirt_hosted_engine_setup/tasks.py
M src/plugins/ovirt-hosted-engine-setup/engine/add_disk.py
M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py
M src/plugins/ovirt-hosted-engine-setup/engine/os_install.py
M src/plugins/ovirt-hosted-engine-setup/ha/ha_services.py
M src/plugins/ovirt-hosted-engine-setup/network/bridge.py
M src/plugins/ovirt-hosted-engine-setup/storage/nfs.py
M src/plugins/ovirt-hosted-engine-setup/storage/storage.py
M src/plugins/ovirt-hosted-engine-setup/system/vdsmenv.py
M src/plugins/ovirt-hosted-engine-setup/vdsmd/cpu.py
M src/plugins/ovirt-hosted-engine-setup/vm/boot_disk.py
M src/plugins/ovirt-hosted-engine-setup/vm/configurevm.py
M src/plugins/ovirt-hosted-engine-setup/vm/image.py
M src/plugins/ovirt-hosted-engine-setup/vm/runvm.py
16 files changed, 264 insertions(+), 148 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/33996
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0f17abcc4ec83a69832bdb5a986136831504da2c
Gerrit-PatchSet: 18
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <toni+ovirt(a)midokura.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
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: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
9 years, 11 months
Change in ovirt-hosted-engine-setup[master]: packaging: setup: allow to disable iptables overriding
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: setup: allow to disable iptables overriding
......................................................................
packaging: setup: allow to disable iptables overriding
When iptables automatic configuration is not enabled,
avoid to override iptables configuration when adding the host.
Change-Id: I8244440989f486b1006fbe05151cb1fc9aa1fa1d
Bug-Url: https://bugzilla.redhat.com/1080823
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/37535
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8244440989f486b1006fbe05151cb1fc9aa1fa1d
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
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: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
9 years, 11 months
Change in ovirt-hosted-engine-setup[master]: packaging: spec: require iptables-services
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: spec: require iptables-services
......................................................................
packaging: spec: require iptables-services
On Fedora >= 21 and EL >= 7.0 iptables-services is needed
for having iptables working properly.
Change-Id: I12139f9a9f5d4d542e42de00c7a479a2fcc0ccce
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M ovirt-hosted-engine-setup.spec.in
1 file changed, 6 insertions(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/37557
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I12139f9a9f5d4d542e42de00c7a479a2fcc0ccce
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
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: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
9 years, 11 months
Change in ovirt-engine[master]: packaging: setup: Save remote engine password in answer file
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: setup: Save remote engine password in answer file
......................................................................
packaging: setup: Save remote engine password in answer file
Change-Id: I12a02e088bb39a78e2c6cc20d9cd21ade94c67d9
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M packaging/setup/plugins/ovirt-engine-common/base/remote_engine/remote_engine_root_ssh.py
1 file changed, 3 insertions(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
oVirt Jenkins CI Server: Looks good to me, but someone else must approve
Yedidyah Bar David: Verified
--
To view, visit http://gerrit.ovirt.org/37491
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I12a02e088bb39a78e2c6cc20d9cd21ade94c67d9
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
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: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
9 years, 11 months
Change in jenkins[master]: jobs: drop hosted engine build on el6 for master
by dcaroest＠redhat.com
David Caro has submitted this change and it was merged.
Change subject: jobs: drop hosted engine build on el6 for master
......................................................................
jobs: drop hosted engine build on el6 for master
oVirt 3.6 won't support 3.6 cluster compatibility level
on EL6.
Hosted Engine won't be supported on EL6 accordingly.
Change-Id: I4bca3e17ac64ece9db7b60b1cfccc6fd314e4d2b
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M jobs/confs/yaml/jobs/ovirt-hosted-engine-ha/ovirt-hosted-engine-ha_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-hosted-engine-setup/ovirt-hosted-engine-setup_create-rpms.yaml
2 files changed, 0 insertions(+), 7 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, but someone else must approve
David Caro: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/37546
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4bca3e17ac64ece9db7b60b1cfccc6fd314e4d2b
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
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: oVirt Jenkins CI Server
9 years, 11 months
Change in ovirt-engine[master]: restapi: Improve logging of exceptions
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: restapi: Improve logging of exceptions
......................................................................
restapi: Improve logging of exceptions
Currently when an exception is detected while rendering an document the
details are swallowed by the REST framework and lost. This complicates
debugging. To avoid that this patch changes the exception handler so
that it will send the details of the exception to the log.
Change-Id: I8085ae6d09ab49f1706b6c38d5b3e79b712f0241
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/IOExceptionMapper.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/JsonExceptionMapper.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/MalformedIdExceptionMapper.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/MappingExceptionMapper.java
4 files changed, 52 insertions(+), 1 deletion(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
Martin Mucha: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/37550
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8085ae6d09ab49f1706b6c38d5b3e79b712f0241
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
9 years, 11 months