Riccardo Magliocchetti

Results 346 comments of Riccardo Magliocchetti

On adding only the `process.runtime` attributes I see that they are *recommended* in the spec: https://opentelemetry.io/docs/specs/semconv/resource/process/#process-runtimes *recommended* is explained there https://opentelemetry.io/docs/specs/semconv/general/attribute-requirement-level/#recommended

I guess the type is int because the sdk is using `time_ns` output as value by default.

@adriangb Please take a look at the failing tests

ICYMI the added test is red

> Before the last SIG meeting people asked what was missing for the release. I pointed out this PR. Correct, I did ask in that SIG > My request was...

> @xrmx @lzchen looks like another release was cut without including this. What do I need to do to get this merged? It's been months. @adriangb In case you missed...

Thanks for reporting, do this reproduce with a minimal application or are you doing something specific with headers?

Haven't looked at the code but the script should add only PR that has been merged to master branch, currently all the PR merged are listed.

@stsewd that switch checks for issues with merged PR not merged to *master* branch.

Why is this strange? If this for the multiple spans on POST handling I think we already have an issue for that.