Bug #16856

Pass through --disable-color

Added by Peter Amstutz about 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
CWL
Target version:
Start date:
09/21/2020
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Subtasks

Task #16863: Review 16856-acr-disable-colorResolvedPeter Amstutz


Related issues

Related to Arvados - Bug #16857: Bad error reporting of missing container imageResolved

Associated revisions

Revision 8d80af89
Added by Peter Amstutz about 1 year ago

Merge branch '16856-acr-disable-color' closes #16856

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <>

History

#1 Updated by Peter Amstutz about 1 year ago

  • Assigned To set to Peter Amstutz

#2 Updated by Peter Amstutz about 1 year ago

  • Status changed from New to In Progress

#3 Updated by Peter Amstutz about 1 year ago

16856-acr-disable-color @ arvados|68eb70370ccf2ec7879146f3980e05019d2b67a1

A few quality-of-life improvements to arvados-cwl-runner:

  • Explicitly disable color logging for submitted containers, since workbench doesn't understand color codes.
  • Move a message "Using empty collection" to debug level since it is entirely meaningless to regular users.
  • Explicitly log the cluster a-c-r is submitting to.
  • Tweak logging of container submit errors so they are displayed better in workbench.

https://ci.arvados.org/view/Developer/job/developer-run-tests/2108/

#4 Updated by Peter Amstutz about 1 year ago

  • Related to Bug #16857: Bad error reporting of missing container image added

#5 Updated by Lucas Di Pentima about 1 year ago

This LGTM, thanks!

#6 Updated by Peter Amstutz about 1 year ago

  • Status changed from In Progress to Resolved

#7 Updated by Peter Amstutz about 1 year ago

  • Release set to 25

Also available in: Atom PDF