Bug 672828

Summary: Spice guest logs are not user-friendly
Product: Red Hat Enterprise Linux 8 Reporter: Marina Kalinin <mkalinin>
Component: spice-vdagent-winAssignee: David Blechter <dblechte>
Status: CLOSED CURRENTRELEASE QA Contact: Desktop QE <desktop-qa-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: ---CC: acathrow, cpelland, dblechte, ddumas, hdegoede, iheim, jkt, lkocman, mkenneth, mkrcmari, Rhev-m-bugs
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Windows   
Whiteboard: gss_support
Fixed In Version: vdagent-win-0.1-9 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-01-21 15:09:57 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
change log times to human readable date & time none

Description Marina Kalinin 2011-01-26 14:47:31 UTC 
Spice logs on the guest machine (vdagent.log, vdservice.log) are not user friendly. They confuse users and does not allow to debug problems properly.

1. The time is on seconds, which is almost meaningless, when the log is received from a customer, specifying the time of his problem occurred. 
Please change to normal date format (i.e. system date and time).

2. Normal behaviour is represented as a failure.
For example:
- 711.218 VDService::execute: GetOverlappedResult failed 109  (failed sounds bad)
- 711.218 VDService::kill_agent: vdagent exit code 1073807364 (not 0, sounds like a problem)

Please remove those messages from the log, or change the text or add different log levels.

Thanks ;)
Comment 4 David Blechter 2011-07-06 11:56:11 UTC 
pm_ack is set to "?", so cleared devel_ack and qa_ack flags
Comment 5 Arnon Gilboa 2011-11-21 09:48:45 UTC 
Created attachment 534750 [details]
change log times to human readable date & time
Comment 6 Arnon Gilboa 2011-11-22 13:14:06 UTC 
This patch helps solving customer issues by GSS support and engineering. It's not risky from other perspectives.
Proposing to rhevm‑3.0.0.
Comment 7 Itamar Heim 2011-11-23 08:21:16 UTC 
(In reply to comment #6)
> This patch helps solving customer issues by GSS support and engineering. It's
> not risky from other perspectives.
> Proposing to rhevm‑3.0.0.

3.0 is past exception phase. you need to suggest is as a blocker, not exception.
also, reminding that last RC build date is November 28th.
Comment 8 Marian Krcmarik 2011-11-28 17:44:01 UTC 
Verified on vdagent-win-0.1-9.
(verified that - "change log times to human readable date & time", not the meaning of messages).