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

Test docker build job #61

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Test docker build job #61

wants to merge 2 commits into from

Conversation

tokheim
Copy link

@tokheim tokheim commented Oct 31, 2021

Signed-off-by: Asmund Tokheim tokheim@outlook.com

*Issue number of the reported bug or feature request: #56

Describe your changes
This introduces a root level dockerfile to build a docker image along with github actions to release the container. Just like the author of #56 I'd really need a docker image to be published in order to make use of this very promising project. The docker job is mostly based on the plain example in https://github.com/docker/metadata-action, and should push versioned images whenever you tag the repo. Still there's a lot of customization options you might want to do, so merely a suggestion from me.

Testing performed
I've tested the new dockerfile locally to be working. I also tested the github action on my branch, but haven't published any images, as that seemed like I was overstepping. You can see the result in https://github.com/tokheim/amqpprox/runs/4061595492

Additional context
Before merging you will need to change the image name, and add DOCKERHUB_USERNAME and DOCKERHUB_TOKEN secrets.

Asmund Tokheim added 2 commits October 31, 2021 20:22
Signed-off-by: Asmund Tokheim <tokheim@outlook.com>
Signed-off-by: Asmund Tokheim <tokheim@outlook.com>
@alaric alaric requested a review from adamncasey November 6, 2021 19:00
@adamncasey
Copy link
Contributor

Thanks for the PR Asmund. I agree that having a published Docker image (or any kind of published, built, artifact is going to make it easier for people to use this project.

We never asked around internally about using Dockerhub but we'll do that now.

One unanswered question here is how the docker image would allow configuring of the amqpprox instance. Did you have any thoughts on how this would work for you?

@tokheim
Copy link
Author

tokheim commented Nov 10, 2021

So I hope to experiment a bit on how we could make use of this. But we would likely want to run it in kubernetes and want some way to dynamically change config across all the amqpproxies.

What I'd hope would work is to run amqpprox in one container, and some project that controls the proxy through amqpprox_ctl in another container (sidecar pattern). The two containers would communicate by both mounting a shared volume like https://stackoverflow.com/questions/45637587/connection-between-docker-containers-via-unix-sockets. TCP sockets would be a more standard docker-pattern and allow a bit more flexibility in the setup, but I'm still hopeful this should work well. What do you think?

By the way, you shouldn't feel any obligation to make use of this PR, I just wanted to do whatever I could to speed up the resolution for #56

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.

2 participants