-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
KEP-3031: Add release artifact signing KEP #3061
KEP-3031: Add release artifact signing KEP #3061
Conversation
saschagrunert
commented
Nov 29, 2021
- One-line PR description: adding initial draft of the artifact signing enhancement
- Issue link: Signing Release Artifacts #3031
65d63c4
to
d661be0
Compare
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.
Nice to see this in, thank you @saschagrunert ! Some notes to consider:
c14cad0
to
20b944a
Compare
20b944a
to
ece96d2
Compare
/assign @justaugustus @cpanato |
Signed-off-by: Sascha Grunert <[email protected]>
ece96d2
to
2099733
Compare
pre-evaluation of the tool has been done already to ensure that it meets the | ||
requirements. | ||
|
||
An [ongoing discussion](https://github.com/kubernetes/release/issues/2227) about |
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.
Would be a great pioneering use case for sigstore trust delegations!
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.
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.
@trishankatdatadog Unfortunately, I don't have access to this document. Is this anything we have to consider for this KEP?
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.
No, I don't think so, I think that's for another KEP that details the specifics of signing and verification. It can be done with cosign in any case, which you have already mentioned 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.
No, I don't think so, I think the details can go into the specific KEP that details signing/verification. Thanks!
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.
Great work here, @saschagrunert!
/lgtm
/approve
Hold to removed at EOD.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: justaugustus, saschagrunert The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Starting the first implementation in kubernetes/release#2383 |