Adriel Perkins
Adriel Perkins
> Overall LGTM, albeit I'd like to have a pair of things called out: > > * On propagating other formats: we are taking for granted we will **only** want...
> Added some minor comments. > > Approving under the assumption that traceparent and tracestate format is intended to be identical to the W3C Trace-Context spec (and that we document...
> [](https://api.easycla.lfx.linuxfoundation.org/v2/repository-provider/github/sign/9522512/638591290/1383/#/?version=2) > > * [❌](https://api.easycla.lfx.linuxfoundation.org/v2/repository-provider/github/sign/9522512/638591290/1383/#/?version=2) - login: @christophe-kamphaus-jemmic / name: Christophe Kamphaus . The commit ([44a1c81](https://github.com/open-telemetry/semantic-conventions/commit/44a1c81ff731894b44361c9e7ae633ed1bd7a208), [4d8e3a9](https://github.com/open-telemetry/semantic-conventions/commit/4d8e3a9c75fa66e4a22952c98c9087fed6b2592a), [592483f](https://github.com/open-telemetry/semantic-conventions/commit/592483f1530ad57334cf9057db1734349482834b), [ef91dfa](https://github.com/open-telemetry/semantic-conventions/commit/ef91dfa616bd776ac951df1827429b027a82735b), [238c764](https://github.com/open-telemetry/semantic-conventions/commit/238c76460c4abfe536ae2645957230f6fe18e30a), [9bd4a95](https://github.com/open-telemetry/semantic-conventions/commit/9bd4a958315ba58893b88efd597e1d1fb2dac087), [8c2e8a7](https://github.com/open-telemetry/semantic-conventions/commit/8c2e8a757b7203e04c3fe63bc1a705bafb0f19ef), [f712312](https://github.com/open-telemetry/semantic-conventions/commit/f71231217f6607fb0d7a1a212b37b578ce922c9d), [700412a](https://github.com/open-telemetry/semantic-conventions/commit/700412af4b8293559d312afd44ee939b32d4374b)) is not authorized under a...
Re: > > Review statistics provided by Github (Pulse / Insights). > > Is there overlap with the metrics defined here? .... I'm not a huge fan of GitHub's insights,...
> I have tried to summarize the discussion we had last Monday in the Semantic Convention SIG as best I can here below: > > ### Summary of Semantic Convention...
@yangskyboxlabs - #258 was geared for this and hasn't stalled as much as I haven't ave been other in progress things taking immediate priority and cognitive load on my part...
This sounds like it would be a good use case for octo-sts @trask
> After the rename, the "provider" part in the name seems a bit (more) awkward. (I think it was me who originally came up with this 🤪). I'm not sure...
@andrzej-stencel - change has been made. For whoever merges, check-links will fail as it points to `main`. It will auto resolve once this PR is merged.
> Out of ignorance, is there any potential of this receiver being able to support logs or traces in the future? (If so, we may not want to have `metrics`...