Skip to content

Conversation

@tjrivera
Copy link

PR #490 Provides the ability to pull from an insecure registry by passing --allow-insecure-ssl. This commit extends the work done in #490 and adds the ability to pass --allow-insecure-ssl to the up and run commands which will attempt to pull dependent images if they do not exist.

Signed-off-by: Tyler Rivera riverat2@email.chop.edu

@pouicr
Copy link

pouicr commented Oct 22, 2014

+1

1 similar comment
@alaz
Copy link

alaz commented Oct 22, 2014

+1

@torbjornvatn
Copy link

I'm waiting for this as well
👍

@bfirsh bfirsh added this to the 1.0.1 milestone Oct 22, 2014
@bfirsh
Copy link

bfirsh commented Oct 22, 2014

@tjrivera Thanks for the contribution! Would really like to get this in Fig 1.0.1. Any chance we could get a test with a mocked client?

@aanand
Copy link

aanand commented Oct 22, 2014

As I understand it, this adds --allow-insecure-ssl to fig up and fig run. Shouldn't it be added to fig pull too?

@bfirsh
Copy link

bfirsh commented Oct 22, 2014

@aanand it already is

@aanand
Copy link

aanand commented Oct 22, 2014

👍

@bfirsh
Copy link

bfirsh commented Oct 22, 2014

@aanand is that an LGTM?

@aanand
Copy link

aanand commented Oct 22, 2014

Yes, LGTM, though I wouldn't mind a test either.

@tjrivera
Copy link
Author

@bfirsh no problem. I had started sketching out a test for this but the implicit pull relies on an docker.errors.APIError here which I was having problems mocking up properly. I can give it another shot later this morning.

@tjrivera
Copy link
Author

@aanand @bfirsh test added!

PR #490 Provides the ability to pull from an insecure registry by passing --allow-insecure-ssl. This commit extends the work done in #490 and adds the ability to pass --allow-insecure-ssl to the up and run commands which will attempt to pull dependent images if they do not exist.

Signed-off-by: Tyler Rivera <riverat2@email.chop.edu>
@aanand
Copy link

aanand commented Oct 22, 2014

LGTM

1 similar comment
@bfirsh
Copy link

bfirsh commented Oct 22, 2014

LGTM

bfirsh added a commit that referenced this pull request Oct 22, 2014
Allow dependent image pull from insecure registry
@bfirsh bfirsh merged commit 9813a8d into docker:master Oct 22, 2014
@bfirsh
Copy link

bfirsh commented Oct 22, 2014

Great stuff. Thanks @tjrivera!

@tjrivera tjrivera deleted the insecure-registry branch October 22, 2014 17:53
yuval-k pushed a commit to yuval-k/compose that referenced this pull request Apr 10, 2015
Allow dependent image pull from insecure registry
Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
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.

6 participants