gtristan commented on PR #2035:
URL: https://github.com/apache/buildstream/pull/2035#issuecomment-3082844646
Suggestion: I think we should have a whole `”artifact”: { … }` block which
contains only things which are loaded from the artifact, and omit it entirely
in the case that the artifact is not present.
This can include:
* the public data as found in the artifact, which may not be the same as the
public data loaded from the bst files before building
* The cas digest(s)
* The file metadata
* The build logs
* The failure state (whether it is a cached failed build)
* whether the artifact has a cached build tree
I think this disambiguates things a bit.
Also… I think that given we present a lot of data in the output json, I
think maybe we should not do the `%{state}` thing at all, and leave that only
to `bst show`. Rationale being that the “state” concept is really very user
facing, and it can anyway be easily computed based on the json reported by `bst
inspect`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
