Pablo Baeyens
Pablo Baeyens
Yes, I think that would make sense
Current uses of `consumererror.NewPermanent`: https://github.com/search?q=repo%3Aopen-telemetry%2Fopentelemetry-collector+OR+repo%3Aopen-telemetry%2Fopentelemetry-collector-contrib+consumererror.NewPermanent+language%3AGo&type=code&p=1 Proposed approach for doing this: 1. Make it so that `IsPermanent` looks something like this, with `defaultIsPermanent` being `false`: ```go func IsPermanent(err error) bool {...
Ah! Yes, I hadn't realized that. Okay, updated my comment to use NewRetryableError and IsRetryable instead
I have not had time to look into this PR yet, I will try to do it next week. What I had in mind in general terms is a page...
Sorry, this slipped through the cracks, let me add it to my queue :+1:
> @mx-psi - just to clarify, can this can go under `gateway` in collector/deployment? I thought that could be an appropriate place for the note, but just wanted to check...
I'm supportive of this PR moving forward
I agree that a code fix is needed. Also please note that as of now FreeBSD is not a [supported platform](https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/platform-support.md), which means that we provide no guarantees that the...
A possible improvement to the situation could be to have an AGENTS.md/CLAUDE.md that requires the LLM to include an `Assisted-By` commit trailer on any commit generated by the LLM. This...