ACK, this shows timestamp now, with one comment inline. <div class="gmail_quote">On Tue, Feb 17, 2009 at 4:23 PM, Ian Main <<a href="mailto:imain@redhat.com">imain@redhat.com</a>> wrote: <blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Sadly there's a bug in rails that overrides the logger format causing the non-existant formatting we were seeing. This is ugly but fixes the issue. Signed-off-by: Ian Main <<a href="mailto:imain@redhat.com">imain@redhat.com</a>> --- src/task-omatic/taskomatic.rb | 13 +++++++++++++ 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/src/task-omatic/taskomatic.rb b/src/task-omatic/taskomatic.rb index 65aac32..f6f7f39 100755 --- a/src/task-omatic/taskomatic.rb +++ b/src/task-omatic/taskomatic.rb @@ -35,6 +35,19 @@ require 'task_vm' require 'task_storage' require 'vnc' +# This sad and pathetic readjustment to ruby logger class is +# required to fix the formatting because rails does the same +# thing but overrides it to just the message. +# +# See eg: <a href="http://osdir.com/ml/lang.ruby.rails.core/2007-01/msg00082.html" target="_blank">http://osdir.com/ml/lang.ruby.rails.core/2007-01/msg00082.html</a> +# +class Logger + def format_message(severity, timestamp, progname, msg) + "#{timestamp} (#{$$}) #{msg}\n"</blockquote><div>Why not include severity here as in default logger, especially since it is passed in? </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> + end +end + + class TaskOmatic include MonitorMixin -- 1.6.0.6 _______________________________________________ Ovirt-devel mailing list <a href="mailto:Ovirt-devel@redhat.com">Ovirt-devel@redhat.com</a> <a href="https://www.redhat.com/mailman/listinfo/ovirt-devel" target="_blank">https://www.redhat.com/mailman/listinfo/ovirt-devel</a> </blockquote></div>