Update LSF.pm to parse year- and year-less responses of LSF bacct cmd (!14) · Merge requests · ensembl-gh-mirror / ensembl-hive

Marek Szuba requested to merge github/fork/tweep/patch-3 into master Jan 27, 2016

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.

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 ?

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

Merge request reports