Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Do not add tracing CLI args to opa-openshift container #1081

Merged

Conversation

andreasgerstmayr
Copy link
Collaborator

The opa-openshift container does not support
--internal.tracing.endpoint and remains in a crashloop if

spec:
  observability:
    tracing:
      jaeger_agent_endpoint: ...
      sampling_fraction: "1"

is enabled.

The opa-openshift container does not support
`--internal.tracing.endpoint` and remains in a crashloop if
```
spec:
  observability:
    tracing:
      jaeger_agent_endpoint: ...
      sampling_fraction: "1"
```
is enabled.

Signed-off-by: Andreas Gerstmayr <agerstmayr@redhat.com>
@pavolloffay
Copy link
Collaborator

Is there anything we need to do for the monolithic deployment?

@andreasgerstmayr
Copy link
Collaborator Author

Is there anything we need to do for the monolithic deployment?

The monolithic deployment only supports ingestion via OTLP, but the internal Tempo instrumentation (currently) only supports exporting traces in Jaeger format. Therefore spec.observability.tracing isn't supported at the moment for monolithic.

The next Tempo version supports OTLP, I plan to add support for the monolithic deployment then.

@andreasgerstmayr andreasgerstmayr merged commit 474e106 into grafana:main Nov 5, 2024
11 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants