Skip to content

Commit

Permalink
Add builderDependencies
Browse files Browse the repository at this point in the history
Signed-off-by: Mark Lodato <lodato@google.com>
  • Loading branch information
MarkLodato committed Nov 7, 2022
1 parent b661ee1 commit f87271e
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 2 deletions.
3 changes: 2 additions & 1 deletion docs/provenance/v1.0.cue
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,8 @@
"runDetails": {
"builder": {
"id": string,
"version": string
"version": string,
"builderDependencies": [...#ArtifactReference]
},
"metadata": {
"invocationId": string,
Expand Down
3 changes: 2 additions & 1 deletion docs/provenance/v1.0.md
Original file line number Diff line number Diff line change
Expand Up @@ -280,6 +280,7 @@ meaning of each field is unchanged unless otherwise noted.
"builder": {
"id": old.builder.id,
"version": null // not in v0.2
"builderDependencies": null // not in v0.2
},
"metadata": {
"invocationId": old.metadata.buildInvocationId,
Expand Down Expand Up @@ -319,7 +320,7 @@ changed slightly.
input artifacts to be specified.
- Moved `entryPoint` out of `buildConfig` so that all artifact references
share a common schema.
- Added `builder.version`.
- Added `builder.version` and `builder.builderDependencies`.
- Added `byproducts`.
- Removed `buildConfig`; can use `inputArtifacts.config` or `byproducts`
instead.
Expand Down
6 changes: 6 additions & 0 deletions docs/provenance/v1.0.proto
Original file line number Diff line number Diff line change
Expand Up @@ -218,6 +218,12 @@ message Builder {

// TODO: Do we want to add this field? (#319)
string version = 2;

// Dependencies used by the orchestrator that are not run within the workload
// and that should not affect the build, but may affect the provenance
// generation or security guarantees.
// TODO: Flesh out this model more.
repeated ArtifactReference builderDependencies = 3;
}

message BuildMetadata {
Expand Down

0 comments on commit f87271e

Please sign in to comment.