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.

Sign up for GitHub

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

Jump to bottom

771 read traffic generator #666

Merged
merged 82 commits into from
Oct 7, 2024
Merged

771 read traffic generator #666

merged 82 commits into from
Oct 7, 2024

Conversation

cody-littley
Copy link
Contributor

@cody-littley cody-littley commented Jul 25, 2024 •
edited
Loading

Why are these changes needed?

The current traffic generation utilty writes blobs to eigenDA, but does not read them back. This PR augments the generator utility so that it is also capable of reading the blobs it has previously written.

Checks

  • I've made sure the lint is passing in this PR.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, in that case, please comment that they are not relevant.
  • Testing Strategy
    • Unit tests
    • Integration tests
    • This PR is not tested :(

@cody-littley
Added comments to the generator config.
af3ac59 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Incremental progress.
8191b7f 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Incremental progress.
fff7c76 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Refactor blob writer.
3089bb0 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Add ability to read from disperser.
463f88f 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Fix formatting.
8a1b37b 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Incremental progress.
86eb80a 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Incremental progress.
f2591d6 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Able to instiantiate retrieval client.
a60dfc5 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Checkin.
9bf4333 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Checkin.
31cf1d5 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Working
a04fef5 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Expose chunks in API.
f57e677 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Extracting needed data.
3c956d8 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Added a few metrics.
9c296bf 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Added a bunch of new metrics.
1307405 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Fix bug.
6fcc825 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Added confirmation metric.
68a6a17 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Cleanup.
c098c35 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Cleanup.
de4c7a7 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Renamed verifier.
c9de527 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Added some settings to the reader.
64c7ae9 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Incremental progress on config.
85b542d 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Add more config.
47a3b7f 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Added config for the graph.
8fafa51 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Added flags for the encoder.
04468fc 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
No more adhoc config in generator.go.
c0214a4 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Moar config.
8784124 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Converted remaining constants into configuration.
e2615e4 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Added overflow table.
6d4559d 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley cody-littley mentioned this pull request Jul 30, 2024
Merged
5 tasks
@cody-littley cody-littley mentioned this pull request Jul 31, 2024
Merged
5 tasks
@cody-littley cody-littley mentioned this pull request Aug 6, 2024
Merged
5 tasks
cody-littley
cody-littley commented Oct 3, 2024
View reviewed changes
tools/traffic/Makefile Outdated Show resolved Hide resolved
@cody-littley
Cleanup.
75de082 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Merge branch 'master' into 771-read-traffic-generator
54af37a 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Revert unintentional changes.
8df796c 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Don't remove old generator yet.
8d4caf6 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Cleanup.
aac8c64 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley
Revert unecessary change.
b2b4e63 
Signed-off-by: Cody Littley <[email protected]>
@cody-littley cody-littley marked this pull request as ready for review October 3, 2024 20:09
@cody-littley cody-littley merged commit ff498a4 into Layr-Labs:master Oct 7, 2024
6 checks passed
@cody-littley cody-littley deleted the 771-read-traffic-generator branch October 7, 2024 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ian-shim ian-shim ian-shim approved these changes

@jianoaix jianoaix Awaiting requested review from jianoaix

Assignees

@cody-littley cody-littley

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cody-littley @ian-shim