search

yast / yast-ruby-bindings

YaST module ruby-bindings
http://en.opensuse.org/Portal:YaST
GNU General Public License v2.0
6 stars 14 forks source link

Nicer backtrace #206

Closed jreidinger closed 7 years ago

jreidinger commented 7 years ago

how output now looks like ( with testing client that just raise error ):

2017-09-26 15:19:23 <3> pepa(2635) [Ruby] yast/wfm.rb:253 Client call failed with 'funny' (RuntimeError).
Backtrace:
/tmp/test.rb:1:in `<top (required)>'
/usr/lib64/ruby/vendor_ruby/2.4.0/yast/wfm.rb:300:in `eval'
/usr/lib64/ruby/vendor_ruby/2.4.0/yast/wfm.rb:300:in `run_client'
/usr/lib64/ruby/vendor_ruby/2.4.0/yast/wfm.rb:206:in `call_builtin'
/usr/lib64/ruby/vendor_ruby/2.4.0/yast/wfm.rb:206:in `call_builtin_wrapper'
/usr/lib64/ruby/vendor_ruby/2.4.0/yast/wfm.rb:195:in `CallFunction'
/usr/lib/YaST2/bin/y2start:56:in `<main>'
lslezak commented 7 years ago

Would it be possible to repeat the log data for each split line? E.g.

2017-09-26 15:19:23 ... Backtrace: 2017-09-26 15:19:23 ... /tmp/test.rb:1:in `<top (required)>'

lslezak commented 7 years ago

The missing tags might confuse some y2log parsers.

lslezak commented 7 years ago

OK, let's keep it simple...

shundhammer commented 7 years ago

We have a lot more things that don't add that huge prefix to y2log lines, so any log parser should already be used to that.