#4075 taskinfo CLI and webUI info message why output is not in the list
Merged by tkopecek. Opened by jcupova.
jcupova/koji issue-4074  into  master

Download 4075.patch

Fixes: https://pagure.io/koji/issue/4074

Seeing this actually displayed, the text seems a bit too noisy, particularly since this is a common case. The repetition in the cli is even more noisy.

Perhaps...

In the web, make the longer explanation a tooltip. E.g.

<div title="Not all tasks produce outputs, and task outputs are not retained indefinitely.">No outputs reported</div>

In the cli, make the longer explanation a (single) footnote. E.g.

Log Files:
  No outputs reported [1]
Output:
  No outputs reported [1]
[1]  Not all tasks produce outputs, and task outputs are not retained indefinitely.

OK, but one more question to CLI. Do you mean footnote once per whole command or when there is more task and subtasks, after every task/subtask where is note No outputs reported [1] will be footnote.

I figured something like the above mockup text. So one footnote at the bottom, with a reference to it each time we print "No outputs" (which would only be once or twice).

rebased onto 1d5774a2b81c7137b7764c519fbd77c6fd228f47

@mikem it's updated

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

The thing that bugs me about this is that we end up adding these notices to tasks that never produced any output and are not expected to. I'm not sure what to do about that without making this a lot more complicated. I guess this is fine for now.

:thumbsup:

Metadata Update from @mfilip:
- Pull-request tagged with: testing-done

Commit 2bf10c70 fixes this pull-request

Pull-Request has been merged by tkopecek

Metadata