Personal opinion: multi-arch builds are cool, but unless somebody asks for additional builds we should not provide all of them and reduce the list, it's just more CI time spent and more potential failure. Of course I don't know if some of them are useful but I doubt we need all of them.

are the container images smaller now?

Agree. Container VM themselves are the mechanism for cross-arch. I object more complexity for the official docker image.

Note there are third party docker build project, you can take a look

@mmmray

are the container images smaller now?

REPOSITORY                   TAG         IMAGE ID      CREATED       SIZE
ghcr.io/mayampi01/xray-core  latest      636ff1ebc191  25 hours ago  43.4 MB
ghcr.io/xtls/xray-core       latest      a687fbc4a8e6  4 days ago    55.7 MB

@mmmray Now it takes about 9 minutes to build all the images. I am going to limit the multi-arch to amd64, arm64, loong64, and riscv64.

@mmmray @yuhan6665 After limiting multi-arch to amd64, arm64, loong64, and riscv64, it now takes about 3 minutes.
It is not much longer than the original 1.5 to 2 minutes.
https://github.com/mayampi01/Xray-core/actions/workflows/docker.yml
https://github.com/XTLS/Xray-core/actions/workflows/docker.yml

I think this is acceptable, loong64 and riscv64 are becoming more popular.
https://www.deepin.org/en/download/

i'm ok with merging this, @yuhan6665 what do you think?

there is still the fact that it changes default geosite to v2fly, but i think it's ok to be consistent with release tarballs

I still prefer original version. What do you think @RPRX

打算第 1001 个 commit 发 v1.8.22，这个 PR 之后再说

我觉得可以趁着用的人大概还不多把默认 geo 改成 v2fly 的，做到和 releases 一致，不过 tag 太多有点复杂，只加个 -ls 就行了

只加个 -ls 就行了

@mmmray 试着改一下？

i can check it later if the feedback isn't addressed by the author, unless you want this for 1.8.22...

本来想 force-push 他的仓库结果没权限