Bug 853883

Summary: [ovirt-engine-backend] logging: vdsBroker logs shows only vds id in (request to add vds name)
Product: Red Hat Enterprise Virtualization Manager Reporter: Dafna Ron <dron>
Component: ovirt-engineAssignee: Yaniv Bronhaim <ybronhei>
Status: CLOSED CURRENTRELEASE QA Contact: Dafna Ron <dron>
Severity: high Docs Contact:
Priority: high    
Version: 3.1.0CC: bazulay, dyasny, hateya, iheim, lpeer, mkenneth, Rhev-m-bugs, yeylon, ykaul, yzaslavs
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard: infra
Fixed In Version: si19 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-04 20:02:10 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
log none

Description Dafna Ron 2012-09-03 08:49:30 UTC
Created attachment 609298 [details]
log

Description of problem:

the engine log is showing only vds id in the logs. since the users do not know what the vds id is without checking the db it makes it really hard to debug. 


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

si16

How reproducible:

100%

Steps to Reproduce:
1. look at commands like GetDeviceList
2.
3.
  
Actual results:

we only see vds id

Expected results:

vds name should be added to all commands as well as the vds id - it will make it easier to debug when there is more than one host in the cluster. 

Additional info: backend log


example - GetDeviceList is sent to vds - how do we know which one it is? 

2012-09-03 11:15:56,731 INFO  [org.ovirt.engine.core.vdsbroker.vdsbroker.GetDeviceListVDSCommand] (ajp-/127.0.0.1:8009-8) [4a7e0b1f] START, GetDeviceListVDSCommand(vdsId = 8c28
9d3a-f4d7-11e1-8cda-001a4a169741, storageType=ISCSI), log id: 3acd1676

Comment 6 Yaniv Bronhaim 2012-09-09 11:59:36 UTC
upstream patch: http://gerrit.ovirt.org/#/c/7861/
and additional fix: http://gerrit.ovirt.org/#/c/7865/

Comment 8 Dafna Ron 2012-09-27 13:02:36 UTC
verified on si19