Skip to content

Conversation

@mtrmac
Copy link
Collaborator

@mtrmac mtrmac commented Aug 17, 2024

Add rekorPublicKeyPaths and rekorPublicKeyDatas options to policy.json.

This is on top of unmerged #2524.

Note an outstanding design question on …Datas: #2524 (comment) .

Currently not tested end-to-end in practice.

@mtrmac mtrmac force-pushed the sigstore-multi-rekor branch from 08f0bdf to 5f51115 Compare August 17, 2024 03:39
@mtrmac mtrmac added the kind/feature A request for, or a PR adding, new functionality label Aug 17, 2024
@mtrmac
Copy link
Collaborator Author

mtrmac commented Aug 19, 2024

Now tested by manually writing policy.json, ready for review (but #2524 needs to merge first).

@TomSweeneyRedHat
Copy link
Member

LGTM
@rhatdan and @mheon PTAL

@rhatdan
Copy link
Member

rhatdan commented Aug 19, 2024

@mtrmac still draft?

@mtrmac
Copy link
Collaborator Author

mtrmac commented Aug 19, 2024

Ready for review, but it can be merged only after #2524 . (If reviewing, compare this with the contents from #2524 , or since the included merge commit.)

@mtrmac mtrmac force-pushed the sigstore-multi-rekor branch from 5f51115 to 5d5cc3f Compare August 20, 2024 12:11
Add rekorPublicKeyPaths and rekorPublicKeyDatas , similar to the primary
root of trust public keys.

Signed-off-by: Miloslav Trmač <[email protected]>
@mtrmac mtrmac force-pushed the sigstore-multi-rekor branch from 5d5cc3f to 7bdb48b Compare August 20, 2024 15:02
@mtrmac
Copy link
Collaborator Author

mtrmac commented Aug 20, 2024

Rebased, ready for review/merging.

@mtrmac mtrmac marked this pull request as ready for review August 20, 2024 15:02
@rhatdan
Copy link
Member

rhatdan commented Aug 20, 2024

LGTM

Copy link
Member

@TomSweeneyRedHat TomSweeneyRedHat left a comment

Choose a reason for hiding this comment

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

LGTM

@TomSweeneyRedHat
Copy link
Member

/lgtm

@TomSweeneyRedHat TomSweeneyRedHat merged commit 6692ffc into containers:main Aug 20, 2024
@mtrmac mtrmac deleted the sigstore-multi-rekor branch August 20, 2024 15:44
@mtrmac mtrmac mentioned this pull request Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature A request for, or a PR adding, new functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants