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

docs(readme): add installation steps for container image for cosign binary #1986

Merged
merged 2 commits into from
Jun 10, 2022

Conversation

developer-guy
Copy link
Member

Signed-off-by: Batuhan Apaydın batuhan.apaydin@trendyol.com

Summary

Adding installation steps for container image for cosign binary

Ticket Link

Fixes #1985

Release Note

docs(readme): add installation steps for container image for cosign binary

…inary

Signed-off-by: Batuhan Apaydın <batuhan.apaydin@trendyol.com>
@developer-guy developer-guy self-assigned this Jun 10, 2022
dlorenc
dlorenc previously approved these changes Jun 10, 2022
README.md Outdated
```shell
FROM gcr.io/projectsigstore/cosign:v1.9.0 as cosign-bin

FROM scratch
Copy link
Member

Choose a reason for hiding this comment

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

This will work, but then will the image actually be usable? You'd probably need ca-certs as well to actually talk to any registries at a minimum. Maybe we should use distroless in the example?

Copy link
Member Author

Choose a reason for hiding this comment

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

That makes a lot of sense, let me update it before merge, thanks 🙏

Copy link
Member Author

Choose a reason for hiding this comment

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

I think we are good with: acd342e

Signed-off-by: Batuhan Apaydın <batuhan.apaydin@trendyol.com>
@codecov-commenter
Copy link

codecov-commenter commented Jun 10, 2022

Codecov Report

Merging #1986 (acd342e) into main (68e8d93) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1986   +/-   ##
=======================================
  Coverage   28.71%   28.71%           
=======================================
  Files         132      132           
  Lines        8087     8087           
=======================================
  Hits         2322     2322           
  Misses       5458     5458           
  Partials      307      307           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 68e8d93...acd342e. Read the comment docs.

@dlorenc dlorenc merged commit 67b1fd1 into sigstore:main Jun 10, 2022
@github-actions github-actions bot added this to the v1.10.0 milestone Jun 10, 2022
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.

add additional installation section for cosign
3 participants