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

Add env subcommand. #2051

Merged
merged 1 commit into from
Jul 6, 2022
Merged

Add env subcommand. #2051

merged 1 commit into from
Jul 6, 2022

Conversation

wlynch
Copy link
Member

@wlynch wlynch commented Jul 6, 2022

Summary

This subcommand prints out cosign/sigstore environment variables. This
is intended for debugging to make it easy to find environment variables
that might be impacting cosign behavior.

This was inspired by sigstore/gitsign#92

Signed-off-by: Billy Lynch billy@chainguard.dev

cc @imjasonh

Release Note

Added env subcommand to print out cosign environment variables.

Documentation

Updated CLI docs.

This subcommand prints out cosign/sigstore environment variables. This
is intended for debugging to make it easy to find environment variables
that might be impacting cosign behavior.

Signed-off-by: Billy Lynch <billy@chainguard.dev>
@wlynch
Copy link
Member Author

wlynch commented Jul 6, 2022

@imjasonh implemented as a new subcommand because version is coupled to sigs.k8s.io/release-utils/version. Figured this was a good alternative since there's already prior art with go env.

@imjasonh
Copy link
Member

imjasonh commented Jul 6, 2022

@imjasonh implemented as a new subcommand because version is coupled to sigs.k8s.io/release-utils/version. Figured this was a good alternative since there's already prior art with go env.

This looks good. Should we add gitsign env too, for consistency?

@cpanato cpanato requested a review from bobcallaway July 6, 2022 16:41
@dlorenc dlorenc merged commit ffcc299 into sigstore:main Jul 6, 2022
@github-actions github-actions bot added this to the v1.10.0 milestone Jul 6, 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.

None yet

3 participants