Skip to content

chore(deps): pin qdrant_client to a known-good version#1180

Open
ksaurabhAparavi wants to merge 1 commit into
rocketride-org:developfrom
ksaurabhAparavi:chore/RR-1167-pin-qdrant-client
Open

chore(deps): pin qdrant_client to a known-good version#1180
ksaurabhAparavi wants to merge 1 commit into
rocketride-org:developfrom
ksaurabhAparavi:chore/RR-1167-pin-qdrant-client

Conversation

@ksaurabhAparavi

@ksaurabhAparavi ksaurabhAparavi commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pin qdrant_client (1.17.1) so the qdrant node installs a known-good version.

Testing

  • CI (./builder test) — relying on GitHub Actions; not runnable in the contributor's local shell (engine build / Maven / torch unavailable). Static checks (compile, no conflict markers) pass.

Linked Issue

Fixes #1167

@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This PR pins the qdrant_client dependency to version 1.17.1 in the Qdrant node requirements file, replacing an unpinned reference with an explicit version constraint to ensure consistent and compatible client installations.

Changes

Qdrant Client Version Pin

Layer / File(s) Summary
Qdrant client version constraint
nodes/src/nodes/qdrant/requirements.txt
qdrant_client dependency is pinned to version 1.17.1 to guarantee compatible client installations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A version pinned, so neat and sure,
No surprises from the client, pure!
Qdrant's harmony, locked in place,
One-point-seventeen, the perfect grace!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The pull request successfully pins qdrant_client to version 1.17.1 in the requirements.txt file, directly addressing the requirement in linked issue #1167.
Out of Scope Changes check ✅ Passed The only change is pinning qdrant_client to version 1.17.1 in requirements.txt, which is directly within scope of issue #1167's objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: pinning qdrant_client to a specific version in the requirements file.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the module:nodes Python pipeline nodes label Jun 8, 2026
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
🤖 Internal: Discord sync marker

Auto-managed by the Discord notification workflow. Stores the linked Discord message ID. Do not edit or delete.

Pin qdrant_client (1.17.1) so the qdrant node installs a known version.

Fixes rocketride-org#1167
@ksaurabhAparavi ksaurabhAparavi force-pushed the chore/RR-1167-pin-qdrant-client branch from bdccc8b to ac4eb22 Compare June 8, 2026 11:51
@ksaurabhAparavi ksaurabhAparavi changed the title fix(qdrant): ADS-395 pin qdrant_client version (#22) chore(deps): pin qdrant_client to a known-good version Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:nodes Python pipeline nodes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pin qdrant_client to a known-good version

2 participants