-
Notifications
You must be signed in to change notification settings - Fork 24
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
adding otel capabilities #273
Conversation
Is there a place where I can leave instructions for how to deploy jaeger and opentelemetry-collector properly for SpinKube? |
I'd add it here https://github.com/spinkube/documentation/tree/main/content/en/docs/topics |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly lgtm, blocking on the discussion around optional/required, not the env cleanup
Following up on this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs a rebase but LGTM
Signed-off-by: Andrew Steurer <[email protected]>
Signed-off-by: Andrew Steurer <[email protected]>
With this PR, a user will be able to define certain OTel parameters (which we pre-approve) that will allow them to view trace information via opentelemetry-collector and jaeger.