Skip to content

Update docs to demonstrate specifying Qdrant version #917

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.

Sign up for GitHub

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

Merged
cristianrgreco merged 5 commits into from
Mar 6, 2025
Merged

Update docs to demonstrate specifying Qdrant version #917

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
f43b9d1
docs: Demonstrate Qdrant version
Anush008 Mar 5, 2025
4131fe4
chore: Bump Qdrant version
Anush008 Mar 5, 2025
f13cc86
chore: Pin to qdrant/qdrant:v1.13.4
Anush008 Mar 5, 2025
5822912
Fix lint
cristianrgreco Mar 5, 2025
d5197b5
fix: Status message qdrant-container.test.ts
Anush008 Mar 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions packages/modules/qdrant/src/qdrant-container.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

// connectQdrantSimple {
it("should connect to the client", async () => {
const container = await new QdrantContainer().start();
const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").start();

const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}` });

Expand All @@ -22,7 +22,7 @@
it("should work with valid API keys", async () => {
const apiKey = crypto.randomUUID();

const container = await new QdrantContainer().withApiKey(apiKey).start();
const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withApiKey(apiKey).start();

const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, apiKey });

Expand All @@ -35,7 +35,7 @@
it("should fail for invalid API keys", async () => {
const apiKey = crypto.randomUUID();

const container = await new QdrantContainer().withApiKey(apiKey).start();
const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withApiKey(apiKey).start();

const client = new QdrantClient({
url: `http://${container.getRestHostAddress()}`,
Expand All @@ -49,7 +49,7 @@

// connectQdrantWithConfig {
it("should work with config files - valid API key", async () => {
const container = await new QdrantContainer().withConfigFile(path.resolve(__dirname, "test_config.yaml")).start();
const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start();

Check failure on line 52 in packages/modules/qdrant/src/qdrant-container.test.ts

View workflow job for this annotation

GitHub Actions / lint 

Replace `.withConfigFile(path.resolve(__dirname,·"test_config.yaml"))` with `⏎······.withConfigFile(path.resolve(__dirname,·"test_config.yaml"))⏎······`

const client = new QdrantClient({ url: `http://${container.getRestHostAddress()}`, apiKey: "SOME_TEST_KEY" });

Expand All @@ -60,7 +60,7 @@
// }

it("should work with config files - invalid API key", async () => {
const container = await new QdrantContainer().withConfigFile(path.resolve(__dirname, "test_config.yaml")).start();
const container = await new QdrantContainer("qdrant/qdrant:v1.13.4").withConfigFile(path.resolve(__dirname, "test_config.yaml")).start();

Check failure on line 63 in packages/modules/qdrant/src/qdrant-container.test.ts

View workflow job for this annotation

GitHub Actions / lint 

Replace `.withConfigFile(path.resolve(__dirname,·"test_config.yaml"))` with `⏎······.withConfigFile(path.resolve(__dirname,·"test_config.yaml"))⏎······`

const client = new QdrantClient({
url: `http://${container.getRestHostAddress()}`,
Expand Down
2 changes: 1 addition & 1 deletion packages/modules/qdrant/src/qdrant-container.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ export class QdrantContainer extends GenericContainer {
private apiKey: string | undefined;
private configFilePath: string | undefined;

constructor(image = "qdrant/qdrant:v1.8.1") {
constructor(image = "qdrant/qdrant:v1.13.4") {
super(image);
this.withExposedPorts(QDRANT_REST_PORT, QDRANT_GRPC_PORT);
this.withWaitStrategy(
Expand Down
Loading