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

Add script to generate docker-compose.yaml for Qdrant cluster #1489

Merged
merged 9 commits into from
May 25, 2023

Conversation

ffuugoo
Copy link
Contributor

@ffuugoo ffuugoo commented Feb 22, 2023

Edited by @timvisee:

This adds a script to generate a Docker compose configuration to easily start a small cluster in Docker.

I've been using this a few times, so I decided to make it ready to merge into our main repo.

Example usage:

./tools/generate_docker_compose_cluster.sh 5 > docker-compose.yaml

docker-compose up

@ffuugoo ffuugoo requested a review from generall February 22, 2023 14:44
@ffuugoo ffuugoo self-assigned this Feb 22, 2023
@ffuugoo
Copy link
Contributor Author

ffuugoo commented Feb 22, 2023

I've cancelled all workflows, cause there is no reason to run them now. ¯\_(ツ)_/¯

@ffuugoo ffuugoo force-pushed the docker-compose-100-nodes branch 2 times, most recently from ea4609d to a678d0e Compare March 3, 2023 15:10
@timvisee timvisee changed the title Generate docker-compose.yaml to run Qdrant cluster of 100 nodes Add script to generate docker-compose.yaml for Qdrant cluster May 18, 2023
@timvisee timvisee marked this pull request as ready for review May 18, 2023 10:40
@ffuugoo
Copy link
Contributor Author

ffuugoo commented May 18, 2023

@timvisee note there's also a small code change in this PR (though, it's a mostly useful one).

@timvisee
Copy link
Member

@timvisee note there's also a small code change in this PR (though, it's a mostly useful one).

I noticed. I didn't think there was a need to remove it. 😄

Copy link
Contributor Author

@ffuugoo ffuugoo left a comment

Choose a reason for hiding this comment

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

🚲

lib/collection/src/operations/types.rs Outdated Show resolved Hide resolved
lib/collection/src/operations/types.rs Outdated Show resolved Hide resolved
@timvisee timvisee force-pushed the docker-compose-100-nodes branch from f9081a3 to 57ed662 Compare May 18, 2023 12:11
@generall generall merged commit 7fa637c into dev May 25, 2023
@ffuugoo ffuugoo deleted the docker-compose-100-nodes branch May 25, 2023 15:01
generall pushed a commit that referenced this pull request Jun 19, 2023
* Add a simple script to generate `docker-compose.yaml` for a cluster of specified number of nodes

* Improve service error from `RequestError` conversion

* fixup! Improve service error from `RequestError` conversion [skip ci]

* fixup! Add a simple script to generate `docker-compose.yaml` for a cluster of specified number of nodes [skip ci]

* Add CPU limit to the `docker-compose-gen.sh` [skip ci]

* fixup! Add CPU limit to the `docker-compose-gen.sh` [skip ci]

* Move docker-compose generator script into tools directory, add docs

* Apply suggestions from code review

Co-authored-by: Roman Titov <ffuugoo@users.noreply.github.com>

* Simplify error message constructing

---------

Co-authored-by: timvisee <tim@visee.me>
Co-authored-by: Tim Visée <tim+github@visee.me>
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