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

Revert "Fix llama stack build docker creation to have correct entrypoint #872

Closed
wants to merge 1 commit into from

Conversation

terrytangyuan
Copy link
Contributor

@terrytangyuan terrytangyuan commented Jan 24, 2025

This reverts commit a8345f5.

Fixes #870.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jan 24, 2025
@ashwinb
Copy link
Contributor

ashwinb commented Jan 24, 2025

This is not the correct fix even though it fixes the issue. The offending commit was landed because it fixed docker running when you built the docker for templates (--template fireworks, e.g.)

We need to maintain that fix and fix the regression caused when passing a config file as an argument with the --template parameter.

Closing this revert PR and I will forward fix this shortly.

@ashwinb ashwinb closed this Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issues with llama stack build's --config and --template args
3 participants