Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Td/values clean #9

Open
wants to merge 15 commits into
base: trunk
Choose a base branch
from
Open

Td/values clean #9

wants to merge 15 commits into from

Conversation

darbha-ram
Copy link

No description provided.

HalosGhost and others added 15 commits May 13, 2024 13:05
Signed-off-by: Sam Stuewe <[email protected]>
…atomizer integration tests

Signed-off-by: James Lovejoy <[email protected]>
Do not skip audit if there was no previous audit running

Signed-off-by: James Lovejoy <[email protected]>
Right now, the values implementation is quite a bit more bloated than
it needs to be; the input side of a transaction only needs to be the
UHS ids (just the hashes), not the full UHS element.

Because the benchmarks were largely written against the cryptographic
commitment code (for which this optimization has an outsized effect),
they assumed this would be the case.

For now, instead of implementing this optimization in the values
solution, we just ensure the benchmarks behave correctly.

In the long-run, if this branch should stay around, we should
correctly minimize the input-side of compact transactions.

Signed-off-by: Sam Stuewe <[email protected]>
Should awaken the TC

Signed-off-by: Sam Stuewe <[email protected]>
Removes blocking SA-validation in the shards, depending on the
coordinator to faithfully handle it.

Signed-off-by: Sam Stuewe <[email protected]>
This sets up the 2PC load-generator to read the config values the Test
Controller already supports* to limit throughput to a set amount,
optionally increasing it according to a definable schedule. Among
other benefits, this allows the tester to constrain load-generators
directly rather than by relying on the preseed count to artifically
reduce their load.

This also includes the relevant config changes to enable respected
per-loadgen log-levels.

Signed-off-by: Sam Stuewe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants