Skip to content

Update LSF.pm to parse year- and year-less responses of LSF bacct cmd

Marek Szuba requested to merge github/fork/tweep/patch-3 into master

Created by: tweep

Our installation of LSF returns the year in the bacct response - which is not parsed by the regex in parse_report_source_line(). The fix will parse year- and year-less responses of bacct. This is a quickfix as the _yearless_2_datetime() method is still invoked - best would be to not invoke it if the year is returned; however I think it increases the complexity of the code too much. Regex now parses both lines:

Sat Jan 23 22:48:28 2016: Completed <exit>; TERM_RUNLIMIT: job killed after reaching LSF run time limit.

Sat Jan 23 22:48:28: Completed <exit>; TERM_RUNLIMIT: job killed after reaching LSF run time limit.

screen shot 2016-01-26 at 4 06 05 pm

We're using LSF version 9.1.2.0 - would be interesting if the different response formats are due to different LSF versions. Maybe it's time to create Bio::EnsEMBL::Hive::Meadow::LSF::SUBVERSION ?

Merge request reports