Skip to content

docker: support docker build --ssh. Fixes issue #2437#2750

Merged
nicks merged 1 commit into
masterfrom
nicks/issue2437/ssh
Jan 9, 2020
Merged

docker: support docker build --ssh. Fixes issue #2437#2750
nicks merged 1 commit into
masterfrom
nicks/issue2437/ssh

Conversation

@nicks

@nicks nicks commented Jan 9, 2020

Copy link
Copy Markdown
Contributor

Hello @jazzdan, @maiamcc,

Please review the following commits I made in branch nicks/issue2437/ssh:

1871117 (2020-01-09 13:30:58 -0500)
docker: support docker build --ssh. Fixes issue #2437

@nicks
nicks requested review from jazzdan and maiamcc January 9, 2020 18:31
@nicks

nicks commented Jan 9, 2020

Copy link
Copy Markdown
Contributor Author

Tested by creating a dockerfile that clones a private repo, and then testing that it works with docker_build(ssh='default') but fails without that parameter

@nicks
nicks force-pushed the nicks/issue2437/ssh branch from 1871117 to 80aef1e Compare January 9, 2020 18:53

@jazzdan jazzdan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@nicks
nicks merged commit e1f020e into master Jan 9, 2020
@nicks
nicks deleted the nicks/issue2437/ssh branch January 9, 2020 21:21
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