Skip to content

[Repo Assist] Better OTel and use Pull Diagnostics #1486

Open
Krzysztof-Cieslak wants to merge 4 commits intomainfrom
rebase-1431
Open

[Repo Assist] Better OTel and use Pull Diagnostics #1486
Krzysztof-Cieslak wants to merge 4 commits intomainfrom
rebase-1431

Conversation

@Krzysztof-Cieslak
Copy link
Member

@Krzysztof-Cieslak Krzysztof-Cieslak commented Feb 27, 2026

Rebased #1341

TheAngryByrd and others added 4 commits February 27, 2026 12:42
… support

- Rebase PR commits onto current main
- Fix formatting in FsOpenTelemetry.fs and AdaptiveServerState.fsi (fantomas)
- Move OTel trace provider init to module level so it works with both
  `dotnet run` and `dotnet test` (YoloDev.Expecto.TestSdk)
- CI mode: failed-test OTLP file exporter initialised at module load,
  flushed via AppDomain.ProcessExit (no longer depends on main() being
  called by the test runner)
- Local dev: use standard OTEL_EXPORTER_OTLP_ENDPOINT env var instead
  of hard-coded localhost:4317
- Restore `dotnet test` in CI workflow with AltCover, GitHubActions
  logger and --blame-hang timeout; keep FAILED_TRACES_DIR + artifact
  upload for failed trace inspection
- Upload artifact path uses ${{ github.workspace }}/failed_traces for
  correct absolute path resolution

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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