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

Add instructions to make minikube's docker env available during 'make docker_build' #586

Merged
merged 1 commit into from
Nov 11, 2020
Merged

Add instructions to make minikube's docker env available during 'make docker_build' #586

merged 1 commit into from
Nov 11, 2020

Conversation

jeremydonahue
Copy link
Contributor

Minikube uses a separate docker daemon from the normal system docker
daemon, so you need to tell your shell how to access the minikube docker
daemon. Otherwise when the Task runs it fails immediately because it
tries to pull from Dockerhub.

This also removes a redundant "to" in the following sentence.

… docker_build'

Minikube uses a separate docker daemon from the normal system docker
daemon, so you need to tell your shell how to access the minikube docker
daemon. Otherwise when the Task runs it fails immediately because it
tries to pull from Dockerhub.

This also removes a redundant "to" in the following sentence.
Copy link
Contributor

@kumare3 kumare3 left a comment

Choose a reason for hiding this comment

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

Thank you so much, welcome.

@kumare3
Copy link
Contributor

kumare3 commented Nov 7, 2020

You have to sign the CLA

@kumare3
Copy link
Contributor

kumare3 commented Nov 7, 2020

Also definitely join our slack, can help in case of any problems

@jeremydonahue
Copy link
Contributor Author

Thanks for the approval, and the warm welcome! I'm working on getting approval from my company to sign the CLA.

@jeremydonahue
Copy link
Contributor Author

@kumare3 I just signed the CLA, please let me know if there's anything else you need from me on this.

@kumare3 kumare3 merged commit edbb361 into flyteorg:master Nov 11, 2020
eapolinario added a commit to eapolinario/flyte that referenced this pull request Jul 24, 2023
Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com>
Co-authored-by: eduardo apolinario <eapolinario@users.noreply.github.com>
eapolinario added a commit to eapolinario/flyte that referenced this pull request Jul 24, 2023
eapolinario added a commit to eapolinario/flyte that referenced this pull request Aug 9, 2023
Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com>
Co-authored-by: eduardo apolinario <eapolinario@users.noreply.github.com>
eapolinario added a commit to eapolinario/flyte that referenced this pull request Aug 21, 2023
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