Skip to content

chore(ai): Remove gen_ai_usage_total_cost attribute and stop double writing costs#5471

Merged
vgrozdanic merged 2 commits intomasterfrom
vg/stop-using-usage-total-cost
Dec 11, 2025
Merged

chore(ai): Remove gen_ai_usage_total_cost attribute and stop double writing costs#5471
vgrozdanic merged 2 commits intomasterfrom
vg/stop-using-usage-total-cost

Conversation

@vgrozdanic
Copy link
Member

@vgrozdanic vgrozdanic commented Dec 10, 2025

3 months have passed since we started double writing costs, so all the data in Sentry can now use the new attribute (gen_ai.cost.total_tokens).

NOTE: this field was never part of the relay-conventions/src/consts.rs so we don't need to update it

Closes TET-1099: Stop double writing cost and update the product to query the new attribute

@vgrozdanic vgrozdanic self-assigned this Dec 10, 2025
@vgrozdanic vgrozdanic force-pushed the vg/stop-using-usage-total-cost branch from e35b63c to 4630369 Compare December 11, 2025 11:58
@vgrozdanic vgrozdanic changed the title chore(ai): Remove gen_ai_usage_total_cost attribute chore(ai): Remove gen_ai_usage_total_cost attribute and stop double writing costs Dec 11, 2025
@linear
Copy link

linear bot commented Dec 11, 2025

@vgrozdanic vgrozdanic force-pushed the vg/stop-using-usage-total-cost branch 4 times, most recently from 689425e to c806bfc Compare December 11, 2025 12:44
@vgrozdanic vgrozdanic force-pushed the vg/stop-using-usage-total-cost branch from c806bfc to 38c356a Compare December 11, 2025 12:44
@vgrozdanic vgrozdanic marked this pull request as ready for review December 11, 2025 12:58
@vgrozdanic vgrozdanic requested a review from a team as a code owner December 11, 2025 12:58
@vgrozdanic vgrozdanic added this pull request to the merge queue Dec 11, 2025
Merged via the queue into master with commit 73c1705 Dec 11, 2025
32 of 33 checks passed
@vgrozdanic vgrozdanic deleted the vg/stop-using-usage-total-cost branch December 11, 2025 13:59
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.

2 participants