Skip to content

Commit

Permalink
[BUG]: Docker entrypoint logging path (chroma-core#1159)
Browse files Browse the repository at this point in the history
## Description of changes

*Summarize the changes made by this PR.*
 - Improvements & Bug fixes
- Initial CLI PR (chroma-core#1032) moved
the logging config inside chromadb. If image is built with the current
setup it will result in Error: Invalid value for '--log-config': Path
'log_config.yml' does not exist.

## Test plan
*How are these changes tested?*

Steps to reproduce (prior to this PR):

- `docker build -t chroma:canary .`
- `docker run --rm -it chroma:canary`


## Documentation Changes
*Are all docstrings for user-facing APIs updated if required? Do we need
to make documentation changes in the [docs
repository](https://github.com/chroma-core/docs)?*
  • Loading branch information
tazarov authored Sep 20, 2023
1 parent 7d2dd01 commit dffc806
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion bin/docker_entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@
echo "Rebuilding hnsw to ensure architecture compatibility"
pip install --force-reinstall --no-cache-dir chroma-hnswlib
export IS_PERSISTENT=1
uvicorn chromadb.app:app --workers 1 --host 0.0.0.0 --port 8000 --proxy-headers --log-config log_config.yml
uvicorn chromadb.app:app --workers 1 --host 0.0.0.0 --port 8000 --proxy-headers --log-config chromadb/log_config.yml

0 comments on commit dffc806

Please sign in to comment.