Skip to content

Add test reporting for Karate v2 - #11923

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 3 commits into
masterfrom
daniel.mohedano/karate-v2-instrumentation
Jul 14, 2026
Merged

Add test reporting for Karate v2#11923
gh-worker-dd-mergequeue-cf854d[bot] merged 3 commits into
masterfrom
daniel.mohedano/karate-v2-instrumentation

Conversation

@daniel-mohedano

@daniel-mohedanodaniel-mohedano commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

What Does This Do

  • Adds a new karate-2.0 instrumentation module to implement test visibility for Karate 2.0+.
    • The implementation follows the same approach used for the original instrumentation in the karate-1.0 module.
    • Also implements skip-based features (ITR and disabled tests), as part of the tracing listener.

Motivation

Karate v2 is a complete ground-up rewrite of the framework. Because of this, the original karate-1.0 module cannot instrument it.

Additional Notes

A following PR will contain the changes regarding execution instrumentation for Karate v2.

Contributor Checklist

Jira ticket: SDTEST-3816

@daniel-mohedanodaniel-mohedano added type: feature Enhancements and improvements comp: ci visibility Continuous Integration Visibility labels Jul 13, 2026
@daniel-mohedano
daniel-mohedanoforce-pushed the daniel.mohedano/karate-v2-instrumentation branch from 4d2d28e to 1d0e84dCompareJuly 13, 2026 08:06
@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - sbt-scalatest

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent57.4955.43$\color{red}{\blacktriangle}$ +2.0654.33$\color{red}{\blacktriangle}$ +3.1676/242
agentEvpProxy55.39n/an/an/an/a-

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - reactive-streams-jvm

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent21.6222.09$\color{green}{\blacktriangledown}$ -0.4721.65$\color{green}{\blacktriangledown}$ -0.0339/126
agentless18.5518.82$\color{green}{\blacktriangledown}$ -0.2718.82$\color{green}{\blacktriangledown}$ -0.2738/123
agentlessCodeCoverage19.8320.39$\color{green}{\blacktriangledown}$ -0.5619.99$\color{green}{\blacktriangledown}$ -0.1638/122
agentlessLineCoverage30.2829.82$\color{red}{\blacktriangle}$ +0.4629.82$\color{red}{\blacktriangle}$ +0.4637/121

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - nebula-release-plugin

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent37.1537.15$\color{gray}{\bullet}$ 0.0036.42$\color{red}{\blacktriangle}$ +0.7338/119
agentless38.4336.42$\color{red}{\blacktriangle}$ +2.0136.42$\color{red}{\blacktriangle}$ +2.0138/119
agentlessCodeCoverage45.1045.38$\color{green}{\blacktriangledown}$ -0.2844.48$\color{red}{\blacktriangle}$ +0.6238/119
agentlessLineCoverage75.0174.82$\color{red}{\blacktriangle}$ +0.1974.82$\color{red}{\blacktriangle}$ +0.1937/118

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - pass4s

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent9.649.35$\color{red}{\blacktriangle}$ +0.2910.33$\color{green}{\blacktriangledown}$ -0.6938/118
agentless10.3310.33$\color{gray}{\bullet}$ 0.0010.13$\color{red}{\blacktriangle}$ +0.2038/118
agentlessCodeCoverage16.7517.03$\color{green}{\blacktriangledown}$ -0.2817.38$\color{green}{\blacktriangledown}$ -0.6337/116

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - heliboard

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent10.889.54$\color{red}{\blacktriangle}$ +1.349.54$\color{red}{\blacktriangle}$ +1.3436/40

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - sonar-kotlin

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent13.2412.87$\color{red}{\blacktriangle}$ +0.3713.13$\color{red}{\blacktriangle}$ +0.1134/117
agentless12.6511.88$\color{red}{\blacktriangle}$ +0.7712.12$\color{red}{\blacktriangle}$ +0.5334/116
agentlessCodeCoverage15.6715.11$\color{red}{\blacktriangle}$ +0.5615.11$\color{red}{\blacktriangle}$ +0.5634/116
agentlessLineCoverage19.3719.20$\color{red}{\blacktriangle}$ +0.1719.20$\color{red}{\blacktriangle}$ +0.1733/116

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@datadog-official

This comment has been minimized.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - jolokia

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent92.2795.12$\color{green}{\blacktriangledown}$ -2.8593.23$\color{green}{\blacktriangledown}$ -0.9641/128
agentless88.3489.58$\color{green}{\blacktriangledown}$ -1.2489.58$\color{green}{\blacktriangledown}$ -1.2439/124
agentlessCodeCoverage98.3099.00$\color{green}{\blacktriangledown}$ -0.7099.00$\color{green}{\blacktriangledown}$ -0.7040/123
agentlessLineCoverage99.4499.00$\color{red}{\blacktriangle}$ +0.4499.00$\color{red}{\blacktriangle}$ +0.4438/121

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - netflix-zuul

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent86.6987.80$\color{green}{\blacktriangledown}$ -1.1187.80$\color{green}{\blacktriangledown}$ -1.1139/126
agentless80.9081.05$\color{green}{\blacktriangledown}$ -0.1581.05$\color{green}{\blacktriangledown}$ -0.1539/125
agentlessCodeCoverage96.4897.04$\color{green}{\blacktriangledown}$ -0.5695.12$\color{red}{\blacktriangle}$ +1.3639/123
agentlessLineCoverage114.34111.62$\color{red}{\blacktriangle}$ +2.72111.62$\color{red}{\blacktriangle}$ +2.7238/122

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - okhttp

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent20.5219.20$\color{red}{\blacktriangle}$ +1.3219.20$\color{red}{\blacktriangle}$ +1.3240/130
agentless18.0119.20$\color{green}{\blacktriangledown}$ -1.1919.20$\color{green}{\blacktriangledown}$ -1.1939/127
agentlessCodeCoverage22.8622.99$\color{green}{\blacktriangledown}$ -0.1322.09$\color{red}{\blacktriangle}$ +0.7738/125
agentlessLineCoverage44.0944.48$\color{green}{\blacktriangledown}$ -0.3944.48$\color{green}{\blacktriangledown}$ -0.3937/131

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - spring_boot

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent16.4316.36$\color{red}{\blacktriangle}$ +0.0716.04$\color{red}{\blacktriangle}$ +0.3938/117
agentless9.689.73$\color{green}{\blacktriangledown}$ -0.059.73$\color{green}{\blacktriangledown}$ -0.0538/118
agentlessCodeCoverage13.6513.67$\color{green}{\blacktriangledown}$ -0.0213.40$\color{red}{\blacktriangle}$ +0.2539/117
agentlessLineCoverage32.8932.95$\color{green}{\blacktriangledown}$ -0.0632.95$\color{green}{\blacktriangledown}$ -0.0637/116

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@dd-octo-sts

dd-octo-stsBot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

🟢 Java Benchmark SLOs — All performance SLOs passed

SuiteStatus
Startup🟢 pass

SLO thresholds are defined here based on automatically generated metrics. A warning is raised when results are within 5% of the threshold.

PR vs. master results
ScenarioCandidatemasterΔ (95% CI of mean)
startup:insecure-bank:iast:Agent14.75 s14.60 s[+0.2%; +1.9%] (maybe worse)
startup:insecure-bank:tracing:Agent13.55 s13.63 s[-1.3%; +0.2%] (no difference)
startup:petclinic:appsec:Agent16.86 s16.71 s[-0.1%; +1.9%] (no difference)
startup:petclinic:iast:Agent16.92 s16.83 s[-0.2%; +1.3%] (no difference)
startup:petclinic:profiling:Agent16.55 s16.75 s[-2.4%; +0.0%] (no difference)
startup:petclinic:sca:Agent16.89 s16.81 s[-0.7%; +1.8%] (no difference)
startup:petclinic:tracing:Agent15.63 s15.96 s[-6.4%; +2.2%] (no difference)

Commit:49a54e8d · CI Pipeline · Benchmarking Platform UI


Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion.

@daniel-mohedano
daniel-mohedanoforce-pushed the daniel.mohedano/karate-v2-instrumentation branch from 1d0e84d to e7ddda7CompareJuly 13, 2026 08:47
@daniel-mohedanodaniel-mohedano added the tag: override groovy enforcement Override the "Enforce Groovy Migration" check label Jul 13, 2026
@cit-pr-commenter-54b7da

cit-pr-commenter-54b7daBot commented Jul 13, 2026

Copy link
Copy Markdown

Test Environment - sonar-java

Job Status: 🟢 success

ScenarioThis PR (%)7d medianΔ 7d30d medianΔ 30druns (7d/30d)
agent-8.8612.37$\color{green}{\blacktriangledown}$ -21.2313.67$\color{green}{\blacktriangledown}$ -22.5336/123
agentless18.7913.40$\color{red}{\blacktriangle}$ +5.3916.04$\color{red}{\blacktriangle}$ +2.7536/122
agentlessCodeCoverage69.0277.88$\color{green}{\blacktriangledown}$ -8.8686.07$\color{green}{\blacktriangledown}$ -17.0536/122
agentlessLineCoverage139.85130.99$\color{red}{\blacktriangle}$ +8.86136.34$\color{red}{\blacktriangle}$ +3.5135/121

Baseline: median of @test.tracer_overhead on main (gitlab) over the last 7/30 days, per OSS project & scenario. Δ = this PR − baseline median; red ▲ = more overhead, green ▽ = less overhead than baseline.

@daniel-mohedano
daniel-mohedanoforce-pushed the daniel.mohedano/karate-v2-instrumentation branch from 36148cd to 294d720CompareJuly 13, 2026 10:22
@daniel-mohedano

Copy link
Copy Markdown
ContributorAuthor

@codex review

@chatgpt-codex-connectorchatgpt-codex-connectorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:2599ffbfcf

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@daniel-mohedano
daniel-mohedanoforce-pushed the daniel.mohedano/karate-v2-instrumentation branch from 2599ffb to cbdc3d4CompareJuly 13, 2026 12:25
@daniel-mohedanodaniel-mohedano added the tag: ai generated Largely based on code generated by an AI or LLM label Jul 13, 2026
@daniel-mohedano
daniel-mohedanoforce-pushed the daniel.mohedano/karate-v2-instrumentation branch from cbdc3d4 to e459f4bCompareJuly 13, 2026 12:32
@daniel-mohedano

Copy link
Copy Markdown
ContributorAuthor

@codex review

@chatgpt-codex-connectorchatgpt-codex-connectorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:e459f4bc8e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@daniel-mohedano
daniel-mohedano marked this pull request as ready for review July 13, 2026 14:36
@daniel-mohedano
daniel-mohedano requested review from a team as code ownersJuly 13, 2026 14:36
@daniel-mohedano
daniel-mohedano requested review from dougqh and vandonr and removed request for a teamJuly 13, 2026 14:36
@daniel-mohedanodaniel-mohedano changed the title Implement test visibility for Karate v2Add test reporting for Karate v2Jul 13, 2026

@datadog-officialdatadog-officialBot 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.

Datadog Autotest: PASS

More details

The karate-2.0 instrumentation is behaviourally correct for its stated scope (basic test reporting + ITR skipping). The most non-obvious risk — that SCENARIO_EXIT fires even when the listener vetoes SCENARIO_ENTER — was confirmed safe: Karate v2 populates the result with skipped-step entries, so afterScenario does not spuriously call onTestSkip, and the unconditional onTestFinish call is a no-op because onTestIgnore already removed the test from inProgressTests. Tag names are stored without the @ prefix, confirming the datadog_itr_unskippable guard works correctly.

Was this helpful? React 👍 or 👎

📊 Validated against 4 scenarios · Open Bits AI session

🤖 Datadog Autotest · Commit e459f4b · What is Autotest? · Any feedback? Reach out in #autotest

@daniel-mohedano

Copy link
Copy Markdown
ContributorAuthor

/merge

@gh-worker-devflow-routing-ef8351

gh-worker-devflow-routing-ef8351Bot commented Jul 14, 2026

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-07-14 15:12:08 UTC ℹ️ Start processing command /merge


2026-07-14 15:12:14 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 2h (p90).


2026-07-14 16:48:08 UTC ℹ️ MergeQueue: This merge request was merged

@gh-worker-dd-mergequeue-cf854d
gh-worker-dd-mergequeue-cf854dBot merged commit 658a3e4 into masterJul 14, 2026
584 checks passed
@gh-worker-dd-mergequeue-cf854d
gh-worker-dd-mergequeue-cf854dBot deleted the daniel.mohedano/karate-v2-instrumentation branch July 14, 2026 16:48
@github-actionsgithub-actionsBot added this to the 1.65.0 milestone Jul 14, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: ci visibilityContinuous Integration Visibilitytag: ai generatedLargely based on code generated by an AI or LLMtag: override groovy enforcementOverride the "Enforce Groovy Migration" checktype: featureEnhancements and improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@daniel-mohedano@dougqh@gnufede