-
Notifications
You must be signed in to change notification settings - Fork 120
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 GitHub CD release build workflow #661
Conversation
I still need to give this another once over. But it's probably correct. EDIT: OK, I've done that, still seems ok. I'll now request reviews. |
029c3a4
to
bd467c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm, cross-checked with what we use in lnd and seems to make sense
guess we just have to test it out live now 💯
bd467c3
to
12f23cf
Compare
I've tried to recreate the manifest file for release v0.3.1 but the hashes did not match. My steps were:
The hashes I got were:
|
I also did a build locally after installing go And set that as the default go toolchain. The manifest for that local build matches my docker build and the manifest posted above. |
12f23cf
to
ba5a7a5
Compare
The 0.3.1 release manifest hashes don't seem to matchup on individual binaries. For example, locally I generated:
Where as the manifest file here on the release page I see:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Solid updates!
I tested verify-install.sh
for v0.3.0 and it correctly picked up the 3 posted sigs + all keys.
Final nit would be to rename the new scripts + targets for go versioning check, since we aren't actually linting the Dockerfiles or GH workflow files, just version checking. Non-blocking.
We should resolve reproducible builds for v0.3.1 before merging this though. |
We can now remove these comments now that our linter performs Go version consistency checks across Dockerfile files and YAML files.
Adding roasbeef's public key so that a hyperlink in our release verification example is valid.
ba5a7a5
to
44c83d7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Tag I spoke to @Roasbeef and he said that his staging environment where he built the release assets was at commit So I think |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My local build hashes for v0.3.1
match those reported here by @ffranr #661 (comment)
This new release build workflow is triggered when a new git tag is pushed.
Closes #307