fix(qdrant): remove deprecated upload_records from instrumentation method lists (#3492)

[LeC-D]

Summary

Fixes #3492

Problem

QdrantClient.upload_records was removed from qdrant-client in v1.13 and is not present in v1.16.1+. The method was still listed in both qdrant_client_methods.json and async_qdrant_client_methods.json, causing an AttributeError at instrumentation time for users on newer qdrant-client versions:

AttributeError: type object 'QdrantClient' has no attribute 'upload_records'

Note: The _instrument method already guards against missing attributes via hasattr(), so the crash only affects older published versions of this package. However, keeping stale entries is misleading — they suggest telemetry is being captured for a method that no longer exists.

Fix

Remove upload_records from both:

• qdrant_client_methods.json
• async_qdrant_client_methods.json

Verification

Installed qdrant-client==1.16.1 and confirmed via dir(QdrantClient) that upload_points and upload_collection are present, but upload_records is not.

Summary by CodeRabbit

• Chores
  • Removed observability/tracing spans for the upload_records operation in both synchronous and asynchronous Qdrant client integrations.

[CLAassistant]

All committers have signed the CLA.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2d425bd4-dd23-4308-9fae-004759fdb837

📥 Commits

Reviewing files that changed from the base of the PR and between ca20848 and e952441.

📒 Files selected for processing (2)

• packages/opentelemetry-instrumentation-qdrant/opentelemetry/instrumentation/qdrant/async_qdrant_client_methods.json
• packages/opentelemetry-instrumentation-qdrant/opentelemetry/instrumentation/qdrant/qdrant_client_methods.json

💤 Files with no reviewable changes (2)

• packages/opentelemetry-instrumentation-qdrant/opentelemetry/instrumentation/qdrant/qdrant_client_methods.json
• packages/opentelemetry-instrumentation-qdrant/opentelemetry/instrumentation/qdrant/async_qdrant_client_methods.json

---

📝 Walkthrough

Walkthrough

Removed upload_records entries from OpenTelemetry instrumentation mappings for both sync and async Qdrant clients; no other mappings were changed.

Changes

Cohort / File(s)	Summary
Qdrant Instrumentation Mappings packages/opentelemetry-instrumentation-qdrant/opentelemetry/instrumentation/qdrant/async_qdrant_client_methods.json, packages/opentelemetry-instrumentation-qdrant/opentelemetry/instrumentation/qdrant/qdrant_client_methods.json	Deleted the upload_records method mapping for AsyncQdrantClient and QdrantClient, removing associated span names (e.g., qdrant.upload_records). No other entries were modified.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped through JSON, tidy and spry,

Found upload_records waving goodbye,
I nudged the mapping, neat and small,
Now instrumentation stands tall! 🥕

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing deprecated upload_records from instrumentation method lists.
Linked Issues check	✅ Passed	The PR directly addresses issue #3492 by removing upload_records from both qdrant_client_methods.json and async_qdrant_client_methods.json, resolving the AttributeError and ensuring compatibility with qdrant-client v1.16.1+.
Out of Scope Changes check	✅ Passed	All changes are in-scope, removing only the stale upload_records method references from the instrumentation method lists as required by issue #3492.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

Tip

You can disable the changed files summary in the walkthrough.

Disable the reviews.changed_files_summary setting to disable the changed files summary in the walkthrough.