search

LLNL / maestrowf

A tool to easily orchestrate general computational workflows both locally and on supercomputers
https://maestrowf.readthedocs.io
MIT License
134 stars 43 forks source link

Introduces additional states for handling new versions of Flux #407

Closed FrankD412 closed 1 year ago

FrankD412 commented 1 year ago

This MR introduces state changes that allow Maestro to work with newer versions of Flux that have new state abbreviations for job status. The change is pulled from @Jmast's comment here.

FrankD412 commented 1 year ago

@jwhite -- might it be time to drop some older support for Flux versions and settle on the latest going forward?

jwhite242 commented 1 year ago

@jwhite -- might it be time to drop some older support for Flux versions and settle on the latest going forward?

we should definitely make the newest one the default. should we mark a few of the old ones deprecated first? -> i.e. drop 0.12 and 0.17 in a future release (dunno if we need to wait for 2.0 for this?)

FrankD412 commented 1 year ago

Yeah -- I'm starting to think that might be worthwhile as Flux becomes more mature.