-
Notifications
You must be signed in to change notification settings - Fork 183
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
feat(ot-operator): add opentelemetry-operator for tracing #2172
Conversation
fb3818b
to
ab56462
Compare
tests/integration/main_test.go
Outdated
// Helm chart installation extra arguments list | ||
helmExtraArgs := []string{"--wait"} |
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.
These are not really helm extra args but helm install extra args.
I believe we can actually put this into the SetHelmOptionsTestOpt
and not expose here 🤔
# Prevent snowball effect by filtering out receiver mock logs | ||
fluent-bit: | ||
config: | ||
filters: | | ||
[FILTER] | ||
Name grep | ||
Match containers.var.log.containers.receiver-mock* | ||
Exclude log .* |
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.
I believe that given the logs are disabled now, this is not needed anymore.
Co-authored-by: Dominik Rosiek <58699848+sumo-drosiek@users.noreply.github.com>
# Conflicts: # CHANGELOG.md # tests/integration/go.mod # tests/integration/go.sum
# Conflicts: # tests/integration/go.mod # tests/integration/go.sum
Description
OpenTelemetry-Operator functionalities in addition with automatically created
Instrumentation
custom resource in specified namespaces provided byopentelemetry-operator.manager.env.WATCH_NAMESPACE
.cert-manager
is not required to be installed. Needed certificates are generated during chart deployment.Checklist
Testing performed