Skip to content

Better OTel and use Pull Diagnostics for tests#1341

Draft
TheAngryByrd wants to merge 3 commits intoionide:mainfrom
TheAngryByrd:better-otel-tests
Draft

Better OTel and use Pull Diagnostics for tests#1341
TheAngryByrd wants to merge 3 commits intoionide:mainfrom
TheAngryByrd:better-otel-tests

Conversation

@TheAngryByrd
Copy link
Member

No description provided.

jaeger:
profiles:
- jaeger
image: jaegertracing/all-in-one
Copy link
Contributor

Choose a reason for hiding this comment

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

OMG travesty - not using the Aspire Dashboard? 😱


let! (server, events) = serverInitialize path analyzerEnabledConfig state
let scriptPath = Path.Combine(path, "Script.fsx")
do! Async.Sleep(TimeSpan.FromSeconds 5.)
Copy link
Contributor

Choose a reason for hiding this comment

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

ooh it's very promising to remove the random sleeps

@github-actions
Copy link
Contributor

🤖 This is an automated response from Repo Assist.

Hi @TheAngryByrd! This draft PR ("Better OTel and use Pull Diagnostics for tests") has been quiet since December 2025 — just checking in to see if it's still in progress.

Better OTel tracing and pull-based diagnostics for tests would be a great improvement. If you'd like to continue it, need a review of the current state, or would like help rebasing against main, feel free to reach out!

If the approach has changed or there are blockers, it'd be helpful to know so maintainers can plan accordingly. Thanks for all the work you've put in here! 🙂

Generated by Repo Assist

To install this workflow, run gh aw add githubnext/agentics/workflows/repo-assist.md@b87234850bf9664d198f28a02df0f937d0447295. View source at https://github.com/githubnext/agentics/tree/b87234850bf9664d198f28a02df0f937d0447295/workflows/repo-assist.md.

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