February 2015 - Engine-commits - oVirt List Archives

Change in ovirt-dwh[ovirt-engine-dwh-3.5]: history: fixed bug in users details sync
by sradco＠redhat.com 08 Feb '15

08 Feb '15

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

1 0

Change in ovirt-dwh[master]: history: fixed bug in users details sync
by sradco＠redhat.com 08 Feb '15

08 Feb '15

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

1 0

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 06 Feb '15

06 Feb '15

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

1 0

Change in ovirt-live[master]: Add CentOS 7 build
by sbonazzo＠redhat.com 06 Feb '15

06 Feb '15

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

1 0

Change in ovirt-hosted-engine-setup[master]: vdsm: use vdscli instead of vdsClient
by sbonazzo＠redhat.com 06 Feb '15

06 Feb '15

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

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: allow to disable iptables overriding
by sbonazzo＠redhat.com 06 Feb '15

06 Feb '15

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

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: spec: require iptables-services
by sbonazzo＠redhat.com 06 Feb '15

06 Feb '15

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

1 0

Change in ovirt-engine[master]: packaging: setup: Save remote engine password in answer file
by sbonazzo＠redhat.com 06 Feb '15

06 Feb '15

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

1 0

Change in jenkins[master]: jobs: drop hosted engine build on el6 for master
by dcaroest＠redhat.com 05 Feb '15

05 Feb '15

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

1 0

Change in ovirt-engine[master]: restapi: Improve logging of exceptions
by juan.hernandez＠redhat.com 05 Feb '15

05 Feb '15

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

1 0