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

feat(secret): add create notation secret handler #4735

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

JasonTheDeveloper
Copy link
Contributor

This PR is in relation to #4692.

The purpose of this PR is to aid in the creation of notation configuration secret used to validate artifacts signed by notation. The create secret notation will only add root CA certificates ending in .crt and .pem and valid trust-policy.json. You can pass in multiple root CA certificates as well as a folder containing multiple certs.

cmd/flux/create_secret_notation.go Outdated Show resolved Hide resolved
cmd/flux/create_secret_notation.go Outdated Show resolved Hide resolved
cmd/flux/create_secret_notation.go Outdated Show resolved Hide resolved
cmd/flux/create_secret_notation.go Outdated Show resolved Hide resolved
cmd/flux/create_secret_notation.go Outdated Show resolved Hide resolved
cmd/flux/create_secret_notation.go Outdated Show resolved Hide resolved
cmd/flux/create_secret_notation.go Outdated Show resolved Hide resolved
@souleb
Copy link
Member

souleb commented Apr 22, 2024

can you please squash your commits?

@stefanprodan
Copy link
Member

@JasonTheDeveloper can you please squash and rebase with main please.

@JasonTheDeveloper JasonTheDeveloper force-pushed the feat/4692 branch 4 times, most recently from 997eb0d to 9a6f519 Compare April 23, 2024 12:27
Signed-off-by: Jason <jagoodse@microsoft.com>
@JasonTheDeveloper
Copy link
Contributor Author

@stefanprodan @souleb all commits have been squashed.

Copy link
Member

@souleb souleb left a comment

Choose a reason for hiding this comment

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

LGTM! thanks @JasonTheDeveloper

@stefanprodan stefanprodan added the area/ux In pursuit of a delightful user experience label Apr 23, 2024
@stefanprodan stefanprodan merged commit acad6ca into fluxcd:main Apr 23, 2024
8 checks passed
@JasonTheDeveloper JasonTheDeveloper deleted the feat/4692 branch April 23, 2024 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ux In pursuit of a delightful user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants