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

test(explorer): set a max_batch_size to the explorer #1202

Closed
wants to merge 38 commits into from
Closed

test(explorer): set a max_batch_size to the explorer #1202

wants to merge 38 commits into from

Conversation

avilagaston9
Copy link
Collaborator

@avilagaston9 avilagaston9 commented Oct 8, 2024 •
edited
Loading

Motivation

This PR is solely to test #1198

Description

How to Test

  1. Start anvil:
make anvil_start_with_block_time
  1. Start the explorer
make explorer_build_db
make explorer_create_env
make run_explorer
  1. Start the test:
cd operator/pkg
go test -run TestGetBatchExplorerOOM -v -timeout 0

In the explorer, you should be able to see the following output:

[error] Error calculating proof hashes: {:http, :body_too_large}

@avilagaston9 avilagaston9 mentioned this pull request Oct 8, 2024
Merged
avilagaston9 and others added 19 commits October 8, 2024 16:48
@avilagaston9
refactor: improve lgos
3bbd2a6
@avilagaston9
refactor: add comment
88aaca7
@uri-99
refactor: error message
e54d15b
@uri-99
refactor: better error handling
f2f3fcd
@entropidelic
Merge branch '1016-fixexplorer-fetch_batch_data_pointer-in-explorer-r…
a8066b2 
…eads-the-entire-response-body-without-any-limitation' into test-max-batch-size-limit-explorer
@uri-99
Merge branch '1016-fixexplorer-fetch_batch_data_pointer-in-explorer-r…
a8afa27 
…eads-the-entire-response-body-without-any-limitation' into test-max-batch-size-limit-explorer
@uri-99
Merge branch 'test-max-batch-size-limit-explorer' of https://github.c…
0004e82 
…om/yetanotherco/aligned_layer into test-max-batch-size-limit-explorer
@uri-99
Update explorer/lib/explorer_web/live/utils.ex
e69bc14
@uri-99
fix: calculate_proof_hashes
3875037
@avilagaston9
refactor: check content lenght in the headers
5cee221
@avilagaston9
fix: set invalid proof hashes to 0
9c5ef15
@avilagaston9
fix: use a binary zero
8ab0e93
@avilagaston9
Merge branch '1016-fixexplorer-fetch_batch_data_pointer-in-explorer-r…
13838a2 
…eads-the-entire-response-body-without-any-limitation' into test-max-batch-size-limit-explorer
@avilagaston9
fix: set a default content length
0466b3b
@avilagaston9
feat(explorer): add is_valid field to batches
a735611
@avilagaston9
fix: change is_verified to is_valid
abf078e
@avilagaston9
fix: set true as a default for is_valid field
48c61df
@avilagaston9
feat: add is valid status to frontend
f6d3cea
@avilagaston9
fix: convert content-length to integer
bcc9a52
Base automatically changed from 1016-fixexplorer-fetch_batch_data_pointer-in-explorer-reads-the-entire-response-body-without-any-limitation to staging October 14, 2024 20:33
@entropidelic
Copy link
Contributor

closing this one now that #1198 has been merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@avilagaston9 @entropidelic @uri-99