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

Move Qdrant to the cloud version #320

Merged
merged 3 commits into from
Dec 7, 2022
Merged

Move Qdrant to the cloud version #320

merged 3 commits into from
Dec 7, 2022

Conversation

joanreyero
Copy link
Contributor

Changes proposed ✍️

  • Moved Qdrant to the cloud version. Modified the sync script to bring data from the self-hosted Qdrant to the cloud.

The sync script will be removed after deploy. So will the if cloud checks when initializing the vector API

Checklist ✅

  • Label appropriately with Feature, Enhancement, or Bug.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated:
    • Local frontend configuration: frontend/.env.dist.local, frontend/.env.dist.composed.
    • Local backend: backend/.env.dist.local, backend/.env.dist.composed.
    • Configuration docs have been updated.
    • Team members only: update environment variables in override, staging and production env. files and trigger update config script.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.
  • All changes have been tested in a staging site.
  • All changes are working locally running crowd.dev's Docker local environment.

@joanreyero joanreyero added the Bug Created by Linear-GitHub Sync label Dec 7, 2022
host = "localhost"
else:
host = QDRANT_HOST
if cloud:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This check will be removed. I need this only for syncing after merge

@@ -1,64 +1,34 @@
from pprint import pprint as pp
from crowd.eagle_eye.apis.vector_api import VectorAPI
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This whole file is temporary

@joanreyero joanreyero merged commit d42e2e6 into main Dec 7, 2022
@joanreyero joanreyero deleted the bugfix/qdrant-cloud branch December 7, 2022 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants