Skip to content

Add support for remote DOCKER_HOST #270

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

Closed
wants to merge 5 commits into from
Closed

Conversation

str3tch
Copy link

@str3tch str3tch commented Oct 19, 2018

Hi,

On our corporate network, we have a remote docker machine (defined in the DOCKER_HOST env var), rather than being able to spin up containers locally. This means that when you use Dockerizepip, it fails when trying to mount the volume (can't mount local filesystem to a remote docker server). So.. the fix is that instead of mounting the local folder, create a docker volume, copy the files into it, mount that to the container, do the pip install magic, then copy the files back off the container.

I've got this working, see this PR. I have run the tests and they pass, but I can't get the BATS tests to run on my Mac. so not quite there.

I am willing to beef up the testing if it's required.. but first I'd like feedback that this PR is something that you're actually interested in.

Let me know..

cheers!
Str3tch

@dschep
Copy link
Contributor

dschep commented Oct 22, 2018

Nice PR! Thanks @str3tch.

Could you run yarn format to get the CI build passing? For the testing, feel free to implement your test in the tape based tests in test.js instead of worrying about getting bats running.

@str3tch
Copy link
Author

str3tch commented Oct 23, 2018

ok great!

I'll try and get around to this in the next week or so.

cheers!

ChrisD-Woodside and others added 3 commits November 8, 2018 12:52
pythonRequirements:
dockerizePip: true
dockerRemoteHost: true
proxy: http://yourproxy:8080
Copy link
Contributor

@dschep dschep Nov 8, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just merged #271. Please remove the proxy option as users can use the dockerEnv feature to pass proxies in.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok great, will do

custom:
pythonRequirements:
dockerizePip: true
dockerRemoteHost: true
Copy link
Contributor

@dschep dschep Nov 8, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the plugin maybe not actually have an option for this, and automatically do all the volume stuff if DOCKER_HOST is set?

@str3tch
Copy link
Author

str3tch commented Nov 9, 2018

Sorry for delay on getting this sorted, smashed at work at the mo.

I did do a yarn format but still seems to complain.. will try and take a look next week and get some tests together.

@jamesryanbell
Copy link

Is there any update on this? 😃

@alexfre
Copy link

alexfre commented Oct 13, 2019

Any news on this? It's required to run on CI engines like CircleCI which utilize remote docker machine

@str3tch
Copy link
Author

str3tch commented Oct 13, 2019

Hey guys, apologies.. been snowed under at work and have a newborn at home - pretty hectic! I’ll try and get to it, but it’s not going to be asap .. if anyone wants to tidy it up, not much to do to get it passing another review.

@miketheman miketheman added Docker waiting-on-response If an issue goes without response for a while, close it. enhancement labels Feb 22, 2020
@Dunedan
Copy link

Dunedan commented Dec 22, 2020

Any update on this? I'd much appreciate being able to run the packaging on a remote docker host as well.

@pgrzesik
Copy link
Contributor

Hey @str3tch - it's been a long time since this PR was proposed. I'm going to close it, if you feel like the issue is valid, please open a new issue or a new PR against the latest main branch. Thanks 🙇

@pgrzesik pgrzesik closed this Sep 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docker enhancement waiting-on-response If an issue goes without response for a while, close it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants