Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make flux resource drain -o long reason expandable #6338

Merged
merged 1 commit into from
Oct 2, 2024

Commits on Oct 2, 2024

  1. make flux resource drain -o long reason expandable

    Problem: the "long" format of 'flux resource drain' doesn't show the
    un-truncated drain reason, but one might expect that to be the case.
    
    Now that we have expandable field width support, use it for {reason}
    in the "long" format.  In addition, to conserve space, {timestamp} is
    changed to use same format as "default", since the longer representation
    wasn't particularly helpful.
    
    $ flux resource drain
    TIME         STATE    REASON                         NODELIST
    Oct02 13:06  drained  This is a long drain message + pop-os
    $ flux resource drain -o long
    TIME         STATE    RANKS    REASON                                NODELIST
    Oct02 13:06  drained  0        This is a long drain message isnt it? pop-os
    
    Same for {reason} in the "long" format of 'flux resource status':
    
    $ flux resource status -o long
           STATE UP NNODES REASON                                NODELIST
         drained  ✔      1 This is a long drain message isnt it? pop-os
    garlick committed Oct 2, 2024
    Configuration menu
    Copy the full SHA
    e5f695e View commit details
    Browse the repository at this point in the history