aiidateam / aiida-prerequisites

Docker image that contains all prerequisites allowing to run AiiDA.
2 stars 2 forks source link

Bug: only arm64 architecture is pushed to DockerHub. #54

Closed yakutovicha closed 2 years ago

yakutovicha commented 2 years ago

After #52 has been merged, only one architecture image is available on DockerHub (arm64): Screenshot 2022-07-28 154210

@unkcpz do you have an idea how this could be fixed?

unkcpz commented 2 years ago

I think this issue is exactly the same as we encountered https://github.com/docker/cli/issues/3350. Unfortunately, it is not supported by buildx to have different args for different platform. After some googling I think it worth to try the solution mentioned in the thread https://github.com/docker/cli/issues/3350#issuecomment-1084339451.

unkcpz commented 2 years ago

Please have a look at #55, it will solve the issue.

unkcpz commented 2 years ago

The problem that there is only one entity on dockerhub is caused by the second build_and_push will push with its own manifest list which doesn't include the manifest of the previous one. The buildx of bulid_and_push is supposed to handle creating a manifest list of multiple architectures.