[golang] Add crashtracking parametric coverage#7312
Draft
kakkoyun wants to merge 3 commits into
Draft
Conversation
Go crashtracking uploads Error Tracking reports through the Agent EVP proxy instead of telemetry logs. Add a Go-specific assertion path and enable the parametric crash tests against the next Go tracer dev version so the unmerged dd-trace-go implementation can be tested from this branch.
Contributor
|
|
|
dd-trace-go main and the crashtracker PR branch report v2.11.0-dev, while the latest published release is v2.9.1. Gate the new Go crashtracking parametric coverage on that dev version instead of the stale v2.5.0-dev value.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/trace/crashendpoint that startscrashtrackerand crashes the processv2.5.0-devContext
This is for testing unmerged dd-trace-go crashtracker work:
The Go package is not on dd-trace-go main yet, so this PR is draft until those changes are available to system-tests CI.
Test
ruff check tests/parametric/test_crashtracking.pybash -n utils/build/docker/golang/install_ddtrace.shshellcheck utils/build/docker/golang/install_ddtrace.shshfmt -d utils/build/docker/golang/install_ddtrace.shgofmt -l utils/build/docker/golang/parametric/main.gogithub.com/DataDog/dd-trace-go/v2@feat/crashtracker-api:go mod tidy && go test ./...