Skip to content

Commit

Permalink
first cut at redis (onyx-dot-app#2226)
Browse files Browse the repository at this point in the history
* first cut at redis

* fix startup dependencies on redis

* kombu cleanup - fail silently

* mypy

* add redis_host environment override

* update REDIS_HOST env var in docker-compose.dev.yml

* update the rest of the docker files

* update contributing guide

* renaming cache to cache_volume

* add redis password to various deployments

* try setting up pr testing for helm

* fix indent

* hopefully this release version actually exists

* fix command line option to --chart-dirs

* fetch-depth 0

* edit values.yaml

* try setting ct working directory

* bypass testing only on change for now

* move files and lint them

* update helm testing

* some issues suggest using --config works

* add vespa repo

* add postgresql repo

* increase timeout

* try amd64 runner

* fix redis password reference

* add comment to helm chart testing workflow

* rename helm testing workflow to disable it

---------

Co-authored-by: Richard Kuo <rkuo@rkuo.com>
  • Loading branch information
2 people authored and rajiv chodisetti committed Oct 2, 2024
1 parent 7304c12 commit 7a015de
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 1 deletion.
6 changes: 6 additions & 0 deletions .github/workflows/pr-helm-chart-testing.yml.disabled.txt
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,15 @@ jobs:
backend/requirements/model_server.txt
- run: |
python -m pip install --upgrade pip
<<<<<<< HEAD
pip install --retries 5 --timeout 30 -r backend/requirements/default.txt
pip install --retries 5 --timeout 30 -r backend/requirements/dev.txt
pip install --retries 5 --timeout 30 -r backend/requirements/model_server.txt
=======
pip install -r backend/requirements/default.txt
pip install -r backend/requirements/dev.txt
pip install -r backend/requirements/model_server.txt
>>>>>>> 6daea805 (first cut at redis (#2226))

- name: Set up chart-testing
uses: helm/chart-testing-action@v2.6.1
Expand Down
2 changes: 1 addition & 1 deletion backend/danswer/background/celery/celery_app.py
Original file line number Diff line number Diff line change
Expand Up @@ -1216,4 +1216,4 @@ def on_setup_logging(
"options": {"priority": DanswerCeleryPriority.HIGH},
},
}
)
)
1 change: 1 addition & 0 deletions deployment/docker_compose/docker-compose.gpu-dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -365,6 +365,7 @@ services:


volumes:
cache_volume:
db_volume:
vespa_volume:
# Created by the container itself
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -219,6 +219,7 @@ services:


volumes:
cache_volume:
db_volume:
vespa_volume:
# Created by the container itself
Expand Down
1 change: 1 addition & 0 deletions deployment/docker_compose/docker-compose.prod.yml
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,7 @@ services:


volumes:
cache_volume:
db_volume:
vespa_volume:
# Created by the container itself
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -215,6 +215,7 @@ services:


volumes:
cache_volume:
db_volume:
driver: local
driver_opts:
Expand Down

0 comments on commit 7a015de

Please sign in to comment.