Fallback to squeue when sacct is missing in SlurmProvider

[tylern4]

Description

Adds internal check to test whether the slurm provider should use the sacct or squeue command.

Changed Behaviour

Some slurm clusters might not use the accounting database sacct uses. This allows slurm clusters that use the database to use the sacct command which can be easier on the slurm scheduler, or if the database is not present switch to the squeue command which will should work on all clusters.

Fixes

Fixes #3590

Type of change

• Bug fix

[benclifford]

Great!

Can you confirm you tested it with both sacct and squeue?

and @raymondEhlers did you test it?

If so I can avoid any testing myself and merge it for the auto-release on Monday.

[raymondEhlers]

Yes, we will test it and report back, although probably not before Monday (I missed you reply, apologies)

[benclifford]

Yes, we will test it and report back, although probably not before Monday (I missed you reply, apologies)

cool thanks. There's no tight deadline on this.

[fjonasALICE]

Apologies for the late reply and thank you for the fast fix. I tested locally on our cluster and the fix works as expected. Thanks!