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

goreleaser: adjust goreleaser to dynamically link rpk on linux #17230

Merged
merged 5 commits into from
Mar 29, 2024

Conversation

andrewhsu
Copy link
Member

@andrewhsu andrewhsu commented Mar 21, 2024

fixes https://redpandadata.atlassian.net/browse/PESDLC-1005

add goreleaser build id rpk-linux-microsoft-go for use with microsoft's golang compiler. this configures the compiler to dynamically link rpk for deb/rpm packages.

the zip files of rpk generated by goreleaser that are attached as assets to redpanda github releases are left as statically compiled.

this PR needs to be merged in sync with PR https://github.com/redpanda-data/vtools/pull/2588
EDIT: i added git commit e6e39e7 so existing vtools can work without changes

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.3.x
  • v23.2.x

Release Notes

  • none

@andrewhsu andrewhsu changed the title goreleaser: dynamically compile rpk goreleaser: dynamically compile rpk on linux Mar 21, 2024
@andrewhsu andrewhsu force-pushed the PESDLC-1005-dyn-compile-rpk branch 2 times, most recently from 1d1f529 to 66e3b6b Compare March 21, 2024 01:12
@andrewhsu
Copy link
Member Author

/dt

@andrewhsu andrewhsu force-pushed the PESDLC-1005-dyn-compile-rpk branch 4 times, most recently from dcd7a4c to 8b8ece5 Compare March 26, 2024 18:30
@andrewhsu andrewhsu changed the title goreleaser: dynamically compile rpk on linux goreleaser: adjust goreleaser to dynamically link rpk on linux Mar 28, 2024
@andrewhsu andrewhsu marked this pull request as ready for review March 28, 2024 13:52
@andrewhsu andrewhsu requested a review from ivotron March 28, 2024 13:52
@andrewhsu
Copy link
Member Author

this is ready for review

Copy link
Contributor

@twmb twmb left a comment

Choose a reason for hiding this comment

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

I think this looks correct but this should have another eye

@twmb
Copy link
Contributor

twmb commented Mar 28, 2024

Wait does this mean an RP release will now have two versions of rpk in one zip file?

@andrewhsu
Copy link
Member Author

Wait does this mean an RP release will now have two versions of rpk in one zip file?

@twmb there will still be 1 zip file per rpk binary, e.g. rpk-linux-amd64.zip. the goarch section of each id under builds can be looked at as "match patterns".

ivotron
ivotron previously approved these changes Mar 29, 2024
@andrewhsu
Copy link
Member Author

andrewhsu commented Mar 29, 2024

i added git commit f2772f9 e6e39e7 as a result of comment on linked PR https://github.com/redpanda-data/vtools/pull/2588#issuecomment-2027209719

so this one can be merged safely by itself. ready for re-review.

cc @ivotron

Removal of rpk-windows-and-linux will happen in future after vtools
references to it are pointing to alternative ids.
@andrewhsu andrewhsu merged commit 24e7b78 into redpanda-data:dev Mar 29, 2024
12 checks passed
@andrewhsu andrewhsu deleted the PESDLC-1005-dyn-compile-rpk branch March 29, 2024 18:09
@andrewhsu andrewhsu mentioned this pull request Mar 30, 2024
6 tasks
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.

None yet

3 participants