Skip to content

Commit

Permalink
Amend 0037 with BuildTimestamp
Browse files Browse the repository at this point in the history
We concluded that the build timestamp option was forgotten in the initial design of the SHIP. Add `BuildTimestamp` to define that the image timestamp should be set to the timestamp of the current Build Run.
  • Loading branch information
HeavyWombat authored Dec 11, 2023
1 parent 48dfddc commit d8894b4
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion ships/0037-build-output-timestamp.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ approvers:
- "@adambkaplan"
- "@qu1queee"
creation-date: 2023-04-24
last-updated: 2023-04-24
last-updated: 2023-12-11
status: implementable
---

Expand Down Expand Up @@ -80,6 +80,7 @@ The `timestamp` entry is an optional string value field with the default being `

- `Zero` refers to the Unix epoch start date of January, 1st 1970.
- `SourceTimestamp` would lead to taking the provided source code meta data to define the timestamp. In case the source is a Git repository, the Git commit that is used for the build defines the source timestamp. In any other case, the newest creation time stamp of the source files is used. Note, the timestamp needs to be taken from the user provided source files since the build process might introduce new files as part of the build itself. The source step needs to be extended to introduce a reliable point and single spot in which the source timestamp is being obtained.
- `BuildTimestamp` defines that the image timestamp is to be set to the current actual execution timestamp of the Build Run.
- `<timestamp>` must be a parsable Unix epoch timestamp that lies in the past and will be then used as-is for the creation timestamp of the image, i.e. custom neutral timestamp. An empty string is considered not parsable and therefore results in an error.
- Null (unset) results in no timestamp usage at all, which means that the respective build strategy timestamp is used.

Expand Down

0 comments on commit d8894b4

Please sign in to comment.