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 context to compose file so it runs in older "docker-compose" #249

Merged
merged 2 commits into from Jan 2, 2023

Conversation

bitplane
Copy link
Collaborator

@bitplane bitplane commented Jan 1, 2023

Looks like the reason it was failing in the older docker-compose tool but not docker compose is because recent versions use the current working dir as a default build context, while older versions need it to be explicitly defined.

This should fix running it on machines or in tools that haven't upgraded to docker compose and still call docker-compose instead.

Copy link
Collaborator

@yk yk 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!

@yk yk merged commit a196af7 into LAION-AI:main Jan 2, 2023
@bitplane bitplane deleted the docker-compose-context branch February 10, 2023 17:51
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.

None yet

2 participants