-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use otel collector #114
Use otel collector #114
Conversation
…for language instrumentations.
@johnbley could you give some more explanation of what this change is doing? Are the steps to set |
@dude0001 they are used by the collector that is included in the layer, instead of by the individual SDKs, which is why its |
Thank you @tsloughter for the further clarification. Is there a complete example of how to consume this layer after this change? I'm trying to upgrade to latest to get some other fixes. However, on latest I'm seeing 401 Unuauthorized errors assumingly because the |
Just noticed my response yesterday didn't send somehow, sorry about that. @dude0001 what version of the layers are you using? Are you setting any env variables besides realm and access token? |
@tsloughter I had to step away from this for a bit, but looking at it again @tsloughter. I am using |
Prepare to use the otel collector rather than having each language instrumentation post directly to ingest. This should reduce latency for customers' apps/functions.
The splunk-otel-lambda side of this work involves modifying the scripts to
unset SPLUNK_REALM
and friends so they fall back to the otel default behavior of talking OTLP to localhost. Along the way, I noticed and fixed a problem where we had forked some upstream otel-lambda wrapper scripts rather than simply applying small edits to them during our build process.NB: based on past experience, this process will likely take a few cycles of CI to iron out tiny script problems.