Skip to content

Ask squeue for the single purpose reason code - #49

Merged
fedonman merged 1 commit into
mainfrom
fixed/reason-column-uses-lowercase-r
Sep 28, 2026
Merged

fedonman merged 1 commit into
mainfrom
fixed/reason-column-uses-lowercase-r

Conversation

@fedonman

@fedonman fedonman commented Sep 6, 2026

Copy link
Copy Markdown
Owner

The Reason column asked squeue for %R, which prints a reason only for pending and failed jobs and the allocated node list for every other state, so on a running row it just repeated the Node column. It now asks for %r, the single-purpose code, which also means sorting by the column sorts by the reason. squeue writes None for a job with no reason, so that decodes as unset and the table shows the same - it uses for any other blank cell.

Closes #13

@fedonman
fedonman force-pushed the fixed/reason-column-uses-lowercase-r branch from a29e71f to c01a36f Compare September 28, 2026 13:49
The Reason column requested %R, which prints a reason only for pending
and failed jobs and the allocated node list for everything else, so on a
running row it just repeated the Node column. %r is the single purpose
code. squeue writes None for a job with no reason, so that is decoded as
unset and the table shows its usual placeholder.
@fedonman
fedonman merged commit 0adac90 into main Sep 28, 2026
6 checks passed
@fedonman
fedonman deleted the fixed/reason-column-uses-lowercase-r branch September 28, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Reason column shows the nodelist because it requests %R instead of %r

1 participant