chore(antithesis): Antithesis test harness and workload#1767
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
Binary Size Analysis (Agent Data Plane)Baseline: 1bd1613 · Comparison: 476a5b6 · diff ✅ Binary size difference within thresholdChanges by Module
Detailed Symbol Changes |
Regression Detector (Agent Data Plane)Run ID: Optimization Goals: ✅ No significant changes detectedFine details of change detection per experiment (35)Experiments configured
Bounds Checks: ✅ Passed (5)
ExplanationA change is flagged as a regression when |Δ mean %| > 5.00% in the regressing direction for its optimization goal AND SMP marks the experiment as a regression ( |
25934ae to
330a22f
Compare
330a22f to
f8bab09
Compare
f8bab09 to
d048536
Compare
8b14d24 to
29f9642
Compare
d048536 to
e4defed
Compare
29f9642 to
1de5717
Compare
9994924 to
e46f656
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e46f6562d6
ℹ️ 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".
e46f656 to
fb9e655
Compare
fb9e655 to
4d533a8
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fb9e655158
ℹ️ 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".
ac323be to
6b922f8
Compare
Adds an Antithesis test harness for agent-data-plane under `test/antithesis/`. It brings ADP up in a minimal, runnable topology and gives Antithesis a workload to drive: - An instrumented ADP image (LLVM SanitizerCoverage), gated behind a new, off-by-default `antithesis` cargo feature, with DWARF symbols exposed under `/symbols`. - A mock `datadog-intake` and a `millstone`-based workload (the `adp-antithesis-workload` crate: a high-cardinality DogStatsD driver plus an end-to-end delivery verifier). - `docker-compose.yaml`, a standalone ADP config with a self-signed IPC cert, `setup_complete` wiring, and a `.dockerignore` re-include so `test/antithesis/` reaches the Docker build context. The `antithesis` feature and SDK are net-new and off by default, so production builds are unchanged. - [ ] Bug fix - [ ] New feature - [x] Non-functional (chore, refactoring, docs) - [ ] Performance - `docker compose -f test/antithesis/config/docker-compose.yaml build` builds all three images (amd64; the in-build instrumentation symbol checks pass). - `snouty validate test/antithesis/config` brings the stack up healthy, observes `setup_complete`, and discovers the workload test commands. - `cargo check` (default features) is unaffected; `cargo fmt --check` is clean. CI notes (please action before merge): - This adds optional deps to `Cargo.lock`, so `check-licenses` (`make check-licenses`) will fail until `LICENSE-3rdparty.csv` is regenerated (`make license`). - Worth confirming `check-deny` and `check-unused-deps` are happy with the feature-gated `antithesis_sdk` / `antithesis-instrumentation` deps. - The research and property catalog this harness targets live in `test/antithesis/scratchbook/` (the next PR in this stack). - Antithesis Rust SDK and coverage-instrumentation documentation.
6b922f8 to
476a5b6
Compare

Summary
Adds an Antithesis test harness for agent-data-plane under
test/antithesis/. It brings ADP up in a minimal, runnable topology and gives Antithesis a workload to drive. Right now I've mostly yanked things from millstone. I will expand on this in follow-on PRs.The new
antithesisfeature and SDK are not enabled by default, so production builds are unchanged.Change Type
How did you test this PR?
I fired off a job to our antithesis tenant cluster.
References
N/A