Skip to content

Conversation

@TheoBrigitte
Copy link
Contributor

What is this feature?

This PR improve the Tempo datasource configuration to enable gRPC over TLS based on the datasource url scheme (http or https) regardless of the auth mechanism.

Why do we need this feature?

It is currently confusing that the toggle to enable gRPC over TLS is based on the presence of the basic auth setting.

Basic auth and TLS serve different purposes, each setting should work independently: basic auth should work regardless of TLS and TLS regardless of basic auth.

Who is this feature for?

Grafana Admin user managing datasources.

Before

Basic auth was required in order for gRPC streaming capabilities to work over TLS

image
After

No need for basic auth settings anymore in order for gRPC streaming capabilities to work over TLS

image

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

@TheoBrigitte TheoBrigitte requested review from a team as code owners September 29, 2025 17:31
@github-actions github-actions bot added this to the 12.3.x milestone Sep 29, 2025
@zoltanbedi zoltanbedi added the pr/external This PR is from external contributor label Sep 29, 2025
@adrapereira adrapereira removed the request for review from a team October 6, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/backend datasource/Tempo pr/external This PR is from external contributor

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants