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(CI): cancel build of 32-bit docker #5144

Merged
merged 1 commit into from
Jul 6, 2021

Conversation

Rory-Z
Copy link
Member

@Rory-Z Rory-Z commented Jun 30, 2021

@turtleDeng @tigercl @zmstone @qzhuyan @wivwiv

Quic does not support 32-bit builds, do we still need to continue to support 32-bit docker images ?

@qzhuyan
Copy link
Contributor

qzhuyan commented Jun 30, 2021

I will make a PR for emqx to disable quic build in rebar3 script so the quic support will be disabled on these platforms.

With that emqx can stil deliver packages for these platforms while only the quic support function will be missing.

@Rory-Z
Copy link
Member Author

Rory-Z commented Jul 1, 2021

I will make a PR for emqx to disable quic build in rebar3 script so the quic support will be disabled on these platforms.

With that emqx can stil deliver packages for these platforms while only the quic support function will be missing.

Yes, it's a great feature, but I think we should try to make sure that the published images are consistent

Users can build their own images on a 32-bit system that does not support quic

@qzhuyan
Copy link
Contributor

qzhuyan commented Jul 3, 2021

I fixed the 32bit build of quicer but now the problem is the docker image needs a compiler upgrade to gcc 5 or above.

We are using alpine-i386 for compiling i386 images, can we switch it to something else? like centos7 32bit or ubuntu 32bit?

@Rory-Z
Copy link
Member Author

Rory-Z commented Jul 3, 2021

I fixed the 32bit build of quicer but now the problem is the docker image needs a compiler upgrade to gcc 5 or above.

We are using alpine-i386 for compiling i386 images, can we switch it to something else? like centos7 32bit or ubuntu 32bit?

We need to use alpine as base image because we want to minimize the size of the docker

If gcc5 cannot be installed on alpine 386, I think we can discard the support for 386

s390x I think we can give up support, we just need to support amd64, arm64, and arm32v7

@Rory-Z Rory-Z force-pushed the chore/cancel-build-of-32-bit-docker branch from 034fe11 to 1c6447e Compare July 5, 2021 09:30
@@ -218,7 +218,7 @@ jobs:
os:
- ubuntu20.04
- ubuntu18.04
- ubuntu16.04
Copy link
Contributor

Choose a reason for hiding this comment

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

I have tested that quic can build on 16.04

Copy link
Member Author

Choose a reason for hiding this comment

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

Cool,let's keep it

@@ -341,14 +341,6 @@ jobs:
arch:
- [amd64, x86_64]
- [arm64v8, aarch64]
- [arm32v7, arm]
Copy link
Contributor

Choose a reason for hiding this comment

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

Worth to keep it? Since quicer can support it and the majority of raspberrypi are running this arch I think.

Copy link
Member Author

Choose a reason for hiding this comment

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

If the quic can support armv7, let's keep it

@qzhuyan
Copy link
Contributor

qzhuyan commented Jul 5, 2021

See also: #5144

@Rory-Z Rory-Z force-pushed the chore/cancel-build-of-32-bit-docker branch from 1c6447e to bfa6025 Compare July 5, 2021 10:02
@Rory-Z Rory-Z merged commit b280129 into emqx:master Jul 6, 2021
@Rory-Z Rory-Z deleted the chore/cancel-build-of-32-bit-docker branch July 6, 2021 01:26
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.

3 participants