New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 683275 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

DUT timestamps in kernel log vary between GMT and PST/PDT

Project Member Reported by semenzato@chromium.org, Jan 20 2017

Issue description

We can deal with it, but it's one additional mental step when matching times of events across logs, and can easily confuse less experienced sheriffs.

My guess is that the CMOS clock is kept in GMT all start in GMT, then figure out what time zone they are in, and adjust it.  However, most of the kernel logs I see have a preponderance of GMT time stamps.  Maybe there is a bias because I tend to look at logs for "device not pingable" or "DUT did not reboot" problems.

Adding the time zone to the time stamps may be overkill.  Maybe add a single character (like *) for GMT, and leave alone otherwise?

Or maybe do nothing. :/



 

Comment 1 by autumn@chromium.org, Jan 24 2017

Labels: Hotlist-Fixit
Owner: semenzato@chromium.org
Status: Assigned (was: Untriaged)
What's actually being requested here?  There's a known issue
with /var/log/messages in Chrome OS shifting from GMT to the
local timezone during boot.  That's not a test infrastructure
bug, it's a Chrome OS bug (and one we likely can't fix).

I think it would be relatively easy to tinker with the timestamp
in autotest logs to include a timezone, or to otherwise change
the time format.  Adding a timezone makes me nervous, though.
That change would add quite a few columns to log files that get
quite wide as is.  I'm not sure that cost is worth the benefit.

Cc: jrbarnette@chromium.org
Status: WontFix (was: Assigned)
The question was whether to add a '*' to the time stamp when it's in GMT.  We could also add 'GMT', but I agree that's a lot of columns.

But I guess that the * won't really help with the confusion.

Labels: cros-infra-fixedit-q117

Sign in to add a comment