Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Add counter for traced requests to Fireworks#63953

Merged
rafax merged 4 commits into
mainfrom
rg/add_traced_counter
Jul 19, 2024
Merged

Add counter for traced requests to Fireworks#63953
rafax merged 4 commits into
mainfrom
rg/add_traced_counter

Conversation

@rafax

@rafax rafax commented Jul 19, 2024

Copy link
Copy Markdown
Contributor

Adds a otel counter to measure how many traced requests we sent to Fireworks from Cody Gateway - context

Test plan

  • tested locally by sending a request without the X-Fireworks-Genie header, with the header but with value != true and with the header and value true and observing fireworks-traced-requests Prometheus metric

@cla-bot cla-bot Bot added the cla-signed label Jul 19, 2024
@rafax rafax requested review from a team and olafurpg July 19, 2024 13:25
@rafax rafax enabled auto-merge (squash) July 19, 2024 13:25

@olafurpg olafurpg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@rafax rafax merged commit 8bd9c5d into main Jul 19, 2024
@rafax rafax deleted the rg/add_traced_counter branch July 19, 2024 14:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants