Skip to content

Conversation

@Mpdreamz
Copy link
Member

@Mpdreamz Mpdreamz commented Dec 4, 2025

No description provided.

@Mpdreamz Mpdreamz requested a review from a team as a code owner December 4, 2025 14:32
@Mpdreamz Mpdreamz requested a review from reakaleek December 4, 2025 14:32
@Mpdreamz Mpdreamz added the fix label Dec 4, 2025
@Mpdreamz Mpdreamz self-assigned this Dec 4, 2025
Copy link
Member

@reakaleek reakaleek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm afraid, this could break the OTel setup on our API.

@Mpdreamz
Copy link
Member Author

Mpdreamz commented Dec 4, 2025

Both application entries call their own OpenTelemetry bootstrap explicitly now:

_ = builder.AddDocsApiOpenTelemetry();

.AddOpenTelemetryDefaults();

This is now longer auto wirted by ServiceDefaults() project.

I do think we should create a single funnel but will follow up with that at some point.

@Mpdreamz Mpdreamz enabled auto-merge (squash) December 4, 2025 14:43
@Mpdreamz Mpdreamz merged commit 9e8b325 into main Dec 4, 2025
28 checks passed
@Mpdreamz Mpdreamz deleted the fix/search-yml branch December 4, 2025 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants