Adriel Perkins

Results 68 comments of Adriel Perkins

> I'm aware that more discussion is going on in other issues and CNCF Slack around direction to go here, but I'm approving as I'm fine either way. We can...

Cross posting the outcome from the SIG decision today https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/27460#issuecomment-2302537838 will update this PR to reflect.

This is now ready for merge. In future iterations I may make some small internal shifts given that we will be allowing for more signals within this singular component. This...

> Thanks for making the rename! One potential change here would be to move the `internal/scraper/githubscraper/` directory up a level to just be `internal/scraper`, since there's no need to qualify...

@mx-psi - conflicts resolved. Unrelated and unexpected failure observed in build & test workflows for the redaction processor. Fixed once #34937 is merged. All workflows are good otherwise.

> Looks like this > > > > ``` > > [✖] https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/githubreceiver/internal/scraper/githubscraper/README.md#github-limitations → Status: 404 > > ``` > > > > Is a real error though? @mx-psi it...

CC. @niwoerner I can't assign it to you because I think you've not interacted with this repository yet, but I'm going to CC you in the issue and leave it...

Since the creation timestamp isn't updated on re-runs, and there is no way to get a `run started` timestamp, then we can't do really deterministic traces from within the runner...

Has been merged, next step is semantic attributes on the spans as outlined in [this comment](https://github.com/open-telemetry/semantic-conventions/issues/1749#issuecomment-2888583995)

Next iteration is planned within the next couple weeks https://cloud-native.slack.com/archives/C0598R66XAP/p1750670554727959?thread_ts=1750336408.039169&cid=C0598R66XAP