-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Docker multi-arch base #1985
Docker multi-arch base #1985
Conversation
For information on how it works : Now we only need one |
da5f6a6
to
4c17c4e
Compare
Rebased & use sapk/gitea-base as base maintained by a simple script (trigerred by travis): https://github.com/sapk/dockerfiles/blob/master/gitea/build.sh. |
@go-gitea/owners anyone up for this or should we move this to 1.4? |
I suppose to move it to 1.4 |
I don't believe that this will work like it is. |
@tboerger what do you think is missing ? |
The schema simply looks wrong... You are just referencing base images... |
@tboerger base image that is multi-arch so that you can build the same Dockerfile on any arch (arm, x64). This will not give a multi-arch image of gitea on docker hub but it is the first step and since my previous PR was a lot complicated, I split it into "small" steps and first one is to have one Dockerfile for all platform. |
The best would be to run a script like https://github.com/sapk/dockerfiles/blob/master/gitea/build.sh (simply make docker-multi-arch-push-manifest when merged) with gitea docker hub creds to use gitea/gitea-base in place of sapk/gitea-base. The next step is to build the image for each arch and use |
0988819
to
3c88922
Compare
Rebase on master. |
During the opening of this PR and now alpine base image is multi-arch so no need any more for custom multi-arch base. (7 hours ago ^^) |
|
This PR is now very simple. I kept a separate Makfile for docker since next steps will add more tasks. |
Codecov Report
@@ Coverage Diff @@
## master #1985 +/- ##
==========================================
- Coverage 27.26% 26.85% -0.41%
==========================================
Files 89 89
Lines 17640 17600 -40
==========================================
- Hits 4809 4727 -82
- Misses 12144 12187 +43
+ Partials 687 686 -1
Continue to review full report at Codecov.
|
LGTM |
@sapk Can you resolve conflicts. Otherwise LGTM |
serve as base for build docker image for most platform (386,amd64,arm,arm64)
To update references of a multi-arch image on docker registry.
Delete Dockerfile.rpi
and replace deprecated MAINTAINER by LABEL (https://docs.docker.com/engine/reference/builder/#maintainer-deprecated)
Could use in later version edge of alpine official library that support multi-arch for armhf.
2cadeec
to
1f5c252
Compare
@lafriks should be good |
This doesn't make any sense to me but I'm out of that. Have fun with it... |
This PR permit to have only one Dockerfile that is buildable on most platform (386,amd64,arm,arm64) since it will use a multi-arch image as a base.
For this to work, we need a multi-arch image (gitea/gitea-base) that can be published via
make docker-multi-arch-push-manifest
with default params (DOCKER_MANIFEST ?= docker/manifest/base.yml
) by any one that have acces to docker hub gitea account.For testing purpose you could use sapk/gitea-base as a base. (made with DOCKER_IMAGE=sapk/gitea make docker-multi-arch-push-manifest).
First part of #1615