-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Server: buildx for server images #11582
base: dev
Are you sure you want to change the base?
Conversation
Do you know how difficult it would be to add ARM64 support if this pull request is merged? It seems to be particularly challenging to add since several people (including myself) unsuccessfully tried. |
Most likely one line to Docker descriptor and a line to place from were you trigger image building. |
Oh, BTW I just realized that I have not checked your GH actions. Perhaps we should adjust this workflow and think about proper builder for arm (sorry I knows nothing about your workers infrastructure). |
Indeed any new dependencies should be added there, but I guess the check would tell us that - if it's still passing it's probably that the new dependency is already included as part of something else, and so we don't need to change anything. |
5ae0d22
to
aa266fa
Compare
OK, server related checks - pass. Means we built at least However there are few more questions/ideas around this topic:
|
7d264e2
to
017efb6
Compare
f916c27
to
2190517
Compare
Yes, probably, the image is built on GitHub Action
Same, "we" can have everything as long as it works on GitHub Action and most things work. The image is built from GitHub Action by running this script: joplin/.github/scripts/run_ci.sh Line 280 in c6956df
which calls
But I guess as long as the way the script is called doesn't change it should work the same? |
Yes, current behavior should be the same as previously. If I got it right, easiest way is to install QEMU and build amr64 part using it (need to specify |
I thought GitHub supported ARM64 but apparently that's very recent: https://discourse.joplinapp.org/t/github-actions-now-provides-arm64-for-free/43173 Does that make a difference for your pull request? Maybe we don't need QEMU anymore? |
Probably. Arm native builder should be way faster then emulation. In any case switching to BuildX (this PR) and building multiplatform images arr pretty non related steps and should be done in separated PRs. |
189df81
to
95ccabe
Compare
`--addLatestTag` will force `latest` tag creation even for pre-release versions
Just run `yarn buildServerDocker`
95ccabe
to
7cad438
Compare
Switching to Docker BuildX toolset. It's faster, have more benefits for development process (e.g. caching mount points) and open possibility to build multi platform images.