Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Please mirror your docker images to other registries #10977

Closed
duritong opened this issue Oct 3, 2021 · 9 comments · Fixed by #15282
Closed

Please mirror your docker images to other registries #10977

duritong opened this issue Oct 3, 2021 · 9 comments · Fixed by #15282
Assignees
Labels
A-Docker Docker images, or making it easier to run Synapse in a container. P4 (OBSOLETE: use S- labels.) Okay backlog: will not schedule, will accept patches T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Comments

@duritong
Copy link

duritong commented Oct 3, 2021

The docker hub rate limit is affecting fetching the image - it would be nice if the images are also available on other registries such as quay.io where no such limits apply.

@babolivier
Copy link
Contributor

We're intentionally being cautious not to add too many release channels as doing so would add more maintenance overhead. However, people from the community should feel free to set up community-driven channels and PR instructions on how to use them to the Synapse install documentation.

@ibotty
Copy link

ibotty commented Mar 30, 2022

Can you reconsider? It makes consuming that image very hard for people with many computers running behind a single natted gateway.

@richvdh richvdh reopened this Mar 30, 2022
@richvdh
Copy link
Member

richvdh commented Mar 30, 2022

It might well be worth reconsidering this now that Docker are beginning to ratelimit access.

@DMRobertson
Copy link
Contributor

@babolivier babolivier added the T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. label Mar 31, 2022
@callahad
Copy link
Contributor

Not opposed, but concerned about long-term maintenance... We'd be open to pushing to the GitHub Container Registry, but other solutions like Quay are probably too niche to be worth the fragmentation.

@callahad callahad added the P4 (OBSOLETE: use S- labels.) Okay backlog: will not schedule, will accept patches label Mar 31, 2022
@reivilibre
Copy link
Contributor

https://github.com/matrix-org/synapse/pull/11852/files (a PR that adds another docker image that was pushed to GHCR) could be a good source of 'inspiration' for a PR which added the GHCR bits to our existing docker image (i.e. pushing to both).

@ibotty
Copy link

ibotty commented Mar 31, 2022

I don't quiet get why quay is considered niche, but having anything else than dockerhub would be great.

@clokep
Copy link
Member

clokep commented Mar 15, 2023

#15270 also discussed recent changes Docker is making, @williamkray said:

Docker as a company has taken several moves that leave me concerned about their friendliness to open source projects, and clearly have an aggressive approach to handling their customers. I would feel much safer knowing that matrix.org published images to multiple "official" locations, perhaps including ghcr.io (a feature built into this repository) or ECR public (hosted by AWS and mirroring many images already) so that I can pull from a different source if/when Dockerhub decides to change their policies again.

for reference, dockerhub's latest move adds fees to open source projects. matrix.org may not be affected by this this time, but as an end user i would prefer not to continue to use their services, and redundancy is always a good idea.

news.ycombinator.com/item?id=35166317

@DMRobertson
Copy link
Contributor

Let's make the release CI upload to GHCR as well as dockerhub---it should be straightforward(?!)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Docker Docker images, or making it easier to run Synapse in a container. P4 (OBSOLETE: use S- labels.) Okay backlog: will not schedule, will accept patches T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants