[vprashrex]

Issue

Closes #1294

Summary

• Before: Different keys and types were used in Sentry, causing filters not to match every signal.
• Now: All telemetry attributes come from core/telemetry/constants.py, eliminating string literals for keys.
• Standard OTel keys are now imported from semconv.
• Key renaming for consistency and clarity.

Checklist

Before submitting a pull request, please ensure that you mark these task.

• Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.
• If you've fixed a bug or added code that is tested and has test cases.

Notes

Before merging, update the Sentry dashboard Kaapi · LLM Call: it still filters on kaapi.organization_id, kaapi.project_id and completion_type. No alert rule uses the old keys.

Original PR description

Issue

Closes #1294

Summary

The same concept went to Sentry under different keys and types (tenant.org_id vs kaapi.organization_id, llm.job_id vs guardrails.job_id, int vs string), so one filter never matched every signal.

• All span attributes, Sentry tags and metric names now come from core/telemetry/constants.py; no key is written as a string literal anymore.
• Standard OTel keys (GenAI, http.request.body.size, db.system, user.id) are imported from semconv.
• Renamed: kaapi.organization_id/project_id → tenant.org_id/project_id (always strings), *.job_id → job.id, kaapi.trace_id → correlation_id, llm.provider/llm.request.model → gen_ai.*, completion_type → llm.completion_type, route_category → http.route_category, llm.call.total/errors → llm.call.count/error. log_context(...) uses org_id.
• Every other key keeps its current name so existing dashboards keep working.

Stacked on #1090.

Checklist

• Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.
• If you've fixed a bug or added code that is tested and has test cases.

Notes

Before merging, update the Sentry dashboard Kaapi · LLM Call: it still filters on kaapi.organization_id, kaapi.project_id and completion_type. No alert rule uses the old keys.

[coderabbitai]

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one): 1

• ready-for-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

• Source mode: Auto
• Repository: ProjectTech4DevAI/kaapi-backend
• Branch: feat/1294-telemetry-key-constants
• File: .coderabbit.yaml
• Commit: af4fd70
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: fa11e35d-444c-48f4-bcd9-0597de236329

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.}