Skip to content

Docker compose improvements and cleanup #54

Merged
hari-kuriakose merged 4 commits into
mainfrom
docker-compose-improvements
Mar 4, 2024
Merged

Docker compose improvements and cleanup #54
hari-kuriakose merged 4 commits into
mainfrom
docker-compose-improvements

Conversation

@jaseemjaskp

Copy link
Copy Markdown
Contributor

What

  • replaced pgvector with official postgres docker image
  • removed worker/requirements.txt
    ...

Why

  • replaced pgvector with qdrant service. so pg vector is not required anymore. so replaced it with official postgres
  • moved into pdm and pyproject.toml from traditional pip install requirements.txt
    ...

How

...

Relevant Docs

Related Issues or PRs

Dependencies Versions / Env Variables

Notes on Testing

...

Screenshots

...

Checklist

I have read and understood the Contribution Guidelines.

@johnyrahul johnyrahul left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hari-kuriakose hari-kuriakose left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hari-kuriakose
hari-kuriakose merged commit 05ba7da into main Mar 4, 2024
@hari-kuriakose
hari-kuriakose deleted the docker-compose-improvements branch March 4, 2024 10:37
pk-zipstack pushed a commit that referenced this pull request Aug 20, 2025
* replaced pgvector with official postgres docker image
* uncommented previously commented proxy_overrides.yaml  for testing
* removed old requirements.txt

---------

Co-authored-by: Hari John Kuriakose <hari@zipstack.com>
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.

3 participants