Bug 903312

Summary: Additional debug logging required within the GenericApiGWTServiceImpl class to ease troubleshooting of the webadmin / userportal.
Product: Red Hat Enterprise Virtualization Manager Reporter: Lee Yarwood <lyarwood>
Component: ovirt-engine-webadmin-portalAssignee: Alexander Wels <awels>
Status: CLOSED CURRENTRELEASE QA Contact: Jiri Belka <jbelka>
Severity: medium Docs Contact:
Priority: medium    
Version: 3.1.1CC: acathrow, dyasny, ecohen, iheim, lpeer, Rhev-m-bugs, sgrinber, yeylon, ykaul
Target Milestone: ---   
Target Release: 3.2.0   
Hardware: All   
OS: Linux   
Whiteboard: ux
Fixed In Version: sf13.1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 902971    

Description Lee Yarwood 2013-01-23 17:18:18 UTC 
Description of problem:

At present the GenericApiGWTServiceImpl class fails to log key information regarding the type of each query being made from a given client.

For example with DEBUG enabled we see lines like :

~~~
28097-2013-01-23 09:55:30,349 DEBUG [org.ovirt.engine.ui.frontend.server.gwt.GenericApiGWTServiceImpl] (ajp-/127.0.0.1:8702-1) Server: RunQuery invoked!
28098:2013-01-23 09:55:30,349 DEBUG [org.ovirt.engine.ui.frontend.server.gwt.GenericApiGWTServiceImpl] (ajp-/127.0.0.1:8702-1) IP [10.30.1.107], Session ID [bBZFi0UO5BB-wtc042UfYMGa]
~~~

It would be helpful if more details were logged here.

Version-Release number of selected component (if applicable):
3.1.1

How reproducible:
Always

Steps to Reproduce:
1. Enable DEBUG messages.
2. Monitor engine.log
  
Actual results:
Lack of key information such as query type etc.

Expected results:
Key information exposed when DEBUG messages are enabled.

Additional info:
Comment 1 Einav Cohen 2013-02-06 14:02:38 UTC 
Need to specify the VdcQueryType in the message.

Also:

- In case VdcQueryType = GetConfigurationValue, need to specify in the message also the configuration-value and version that were requested.

- In case VdcQueryType = Search, need to specify in the message also the search-text requested.
Comment 4 Jiri Belka 2013-04-23 12:51:43 UTC 
OK, sf13.1. I can see lines like:

2013-04-23 14:47:46,875 DEBUG [org.ovirt.engine.ui.frontend.server.gwt.GenericApiGWTServiceImpl] (ajp-/127.0.0.1:8702-4) Query type: GetAllBookmarks, Parameters: refresh: true, filtered: false
Comment 5 Itamar Heim 2013-06-11 08:36:54 UTC 
3.2 has been released
Comment 6 Itamar Heim 2013-06-11 08:36:55 UTC 
3.2 has been released
Comment 7 Itamar Heim 2013-06-11 08:36:57 UTC 
3.2 has been released
Comment 8 Itamar Heim 2013-06-11 08:45:19 UTC 
3.2 has been released