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

ci: Release binaries via CRT #1154

Merged
merged 19 commits into from
Feb 2, 2023
Merged

ci: Release binaries via CRT #1154

merged 19 commits into from
Feb 2, 2023

Conversation

dbanck
Copy link
Member

@dbanck dbanck commented Jan 25, 2023

This is a result of following the CRT onboarding guide.

TODO

@dbanck dbanck added the ci Continuous integration/delivery related label Jan 25, 2023
Copy link
Member

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

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

I understand this is still a draft - so not all comments may be relevant as you're still working on this and you may have already noticed the same thing, but I hope the early review is still helpful. 🤷🏻‍♂️ 😃

@dbanck dbanck force-pushed the onboard-crt branch 5 times, most recently from 98158b0 to 4c9621a Compare January 30, 2023 11:55
@dbanck dbanck changed the title Onboard CRT ci: Release binaries via CRT Jan 30, 2023
@dbanck dbanck requested a review from radeksimko January 30, 2023 11:56
Copy link
Member

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

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

LGTM for the most part, I just left some minor in-line comments/questions. Thanks for all your work on this!

@radeksimko
Copy link
Member

Is it worth removing the dist directory from gitignore now, that we won't build via GoReleaser there?

@dbanck dbanck marked this pull request as ready for review February 1, 2023 10:18
@dbanck dbanck requested a review from a team as a code owner February 1, 2023 10:18
@dbanck
Copy link
Member Author

dbanck commented Feb 1, 2023

I did the following checks to verify the (most) artifacts:

  • terraform-ls_0.31.0-dev_darwin_amd64.zip — ✅ works and has provider schemas
  • terraform-ls_0.31.0-dev_windows_amd64.zip — ✅ works and has provider schemas
  • terraform-ls_0.31.0-dev_linux_amd64.zip — ✅ works and has provider schemas
  • Staged artifacts match published on releases.hashicorp.com/terraform-ls ✅
  • File size of rpm and debian packages matches the binary ✅

Copy link
Member

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

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

LGTM. 🚢

Some random spot checks confirm the artifacts (as available in Artifactory currently) were built as expected:

gpg --verify terraform-ls_0.31.0-dev_SHA256SUMS.72D7468F.sig terraform-ls_0.31.0-dev_SHA256SUMS
gpg: Signature made Tue 31 Jan 16:07:24 2023 GMT
gpg:                using RSA key 374EC75B485913604A831CC7C820C6D5CD27AB87
gpg: Good signature from "HashiCorp Security (hashicorp.com/security) <[email protected]>" [ultimate]
shasum -a256 -c terraform-ls_0.31.0-dev_SHA256SUMS
terraform-ls_0.31.0-dev_darwin_amd64.zip: OK
terraform-ls_0.31.0-dev_darwin_arm64.zip: OK
terraform-ls_0.31.0-dev_freebsd_386.zip: OK
terraform-ls_0.31.0-dev_freebsd_amd64.zip: OK
terraform-ls_0.31.0-dev_freebsd_arm.zip: OK
terraform-ls_0.31.0-dev_linux_386.zip: OK
terraform-ls_0.31.0-dev_linux_amd64.zip: OK
terraform-ls_0.31.0-dev_linux_arm.zip: OK
terraform-ls_0.31.0-dev_linux_arm64.zip: OK
terraform-ls_0.31.0-dev_openbsd_386.zip: OK
terraform-ls_0.31.0-dev_openbsd_amd64.zip: OK
terraform-ls_0.31.0-dev_solaris_amd64.zip: OK
terraform-ls_0.31.0-dev_windows_386.zip: OK
terraform-ls_0.31.0-dev_windows_amd64.zip: OK
terraform-ls_0.31.0-dev_windows_arm64.zip: OK
codesign -dv --verbose=4 ./terraform-ls
Executable=/Users/radeksimko/Downloads/terraform-ls
Identifier=payload
Format=Mach-O thin (x86_64)
CodeDirectory v=20500 size=163539 flags=0x10000(runtime) hashes=5105+2 location=embedded
VersionPlatform=1
VersionMin=657664
VersionSDK=657664
Hash type=sha256 size=32
CandidateCDHash sha1=c91449e674478803a7dbb843ebb9157c77da769e
CandidateCDHashFull sha1=c91449e674478803a7dbb843ebb9157c77da769e
CandidateCDHash sha256=b8f70d10093725bb3c22e77f94c19f41e0714863
CandidateCDHashFull sha256=b8f70d10093725bb3c22e77f94c19f41e07148638c3b365e4439d3b3db9c3623
Hash choices=sha1,sha256
CMSDigest=12b41989d9fbb031684044c8847f832e48c3e5fad2bfc8beae62306d37ff2bc2
CMSDigestType=2
Executable Segment base=0
Executable Segment limit=19476480
Executable Segment flags=0x1
Page size=4096
CDHash=b8f70d10093725bb3c22e77f94c19f41e0714863
Signature size=9127
Authority=Developer ID Application: Hashicorp, Inc. (D38WU7D763)
Authority=Developer ID Certification Authority
Authority=Apple Root CA
Timestamp=31 Jan 2023 at 15:50:13
Info.plist=not bound
TeamIdentifier=D38WU7D763
Runtime Version=10.9.0
Sealed Resources=none
Internal requirements count=1 size=168

@dbanck dbanck merged commit 1775b8a into main Feb 2, 2023
@dbanck dbanck deleted the onboard-crt branch February 2, 2023 10:32
@github-actions
Copy link

github-actions bot commented Mar 5, 2023

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ci Continuous integration/delivery related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants