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

chore: handle GOOS alpine in Makefile #3620

Merged
merged 1 commit into from Sep 1, 2022

Conversation

PeterSchafer
Copy link
Contributor

@PeterSchafer PeterSchafer commented Aug 24, 2022

What does this PR do?

Add handling of alpine builds to the Makefile. It introduces the possibility to run

make build GOOS=alpine GOARCH=amd64

How does it work?

While for the golang cli this will actually result in a linux build, the bundled TS cli will have to be build explicitly for alpine.

@PeterSchafer PeterSchafer requested a review from a team as a code owner August 24, 2022 08:52
@PeterSchafer PeterSchafer marked this pull request as draft August 24, 2022 08:58
@PeterSchafer PeterSchafer force-pushed the chore/cliv2_support_alpine branch 2 times, most recently from a580cb4 to aa94877 Compare August 29, 2022 15:24
Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: add alpine build to workflow

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: run v2 jtest on alpine

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: add test alpine to workflow

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: disable usage of sudo

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: temporarily add libc dependency to enable running

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: add additional dependencies for running tests on alpine

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: remove docker and docker-compose from alpine tests

without docker installed proxy tests will be skipped

unfortunately running docker on alpine was causing more issues so this the attempt to skip these tests

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore:  use musl explictly to compile for alpine

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>
Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: remove glibc installation again

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: install maven for alpine tests

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: temporarily skip failing test

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>
Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

chore: adapt step name

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>
@PeterSchafer PeterSchafer marked this pull request as ready for review September 1, 2022 11:56
@PeterSchafer PeterSchafer requested a review from a team as a code owner September 1, 2022 11:56
Copy link
Contributor

@ipapast ipapast left a comment

Choose a reason for hiding this comment

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

approving to unblock

@PeterSchafer PeterSchafer merged commit d779654 into master Sep 1, 2022
@PeterSchafer PeterSchafer deleted the chore/cliv2_support_alpine branch September 1, 2022 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants