-
-
Notifications
You must be signed in to change notification settings - Fork 78
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
Add config/environment/cli options to configure docker socket URL #195
Add config/environment/cli options to configure docker socket URL #195
Conversation
b9e5156
to
df2e27c
Compare
df2e27c
to
89d6309
Compare
I just updated the PR, to enable the ability use a hostname / container_name to in |
@mageddo any update on this ? |
Well, these are a lot of changes to be done at one shot, not sure if they are dependent. I think we may try isolate the Do you wanna do it by yourself or prefer me to do the job @imartinezortiz ? |
Let me do a test and I'll get back to you. If I remember correctly the
main reason of all these changes were that the docket docket proxy requires
at least docker API v 1.27 and the dependency that DPS is currently using
it is compatible up to v1.21 because it is deprecated.
…On Fri, Jul 10, 2020, 01:12 Elvis Souza ***@***.***> wrote:
Well, these are a lot of changes to be done at one shot, not sure if they
are dependent. I think we may try isolate the Add config/environment/cli
options to configure docker socket URL feature, then we can merge it ASAP.
Do you wanna do it by yourself or prefer me to do the job @imartinezortiz
<https://github.com/imartinezortiz> ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#195 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AALF7P2NU5ZURGPKF2YRYHDR2ZFETANCNFSM4MZBSXOQ>
.
|
I've done some tests just adding the configuration options and seems to work at least for my use case. I'm getting some panics from time to time and that is the reason I tried to update the API, but to be honest, I also got some panics after doing the update (golang and docker API dependency). Reviewing both DPS and tecnativa/docker-socket-proxy, maybe the problem is related due to the fact docker-socket-proxy return a HTML response if the request it is forbidden and DPS panics. To sum up:
|
Hey, I was stale for some time, below some updates:
So I'm closing this one, thanks for the help, sorry we didn't solve it earlier. |
The main objective of this PR is to add a couple configuration options to specify the URL to connect to the docker daemon and to specify the docker API version to use. The idea is to use tecnativa/docker-socket-proxy to restrict and control the access of DPS to the docker demon.
I also created some tests, updated the documentation and provided an example docker compose that shows how to use the new feature.
Besides the PR also does:
If you prefer / think that these additional changes require a separate PR let me know.