-
Notifications
You must be signed in to change notification settings - Fork 2
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
[CAI-185] Chatbot/docker compose with Redis and DynamoDB for local development #1193
Merged
Merged
Changes from 12 commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
ac9b39a
feat(chatbot): session GSI
batdevis 36c89af
feat(chatbot): docker compose
batdevis a2c16c7
fix(chatbot): dynamodb and redis for local development with docker co…
batdevis c633df6
Merge branch 'chatbot/docker-compose-complete' into chatbot/sessions-…
batdevis 08a7fec
chore(chatbot):remove duplicate imports
batdevis 7fe9101
chore(chatbot): linting
batdevis 9b2eb61
fix(chatbot):create index in docker
batdevis b251837
chore(chatbot): llamaindex index id
batdevis 4ecd4b3
fix(chatbot): create vector index with all docs
batdevis a84581c
Merge branch 'main' into chatbot/docker-compose-complete
batdevis ea7d3db
chore(chatbot): terraform lint
batdevis 28695e3
fix(chatbot): terraform syntax
batdevis 238edfd
chore(chatbot): remove dynamodb options
batdevis 5f63560
chore(chatbot): from global to local secondary index
batdevis 859c298
Merge branch 'main' into chatbot/docker-compose-complete
batdevis f43a771
chore: merge main
batdevis d96a9f9
chore: remove old var
batdevis 9526c17
chore: merge main
batdevis a5177df
Update apps/chatbot/docker/compose.yaml
batdevis 4daccf8
chore: remove logs
batdevis f5bdfd3
Merge branch 'chatbot/docker-compose-complete' of github.com:pagopa/d…
batdevis c123b5c
fix(chatbot): compose vars
batdevis aa59ca5
Merge branch 'main' into chatbot/docker-compose-complete
batdevis 5e07dbe
Update modules
mdciri da8a41c
Update config prompts
mdciri b57d55c
Update env example
mdciri 62fffa1
Merge branch 'main' into chatbot/docker-compose-complete
batdevis f7b05e6
Merge branch 'languages/chatbot/cai-198' into chatbot/docker-compose-…
batdevis 278d56d
redis admin port
batdevis File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,55 @@ | ||
services: | ||
api: | ||
build: | ||
context: .. | ||
dockerfile: docker/app.local.Dockerfile | ||
ports: | ||
- "8080:8080" | ||
volumes: | ||
- ..:/app | ||
- ./files/.aws:/root/.aws | ||
- ./build-devp/out:/app/build-devp/out | ||
depends_on: | ||
redis: | ||
condition: service_started | ||
dynamodb: | ||
condition: service_started | ||
networks: | ||
- ntw | ||
|
||
dynamodb: | ||
image: amazon/dynamodb-local:2.5.2 | ||
environment: | ||
- AWS_ACCESS_KEY_ID=dummy | ||
- AWS_SECRET_ACCESS_KEY=dummy | ||
- AWS_DEFAULT_REGION=local | ||
ports: | ||
- "8000:8000" | ||
networks: | ||
- ntw | ||
|
||
redis: | ||
image: redis/redis-stack:7.2.0-v13 | ||
ports: | ||
- "6379:6379" | ||
networks: | ||
- ntw | ||
|
||
create_index: | ||
build: | ||
context: .. | ||
dockerfile: docker/app.local.Dockerfile | ||
ports: | ||
- "8080:8080" | ||
volumes: | ||
- ..:/app | ||
batdevis marked this conversation as resolved.
Show resolved
Hide resolved
|
||
command: "python src/modules/create_vector_index.py --params config/params.yaml" | ||
tty: true | ||
depends_on: | ||
redis: | ||
condition: service_started | ||
networks: | ||
- ntw | ||
|
||
networks: | ||
ntw: |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
#!/bin/bash | ||
docker compose -f docker/compose.yaml -p chatbot up api |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
#!/bin/bash | ||
docker compose -f docker/compose.yaml -p chatbot up create_index |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
#!/bin/bash | ||
docker run -it --env-file ./.env fastapi-local bash |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
[profile default] | ||
region = eu-south-1 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
[default] | ||
aws_access_key_id = 123 | ||
aws_secret_access_key = xyz | ||
christian-calabrese marked this conversation as resolved.
Show resolved
Hide resolved
|
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about the naming:
We currently have a region we use for everything (eu-south-1) and a region we use only for bedrock and S3 (eu-west-3)
What do you think about changing the names to CHB_AWS_DEFAULT_REGION and CHB_AWS_BEDROCK_REGION? cc @mdciri
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same for me, I think @mdciri will update it in another pr