-
Notifications
You must be signed in to change notification settings - Fork 14
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: add release image signing guidance #129
base: main
Are you sure you want to change the base?
docs: add release image signing guidance #129
Conversation
✅ Deploy Preview for ratify-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -41,11 +44,11 @@ cat <<EOF > ./trustpolicy.json | |||
} | |||
EOF | |||
notation policy import ./trustpolicy.json | |||
notation verify ghcr.io/ratify-project/ratify:v1.4.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious about that, since v1.4.0
is not released shall we go with the latest version v1.3.1
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was thinking that updates to the docs here are for the next
version on the website, which corresponds to unreleased changes. The current v1.3.1 release is not signed so we don't want to give that as an example. When v1.4.0 is released, the published release images will be signed and match the docs.
@@ -63,18 +66,18 @@ cosign verify \ | |||
--certificate-identity "https://github.com/ratify-project/ratify/.github/workflows/publish-dev-assets.yml@refs/heads/dev" \ | |||
--certificate-oidc-issuer https://token.actions.githubusercontent.com \ | |||
--certificate-github-workflow-repository ratify-project/ratify \ | |||
ghcr.io/ratify-project/ratify-dev:latest | |||
ghcr.io/ratify-project/ratify:v1.4.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get answered above.
REF: #129 (comment)
Updates security doc to include release image verification guidance