Runner: Stop inlining the contents of stdout/err in `ActionResult`s
As discussed in #37 (closed), inlining the contents of a command's stdout
/stderr
in the ActionResult
is something that depends on the request issued by a client in its GetActionResultRequest
.
Currently the Runner
is inlining those contents whenever they fit inside the message.
-
Change the Runner::executeAndStore()
method to always upload the contents written by the command to stdout and stderr and setActionResult.stdout_digest
andActionResult.stderr_digest
accordingly. -
Stop populating the stdout_raw
andstderr_raw
fields.
Acceptance criteria
Runners upload the contents of stdout and stderr and set the corresponding digest of the data in the ActionResult
(when no data is output, the digest of the empty string is set).
Edited by Santiago Gil