Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add --digest-file flag to output built digest to file. #655

Merged
merged 3 commits into from
May 14, 2019

Conversation

GijsWeterings
Copy link
Contributor

@GijsWeterings GijsWeterings commented May 2, 2019

This flag, when set, takes a file in the container and writes the image digest to it. This can be used to extract the exact digest of the built image by surrounding tooling without having to parse the logs from Kaniko, for example by pointing the file to a mounted volume or to a file used durint exit status, such as with Kubernetes' Termination message policy

When the flag is not set, the digest is not written to file and the executor behaves as before. The digest is also written to file in case of a tarball or a --no-push.

Closes #654

This flag, when set, takes a file in the container and writes the image digest to it. This can be used to extract the exact digest of the built image by surrounding tooling without having to parse the logs from Kaniko, for example by pointing the file to a mounted volume or to a file used durint exit status, such as with Kubernetes' [Termination message policy](https://kubernetes.io/docs/tasks/debug-application-cluster/determine-reason-pod-failure/)]

When the flag is not set, the digest is not written to file and the executor behaves as before. The digest is also written to file in case of a tarball or a `--no-push`.

Closes GoogleContainerTools#654
@GijsWeterings
Copy link
Contributor Author

@priyawadhwa WDYT? Is this something you'd like to include?

@dlorenc
Copy link
Collaborator

dlorenc commented May 10, 2019

Looks like a good idea!

Copy link
Collaborator

@priyawadhwa priyawadhwa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just left a couple comments!

cmd/executor/cmd/root.go Outdated Show resolved Hide resolved
pkg/executor/push.go Outdated Show resolved Hide resolved
Changes the argument flag from `--digestfile` to `--digest-file`.
Skips an unneeded nil check.
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@GijsWeterings GijsWeterings changed the title Add --digestfile flag to output built digest to file. Add --digest-file flag to output built digest to file. May 14, 2019
@priyawadhwa priyawadhwa merged commit 38c1735 into GoogleContainerTools:master May 14, 2019
@priyawadhwa
Copy link
Collaborator

Thanks for adding this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ability to fetch SHA of pushed image
4 participants