ondemand icon indicating copy to clipboard operation
ondemand copied to clipboard

ActiveJobs: ActiveJobs: Use Info#wallclock_limit for wallclock time

Open brianmcmichael opened this issue 8 years ago • 3 comments

Currently the wallclock times are derived via accessing native attributes, like:

attributes.push Attribute.new "Walltime", (info.native.fetch(:Resource_List, {})[:walltime].presence || "00:00:00")

These can be replaced with something like:

attributes.push Attribute.new "Walltime", info.wallclock_limit || "00:00:00"

After updating to the the latest ood_core release.

┆Issue is synchronized with this Asana task by Unito

brianmcmichael avatar Jun 28 '17 19:06 brianmcmichael

I think that the adapters all implement this method. Verify that by looking at the ood_core code. If it looks legit, do this feature.

ericfranz avatar Aug 02 '18 22:08 ericfranz

This can move forward for torque slurm and pbspro. The only other adapter at present that has extended view (LSF) does not yet have this.

johrstrom avatar Mar 09 '20 21:03 johrstrom

Don't know why I thought lsf doesn't have support for this - it did and still does:

https://github.com/OSC/ood_core/blob/4b21ae7bfb9335e248b547296cbac21137fc818f/lib/ood_core/job/adapters/lsf.rb#L205

So in any case, for any implementer of this good first issue -

The line in question that needs updated is here (for torque) in the extended_data_torque method. There will be similar methods for the other schedulers as well. It seems we'd need to support these 4 schedulers: torque, slurm, pbspro and lsf.

https://github.com/OSC/ondemand/blob/43aa06d89689807ed0cae7cf4a85cfb97426bfa6/apps/dashboard/app/models/active_jobs/jobstatusdata.rb#L75

johrstrom avatar May 02 '24 20:05 johrstrom