Skip to content

Conversation

@coalkettler
Copy link
Contributor

@coalkettler coalkettler commented Jun 5, 2020

Overview

This retrieves an up-to-date copy of the Geotrellis public GPG key during the cipublish build stage.

We've pushed a new signature to extend the expiration date on the public key that we've uploaded to public keyservers. The CircleCI build will now retrieve the latest copy of the public key, allowing us to renew the key in the future using the same procedure.

Checklist

  • Description of PR is in an appropriate section of the CHANGELOG and grouped with similar changes if possible

Testing Instructions

  • Verify that updated and valid Geotrellis GPG key is retrieved and verified during cipublish.
    • Verified by @rbreslow during pairing session.

Connects azavea/operations#446

@coalkettler coalkettler self-assigned this Jun 5, 2020
Comment on lines -29 to -30
gpg --batch \
--passphrase "${GPG_PASSPHRASE}" \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because we're not using passphrase for this key, the --batch and --passphrase options are unnecessary.

@coalkettler coalkettler marked this pull request as ready for review June 5, 2020 15:51
@coalkettler coalkettler requested a review from rbreslow June 5, 2020 15:59
Copy link
Contributor

@rbreslow rbreslow left a comment

Choose a reason for hiding this comment

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

👍

@coalkettler coalkettler merged commit 84553e7 into develop Jun 5, 2020
@coalkettler coalkettler deleted the feature/cek/receive-gpg-key branch June 5, 2020 16:07
- run:
name: "Import signing key"
command: |
gpg --keyserver keyserver.ubuntu.com \
Copy link
Contributor

Choose a reason for hiding this comment

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

It's worth surfacing that we're specifying the Ubuntu keyserver here to mirror the approach taken by @jodersky. Through testing, I've found that this keyserver was updated more quickly than the default.

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.

3 participants