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

Update release signing configuration #35

Merged
merged 1 commit into from
Mar 1, 2022
Merged

Conversation

nywilken
Copy link
Contributor

This PR updates the release signing to use the internal HashiCorp signing service (signore).

Update secrets before merging:

Added SIGNORE_CLIENT_ID
Added SIGNORE_CLIENT_SECRET
Added SIGNORE_SIGNER

@nywilken nywilken requested a review from a team as a code owner February 25, 2022 19:10
- cmd: signore
args: ["sign", "--dearmor", "--file", "${artifact}", "--out", "${signature}"]
artifacts: checksum
signature: ${artifact}.sig
Copy link
Contributor

Choose a reason for hiding this comment

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

Ehm, I wonder if packer init will work with sig files🤔.

We would need to add a checksummer type here:

https://github.com/hashicorp/packer/blob/a766d59d82c129a40f8612e648cde2d051a91fba/command/init.go#L73-L75

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Goreleaser will generate the SHA256SUM checksum file. The signore project will sign the generated SHA256SUM file and create the .sig file, which is binary. The created binary file is the same as if we were using the gpg command except signore is a hosted service for signing.

Looking at the checksum code for Packer init, I see it reads the file and then parses the checksum against the hash for the plugin. If we wanted to check the sig file we would want to include a gpg verify process into the init step. This is not needed at this time. In the future it might be worth looking into.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, I see, super cool, thanks for the explanation ! LGTM then !!

@nywilken nywilken merged commit 360dac6 into main Mar 1, 2022
@nywilken nywilken deleted the migrate-to-signore branch March 1, 2022 18:49
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.

2 participants