Hello,
On Tue, 30 Jul 2013 10:48:53 -0300 Antonio Terceiro antonio.terceiro@linaro.org wrote:
Hi Milosz,
On Tue, Jul 30, 2013 at 11:49:59AM +0100, Milosz Wasilewski wrote:
Hi,
I made a small modification to lava-scheduler app code that enabled direct linking to line numbers in full log view. The branch was posted to launchpad: http://bazaar.launchpad.net/~mwasilew/lava-scheduler/log_linenumbers/revisio...
The solution isn't perfect as it hardcodes HTML tags into django templatetag, but I wanted minimal changes in the scheduler app code.
If you like it I will request the merge :)
independent of whether we already like it or not, please request the merge. We'll review the patch in the merge proposal, and let you know what needs to be fixed if anything. :)
But we definitely like the functionally offered and overall discussion which ensued on improving LAVA log view - it's indeed too cumbersome too deal with LAVA logs now. The rule of thumb I reduced to is that if I don't have at least half-day to spend on debugging LAVA job issues, then it's better not to poke at it at all, because an hour can be gone and little figured out (times of course include trying fixes and waiting for job re-runs).