fix(tests): remove stale _dd.compute_stats:1 assertion from logs integration test#1136
Merged
duncanista merged 1 commit intomainfrom Mar 27, 2026
Merged
fix(tests): remove stale _dd.compute_stats:1 assertion from logs integration test#1136duncanista merged 1 commit intomainfrom
duncanista merged 1 commit intomainfrom
Conversation
…gration test The _dd.compute_stats tag was moved from general log tags to trace-only payloads in commit 55659d9. The integration test mock still required this tag in the log body, causing the mock to never match, which triggered an infinite retry loop in Flusher::send() and hung the test indefinitely. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Fixes a hanging test_logs integration test by updating the httpmock request matcher to stop expecting a trace-only tag (_dd.compute_stats:1) in the logs payload.
Changes:
- Remove the
_dd.compute_stats:1body assertion from the logs intake mock. - Eliminate the now-unused
regexp_compute_statevariable from the test.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
duncanista
approved these changes
Mar 27, 2026
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
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.
https://datadoghq.atlassian.net/browse/SVLS-8767
Summary
_dd.compute_statswas moved from general log tags to trace-only payloads in commit55659d93test_logsintegration test mock still required_dd.compute_stats:1in the log bodyFlusher::send()retried infinitely → test hung foreverTest plan
cargo test --test logs_integration_test test_logspasses and completes in < 1s🤖 Generated with Claude Code