Skip to content

Override WithOtlpExporter method for SurrealDB hosting#1117

Merged
aaronpowell merged 1 commit into
CommunityToolkit:mainfrom
Odonno:feat/override-otlp-export-surreal
Jan 29, 2026
Merged

Override WithOtlpExporter method for SurrealDB hosting#1117
aaronpowell merged 1 commit into
CommunityToolkit:mainfrom
Odonno:feat/override-otlp-export-surreal

Conversation

@Odonno

@Odonno Odonno commented Jan 25, 2026

Copy link
Copy Markdown
Contributor

SurrealDB server requires variable SURREAL_TELEMETRY_PROVIDER to be set to otlp to enable exporters.
This PR overrides WithOtlpExporter method to ensures this variable is set.

PR Checklist

  • Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
  • Based off latest main branch of toolkit
  • PR doesn't include merge commits (always rebase on top of our main, if needed)
  • Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Contains NO breaking changes
  • Every new API (including internal ones) has full XML docs
  • Code follows all style conventions

Other information

Can't get the test to work with:

var config = await surrealServer.Resource.GetEnvironmentVariablesAsync();

bool hasValue = config.TryGetValue("SURREAL_TELEMETRY_PROVIDER", out var value);

Assert.True(hasValue);
Assert.Equal("otlp", value);

Anyway, I found a workaround.

@aaronpowell aaronpowell merged commit 1d5117a into CommunityToolkit:main Jan 29, 2026
122 checks passed
@Odonno Odonno deleted the feat/override-otlp-export-surreal branch January 29, 2026 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants