workflow jobID should be a string and show up in 'kubectl get workflows' output

NearNodeFlash / NearNodeFlash.github.io

View this document https://nearnodeflash.github.io/

Apache License 2.0

3 stars 3 forks source link

workflow jobID should be a string and show up in 'kubectl get workflows' output #65

Closed ajfloeder closed 1 year ago

ajfloeder commented 1 year ago

Flux's jobIDs are not numeric, so jobID or its replacement should accept a string.

Also, the jobID should be displayed as part of the kubectl get workflows default output.

roehrich-hpe commented 1 year ago

The "wide" version of the output already prints the job ID.

$ kubectl get workflow -o wide
NAME    STATE      READY   STATUS      AGE   JOBID    DESIREDSTATE   DESIREDSTATECHANGE   ELAPSEDTIMELASTSTATE
test2   Proposal   true    Completed   5s    900002   Proposal       5s                   35.923ms

behlendorf commented 1 year ago

I can see it being convenient to include the job ID in the default output rather than the "wide" version.

roehrich-hpe commented 1 year ago

Done: https://github.com/HewlettPackard/dws/pull/138 https://github.com/NearNodeFlash/nnf-sos/pull/197