Skip to content
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

fix pushing images to docker.io #592

Merged
merged 2 commits into from
Jan 11, 2024
Merged

fix pushing images to docker.io #592

merged 2 commits into from
Jan 11, 2024

Conversation

snpz
Copy link
Contributor

@snpz snpz commented Jan 10, 2024

Hi @gaul.

I took me a lot of time, reading and trial and error but I've finally come up with something to fix #479 and prepare for issue #591 I created myself yesterday.

I never used Github Actions myself so I googled a bit and found out that the official action that is used to push to ghcr.io can also be used to push to docker.io without relying on regctl.

To do this one needs to pass multiple image names that are the base for building the targets to push to.

According to how the tags in the meta action are currently setup these changes resulted in 3 tags getting pushed to both ghcr.io and docker.io each time I pushed to my branch. (docker.io depending on the existence of the DOCKER_PASS secret as before)

docker.io

docker.io

ghrc.io

ghrc.io

I also put the variable that checks for presence of the secret on the top level as it made it more readable and use some never action versions in a few places. And configured caching via github actions. This cache type is still experimental but the recommended one.

I'm not contributing a lot so if I miss something that is expected please let me know.

@snpz snpz marked this pull request as ready for review January 10, 2024 17:28
@gaul gaul merged commit d35eb25 into gaul:master Jan 11, 2024
3 checks passed
@gaul
Copy link
Owner

gaul commented Jan 11, 2024

Let's see if this improves things. Thanks for your contribution @snpz!

@snpz
Copy link
Contributor Author

snpz commented Jan 11, 2024

@gaul Would be even better if it would work like expected, not just on my machine. ;)
I'll have another look and prepare another PR to get it working.

@snpz snpz deleted the ci-fix-docker-push branch January 11, 2024 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI failing
2 participants