Skip to content

add support for o11y export - #86

Merged
darshanime merged 8 commits into
mainfrom
o11y-export
Sep 1, 2026
Merged

add support for o11y export#86
darshanime merged 8 commits into
mainfrom
o11y-export

Conversation

@darshanime

Copy link
Copy Markdown
Contributor

No description provided.

@guptaankit015 guptaankit015 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One minor doubt, Rest LGTM

Comment thread pkg/manager/hooks.go
Comment on lines +34 to +36
TELEMETRY_DRAIN_HOOK,
CLAUDE_OUTPUTS_UPLOAD_HOOK,
CLEANUP_CALLBACK_HOOK,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

assuming first 2 not timetaking/infinite-retry, should have a small timeout... cleanup has infinite retry and should not be blocked for too long

Comment on lines +11 to +23
runs-on: warp-ubuntu-latest-x64-2x
steps:
- uses: actions/checkout@v4

- uses: actions/setup-go@v5
with:
go-version: "1.25"

- name: Vet
run: go vet ./...

- name: Test
run: go test -race ./...
@darshanime
darshanime merged commit 8de71af into main Sep 1, 2026
7 checks passed
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.

3 participants