-
jbk
uugh... one of these days i'm going to completely fix coreadm
-
gitomat
[illumos-gate] 15366 ATA PASS-THROUGH (16) is broken -- Jason King <jason.brian.king⊙gc>
-
gitomat
[illumos-gate] 15810 TSC sync produces unreliable results with caches disabled -- Keith M Wesolowski <wesolows⊙oc>
-
gitomat
[illumos-gate] 15827 allow passing of extra arguments to 'pkg update' when running onu(1ONBLD) -- Hans Rosenfeld <rosenfeld⊙gho>
-
sjorge
andyf any plans for an omnios-bloody releas with the tsc changes ?
-
yuripv
Woodstock: looks like you missed "not" in "I did see any ill side effects" :)
-
Woodstock
oops.
-
jbk
at the risk of bikeshedding, how horrible would if be if when SMF logged events (e.g. `svcs -L logfile`), it included the timezone it was using (yes everyone should use GMT, reality is of course less perfect)
-
jbk
in it's timestamps
-
tsoome
I have to say, I do not like GMT;) When I need log, I need local TZ so I can correlate events as needed.
-
gitomat
[illumos-gate] 15773 loader: serial devices list should be dynamic -- Toomas Soome <tsoome⊙mc>
-
yuripv
tsoome: you can use UTC if you don't like GMT
-
tsoome
ou, really:D
-
yuripv
;)
-
jbk
well the proposal isn't to force the TZ, but just to add what TZ is actually being used at the time of the logging
-
tsoome
thats is definitely nice
-
tsoome
less confusion is good.
-
mns
I would think that would be default, no ?
-
sommerfeld
jbk: what timezone format are you proposing? The excruciatingly unambiguous way would be a GMT offset as I'm not sure the abbreviations are globally unique.
-
jbk
yeah..
-
jbk
just +nnnn at the end
-
sommerfeld
strftime %z does this I believe.
-
jbk
yeah, i don't think it'd be a massive code change -- i suspect it'd just be a matter of updating this:
github.com/illumos/illumos-gate/blo…r/usr/src/cmd/svc/startd/log.c#L358
-
jbk
to include '%z'
-
Woodstock
i wonder how many log parsers that'll break :)
-
jbk
well that's a concern.. though _technically_ i don't think SMF logs are meant to be consumed...
-
jbk
i suppose we could gate it behind a config option
-
jbk
so new installs get it, existing are unchanged unless explicitly updated
-
Woodstock
probably a heads up would be sufficient
-
sommerfeld
Log file format/contents don't appear to be documented. A heads up would be sufficient. If you were going to make the format tunable, probably best to go with a property that specifies a strftime format string rather than an old/new boolean.
-
gitomat
[illumos-gate] 14001 dboot: dump_tables off by one error -- Keith M Wesolowski <wesolows⊙oc>
-
mns
Making it a tunable property, with the default being what the current setting is, would be best option. Combined with a heads up. If anyone is using log parsers, it wont break them.