cylc lint mistakenly picks up PBS -W directive as LSF time limit directive

[MetRonnie]

Description

The -W option allows for the specification of additional job attributes.

https://www.jlab.org/hpc/PBS/qsub.html

Reproducible Example

[runtime]
  [[foo]]
    platform = pbs_platform
    [[[directives]]]
      -W umask = 0022

Expected Behaviour

cylc lint S014 should look for -W = for LSF rather than just -W

[wxtim]

I suspect that this would require us to save more state to allow us to differentiate between PBS and LSF jobs, in a similar way to #6348.

The quicker solution is to artificially remove -W from the list of wallclock directives to check for.

Do we want to do this?

Are any of the others problematic in a similar way?

{
  "loadleveler": "wall_clock_limit",
  "lsf": "-W",
  "moab": "-l walltime",
  "pbs": "-l walltime",
  "pbs_multi_cluster": "-l walltime",
  "sge": "-l h_rt",
  "slurm": "--time",
  "slurm_packjob": "--time"
}

[MetRonnie]

cylc lint S014 should look for -W = for LSF rather than just -W

Is this not feasible? It doesn't need any more state than the current line

[wxtim]

cylc lint S014 should look for -W = for LSF rather than just -W

Is this not feasible? It doesn't need any more state than the current line

I'm not sure what you mean - that I can narrow the scope of the regex to only include valid values for LSF?

[MetRonnie]

The difference between LSF's -W and PBS's -W:

• LSF: -W = 10
• PBS: -W something = 10

[wxtim]

That's what I worked out. Working on the fix.