Skip to content

Conversation

tvoran
Copy link
Member

@tvoran tvoran commented Mar 22, 2022

Release version is specified when manually triggering the build workflow. Go version is determined using the .go-version file. Will build for arm64 as well as amd64.

TODO:

  • Run CI tests in a GitHub workflow
  • Setup CRT for binary releases
  • [ ] Setup AWS layer releases in a GitHub workflow (this will be done in another repo)

tvoran added 7 commits March 22, 2022 14:36
Release version is specified when manually triggering the build
workflow. Go version is determined using the .go-version file. Will
build for arm, arm64, 386, and amd64 now instead of just amd64.
Copy link
Contributor

@tomhjp tomhjp left a comment

Choose a reason for hiding this comment

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

Looking good. FWIW I would support leaving the final layer publish step in CircleCI for now. Rel-eng have a ticket to track adding support for publishing AWS Lambda layers, so switching directly to that in the future might be an easier option for us.

.release/ci.hcl Outdated
github {
organization = "hashicorp"
repository = "vault-lambda-extension"
release_branches = ["main", "VAULT-5260/use-crt"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Leaving a comment as a reminder we should remove the test branch before merging.

Copy link
Contributor

@tomhjp tomhjp left a comment

Choose a reason for hiding this comment

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

Looks like this is ready to go if testing goes smoothly 👍

@tvoran
Copy link
Member Author

tvoran commented Apr 8, 2022

Tested both arm64 and amd64 binaries, and they seem to work, so I think we're good here 👍

@tvoran tvoran merged commit 14a3897 into main Apr 8, 2022
@tvoran tvoran deleted the VAULT-5260/use-crt branch April 8, 2022 04:07
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