rfc27: drop sched.queue as scheduler annotation - #547
Conversation
Problem: RFC 27 lists `sched.queue` as a known scheduler annotation, but this annotation is not used anywhere since schedulers do not assign queues. Drop the annotation to avoid confusion.
|
|
|
I was thinking about this the other day and here's one reason to consider keeping it:
edit: if that data is already collected elsewhere, I'm happy to approve this PR. @milroy agreed with your assessment, btw, that this annotation was probably a relic from early Fluxion. |
I don't think it is, but if that was added, it probably wouldn't be under the |
|
A simple test on tuolumne and a |
|
Queued — the merge queue status continues in this comment ↓. |
|
Thanks! Setting MWP. |
Merge Queue Status
This pull request spent 15 seconds in the queue, including 2 seconds running CI. Required conditions to merge
|
Problem: RFC 27 lists
sched.queueas a known scheduler annotation, but this annotation is not used anywhere since schedulers do not assign queues.Drop the annotation to avoid confusion.
For background, Fluxion did set a
sched.queueannotation early on when it had a default queue config parameter, but this was removed in flux-framework/flux-sched#971.