Skip to content

stress test: Add the ability to trace all operations - #797

Merged
Yuval-Ariel merged 2 commits into
mainfrom
yuval/stress-test-add-trace
Jan 3, 2024
Merged

stress test: Add the ability to trace all operations#797
Yuval-Ariel merged 2 commits into
mainfrom
yuval/stress-test-add-trace

Conversation

@Yuval-Ariel

Copy link
Copy Markdown
Contributor

Controlled by flag trace_ops (on by default).
Trace files will be written to the expected values dir

@Yuval-Ariel Yuval-Ariel added the enhancement New feature or request label Dec 31, 2023
@Yuval-Ariel
Yuval-Ariel requested a review from ofriedma December 31, 2023 15:10
@Yuval-Ariel Yuval-Ariel self-assigned this Dec 31, 2023
Controlled by flag trace_ops (on by default).
Trace files will be written to the expected values dir
@Yuval-Ariel
Yuval-Ariel force-pushed the yuval/stress-test-add-trace branch from b4c5f1e to 29a386a Compare January 1, 2024 06:50
@ofriedma

ofriedma commented Jan 1, 2024

Copy link
Copy Markdown
Contributor

@Yuval-Ariel Could it impact the correctness of the db_stress? (I guess that no)

@Yuval-Ariel

Copy link
Copy Markdown
Contributor Author

IT will not affect the correctness.

@ofriedma ofriedma 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.

LGTM

@Yuval-Ariel
Yuval-Ariel merged commit 6ba2441 into main Jan 3, 2024
@Yuval-Ariel
Yuval-Ariel deleted the yuval/stress-test-add-trace branch January 3, 2024 08:07
Yuval-Ariel added a commit that referenced this pull request Jan 28, 2024
Adding a trace file by default in PR #797 has revealed some incompatibilities between the trace file and several configurations (more details in #813). Keep the trace file and remove the IsDone assertion.
Yuval-Ariel added a commit that referenced this pull request Jan 28, 2024
Adding a trace file by default in PR #797 has revealed some incompatibilities between the trace file and several configurations (more details in #813). Keep the trace file and remove the IsDone assertion.
Yuval-Ariel added a commit that referenced this pull request Jan 29, 2024
Adding a trace file by default in PR #797 has revealed some incompatibilities between the trace file and several configurations (more details in #813). Keep the trace file and remove the IsDone assertion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants