Skip to content

Reduce muzzle planning overhead and support configuration caching - #12778

Closed
sarahchen6 wants to merge 2 commits into
masterfrom
sarahchen6/muzzle-improvements
Closed

sarahchen6 wants to merge 2 commits into
masterfrom
sarahchen6/muzzle-improvements

Conversation

@sarahchen6

Copy link
Copy Markdown
Contributor

What Does This Do

  • Centralize muzzle planning after all projects are evaluated, sharing one Maven resolver session across modules.
  • Skip version resolution and check registration for compile-only requests and modules excluded from CI aggregation.
  • When runMuzzle is invoked for a specific project, include muzzle checks for its subprojects.
  • Make task inputs explicit and revalidate selected dependency versions before reusing a cached task graph.

Motivation

Against master, the median configuration + graph time fell 157->101 ms (36%) for 30 modules and 116->44 ms (62%) for one selected module. Total registered checks fell 120->4 while preserving the selected module’s same 20-task graph.

These results come from Codex measuring four warm runs per variant in alternating order using offline Gradle 9.8, four workers, a shared fake dependency in a local Maven repository, a no-op scanner, and configuration caching disabled.

Additional Notes

Contributor Checklist

Jira ticket: [PROJ-IDENT]

@sarahchen6 sarahchen6 added type: feature Enhancements and improvements comp: tooling Build & Tooling tag: ai generated Largely based on code generated by an AI or LLM labels Oct 7, 2026
@dd-octo-sts

dd-octo-sts Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

🟢 Java Benchmark SLOs — All performance SLOs passed

Suite Status
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
Scenario Candidate master Δ (95% CI of mean)
startup:insecure-bank:iast:Agent 13.95 s 14.00 s [-1.0%; +0.3%] (no difference)
startup:insecure-bank:tracing:Agent 12.81 s 12.90 s [-1.3%; -0.1%] (maybe better)
startup:petclinic:appsec:Agent 17.14 s 16.88 s [+0.5%; +2.6%] (maybe worse)
startup:petclinic:iast:Agent 16.92 s 16.99 s [-1.2%; +0.3%] (no difference)
startup:petclinic:profiling:Agent 16.64 s 16.08 s [-0.9%; +7.8%] (no difference)
startup:petclinic:sca:Agent 17.14 s 16.95 s [+0.2%; +2.1%] (maybe worse)
startup:petclinic:tracing:Agent 16.16 s 16.10 s [-0.5%; +1.2%] (no difference)

Commit: 84c5a27e · 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.

@datadog-official

This comment has been minimized.

Base automatically changed from bdu/muzzle-report-aggregation to master October 8, 2026 14:15
@sarahchen6
sarahchen6 force-pushed the sarahchen6/muzzle-improvements branch from 4fe067d to 84c5a27 Compare October 8, 2026 15:34
@sarahchen6

Copy link
Copy Markdown
Contributor Author

Split into #12800, #12801, and #12804

@sarahchen6 sarahchen6 closed this Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: tooling Build & Tooling tag: ai generated Largely based on code generated by an AI or LLM type: feature Enhancements and improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant