Closed bugy closed 10 months ago
The only thing we need to check is the docker build process, which is only triggered if we create a release :/
Ah no, so the problem is deeper than I thought. Since the Docker image is just copying the process, which is specific for amd64. I will investigate and return back to you later
Sorry, which process are you talking about?
I guess the right thing to do is having a standalone Dockerfile as in https://github.com/bugy/script-server/pull/206. We can use the multi-stage build technique to keep the image size as small as possible (probably approximately equal to the current one) so your concern in this https://github.com/bugy/script-server/pull/206#issuecomment-481225031 is solved
Sorry, which process are you talking about?
Sorry, I mean the output of the CI process, the build/script-server.zip
But the zip file should contain only python files, which are supposed to be platform independent
I missed that part. Could you trigger the docker build ?
I'm trying to make docker image work for PRs
@vnghia there is an error:
unknown flag: --platform
Is there a way I can test it on my PR ?
can you try pushing any changes from your PR? I made a change to travis, probably it should build your PR now
Still not running, but adding three lines here is good I think: https://github.com/bugy/script-server/pull/715/files#diff-6ac3f79fc25d95cd1e3d51da53a4b21b939437392578a35ae8cd6d5366ca5485
Can you show me the log :/
https://app.travis-ci.com/github/bugy/script-server/builds/267081647
don't you have access to this section?
btw, I gave you access to this repo, you can try doing changes in my branch
I see the log now, Travis blocked me while I am on VPN :/
hmm it seems that the CI won't run if there are my trace in the PR, maybe because I am a new contributor and it won't run unless you have to approve the PR once ?
@vnghia I triggered the build manually and it worked! Could you try running arm docker container, please? https://hub.docker.com/repository/docker/bugy/script-server/tags?page=1&ordering=last_updated
It's working! Actually, you can test it on your own computer with docker run --rm -it --platform linux/arm64 -p 5000:5000 bugy/script-server:test-amd-build
assuming that you have qemu
installed.
Cool, thanks! Could you apply the same changes to your PR please? And then we merge it
Done !
Test results: https://script-server-tests.s3-us-west-2.amazonaws.com/master/750/index.html