chore(tests): Antithesis Vector end-to-end ack scenario#25571
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
84df4d7 to
c066e34
Compare
5209ae6 to
24d8e0f
Compare
c066e34 to
8ff610c
Compare
24d8e0f to
2445cb8
Compare
8ff610c to
4811fb8
Compare
2445cb8 to
5e79a45
Compare
4811fb8 to
85cfa4b
Compare
5e79a45 to
487ed7e
Compare
487ed7e to
1dbdb4e
Compare
d40777e to
7ecc846
Compare
1dbdb4e to
b2c70cd
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7ecc846666
ℹ️ 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".
7ecc846 to
b4576c4
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b4576c4a72
ℹ️ 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".
58c9222 to
d3094d8
Compare
This PR introduces a new antithesis scenario confirming that without disk buffers a single Vector instance is capable of maintaining conservation and liveness when e2e acks are enabled. This is a simplification of vector_to_vector_e2e_disk and is intended to demonstrate that e2e acks behave as expected.
b4576c4 to
f538c36
Compare
68acbd8 to
5ecfe7a
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f538c36395
ℹ️ 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".
| # debug=true keeps DWARF for /symbols. | ||
| RUN --mount=type=cache,target=/usr/local/cargo/registry \ | ||
| --mount=type=cache,target=/src/target \ | ||
| cargo build --release -p harness \ |
There was a problem hiding this comment.
Build the actual harness package
When the workload image is built, this cargo build -p harness does not select the Antithesis harness crate: cargo build --help documents -p as a package spec, and cargo pkgid -p harness in this workspace reports that no package matches, while tests/antithesis/harness/Cargo.toml names the package antithesis-harness. As a result the documented docker compose ... build path fails before the workload binaries are copied; use -p antithesis-harness like the existing disk scenario.
Useful? React with 👍 / 👎.

Summary
This PR introduces a new antithesis scenario confirming that without disk buffers a single Vector instance is capable of maintaining conservation and liveness when e2e acks are enabled. This is a simplification of vector_to_vector_e2e_disk and is intended to demonstrate that e2e acks behave as expected.
Vector configuration
How did you test this PR?
Change Type
Is this a breaking change?
Does this PR include user facing changes?
no-changeloglabel to this PR.References
Notes
@vectordotdev/vectorto reach out to us regarding this PR.pre-pushhook, please see this template.make fmtmake check-clippy(if there are failures it's possible some of them can be fixed withmake clippy-fix)make testgit merge origin masterandgit push.Cargo.lock), pleaserun
make build-licensesto regenerate the license inventory and commit the changes (if any). More details on the dd-rust-license-tool.