Skip to content

Commit 2eca999

Browse files
committed
scrub CI vis spans
1 parent d98c88b commit 2eca999

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

tracer/test/Datadog.Trace.TestHelpers.SharedSource/VerifyHelper.cs

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -243,10 +243,15 @@ public static VerifySettings AddSimpleScrubber(this VerifySettings settings, str
243243
// remove propagated tags because their positions in the snapshots are not stable
244244
// with our span ordering. correct position (first span in every trace chunk) is covered by other tests.
245245
?.Where(kvp => !kvp.Key.StartsWith(TagPropagation.PropagatedTagPrefix, StringComparison.Ordinal))
246-
// We must ignore both `_dd.git.repository_url` and `_dd.git.commit.sha` because we are only setting it on the first span of a trace
247-
// no matter what. That means we have unstable snapshot results.
248-
// Also ignoring `_dd.parent_id` since we test specific headers combinations which check for the value, hence why not adding it to the snapshots
249-
.Where(kvp => kvp.Key != Tags.GitRepositoryUrl && kvp.Key != Tags.GitCommitSha && kvp.Key != Tags.LastParentId)
246+
.Where(kvp =>
247+
// We must ignore both `_dd.git.repository_url` and `_dd.git.commit.sha` because we are only setting it on the first span of a trace
248+
// no matter what. That means we have unstable snapshot results.
249+
kvp.Key != Tags.GitRepositoryUrl
250+
&& kvp.Key != Tags.GitCommitSha
251+
// Also ignoring `_dd.parent_id` since we test specific headers combinations which check for the value, hence why not adding it to the snapshots
252+
&& kvp.Key != Tags.LastParentId
253+
// same as git related tags above, process tags are only added to the first span of each payload, which makes snapshots unstable.
254+
&& kvp.Key != Tags.ProcessTags)
250255
.Select(
251256
kvp => kvp.Key switch
252257
{

0 commit comments

Comments
 (0)