-
Notifications
You must be signed in to change notification settings - Fork 28
[WIP] Add support for private Docker repositories / registries #19
[WIP] Add support for private Docker repositories / registries #19
Conversation
680755e
to
beb27c7
Compare
Hey @yamalight 😺 I've decided to try fixing exoframejs/exoframe#117 but as I don't have much time to work on that I'd really appreciate a bit of help from you, to help me understand how things work around. I've noticed that you're using So my question is, how can that be useful? PS: that PR is (probably) far from being done, but I've raised the MR request so that we have a place to exchange ideas about the approach and evolution 😃 |
@maxime1992 this is awesome! thank you for your work ❤️ Yes indeed, this would only work for non-compose deployments. Compose is kinda completely separate from the rest of exoframe. I think |
Thanks!
That makes sense
I wanted to try that one out but unfortunately I keep getting 502 bad gateway while trying to deploy now for some reasons 🤷♂️ Because I've logged into the Gitlab private registry and I think it should work (???) from docker-compose but I wasn't able to test it. In regards of that PR, it might still be useful IMO to get it done. Do you think that too? For simple Docker images. If so, can you please let me know where you think it might make sense to add that config? I think into the project config but I'm not 100% sure |
@maxime1992 I'd absolutely love to have that for normal docker deployments.
If local exists - it's the one that's used, if not - fall back to global, if it doesn't exists - don't attach anything 🤔 Small note: please change the PR target to |
Sorry I completely lost track on that one and would probably be faster to rework from group up. Feel free to reopen or take over! |
Oh no! 😿 |
This closes exoframejs/exoframe#117