Michael Safyan
Michael Safyan
That would seem reasonable to me.
Started draft of pull request here: https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3122 It's still a work in progress, but I figured a draft would be good in that it will make it easier to follow...
I still need to address some style and polish issues, but the PR is essentially there now in: https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3122
@adriangb thanks for raising those issues. I agree that this is not GenAI specific. With respect to pre-signed URLs, I do think there is value in being able to actually...
@samuelcolvin thanks for those suggestions. Deferring the upload until sampling makes sense. I had assumed, perhaps incorrectly, that this code wouldn't effectively get called unless the span was being sampled....
Thanks, @TylerHelmuth . Pardon me, but I'm fairly new to this repo (and most of my experience is closed source)... What is the process for this discussion / consideration? What...
Thanks for the support of the general direction [lmolkova](https://github.com/lmolkova)! ## Event body > I think we need a solution for uploading an event body (or a specific jpath in the...
> Any solution that can avoid putting large values in attributes should be avoided Agreed. This will help us deal with "legacy span attributes that may contain large values", by...
Just checking back in ... what are the next steps here?
It looks like the next step is to actually propose the update. There is a pending PR in https://github.com/open-telemetry/semantic-conventions/pull/1521 It was brought up that VCS already uses `.ref.`, so we...