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

feat: assertoor minimal integration #1327

Merged
merged 21 commits into from
Oct 17, 2024
Merged
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
a2fb27f
Initial assertoor test
rodrigo-o Oct 8, 2024
7382a6c
Added the assertoor task to the CI
rodrigo-o Oct 8, 2024
96fe806
added branch to the ethereum package url
rodrigo-o Oct 8, 2024
dbab954
removed jobs from CI and put them in their own file
rodrigo-o Oct 8, 2024
87ced43
test without our fork
rodrigo-o Oct 8, 2024
32f783e
add our eth pkg fork again
rodrigo-o Oct 8, 2024
b21ca2d
check now that branch is correctly picked up
rodrigo-o Oct 9, 2024
cce3bbd
Change file path to config
rodrigo-o Oct 9, 2024
5449526
changing config file to the root of the project
rodrigo-o Oct 9, 2024
89abeb6
Change in the ethereum-package branch
rodrigo-o Oct 9, 2024
0be4b29
Testing with the participant config
rodrigo-o Oct 9, 2024
7414d9b
Making the file more near to network_params for testing
rodrigo-o Oct 9, 2024
df8d22d
Added syncing and peers endpoint to valdiate assertoor execution
rodrigo-o Oct 9, 2024
a18768f
Push consensus file to take it form the repo
rodrigo-o Oct 11, 2024
ea945ac
cleaned-up unnedded diffs to have a minimal version of assertoor running
rodrigo-o Oct 11, 2024
dbadb49
removed unnedded diffs
rodrigo-o Oct 11, 2024
0aa41f3
Remove stability check
rodrigo-o Oct 11, 2024
a0e096b
Set to false all checks
rodrigo-o Oct 14, 2024
27a9d42
set test to [] in assertoor config
rodrigo-o Oct 14, 2024
b93e0d3
Testing without assertoor as additional service because it stills run…
rodrigo-o Oct 14, 2024
8f3b6de
Merge branch 'main' into assertoor-empty-integration
rodrigo-o Oct 14, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Set to false all checks
rodrigo-o committed Oct 14, 2024
commit a0e096bbb87c65b54dffa675abb1579bd4df9af0
6 changes: 5 additions & 1 deletion assertoor-config.yml
Original file line number Diff line number Diff line change
@@ -14,4 +14,8 @@ additional_services:
- assertoor
assertoor_params:
run_stability_check: false
run_block_proposal_check: false
run_block_proposal_check: false
run_transaction_test: false
run_blob_transaction_test: false
run_opcodes_transaction_test: false
run_lifecycle_test: false

Unchanged files with check annotations Beta

# TODO: This could be an issue regarding OS signals, we should use JSONArgs but shell form is the
# only way to pass args to ENTRYPOINT, specially important because of the cookie. Best
# solution would be to move to releases and avoid starting the node manually through iex.
ENTRYPOINT iex $IEX_ARGS_VALUE -S mix run -- $0 $@

Check warning on line 123 in Dockerfile

GitHub Actions / ethereum-testnet

JSON arguments recommended for ENTRYPOINT/CMD to prevent unintended behavior related to OS signals

JSONArgsRecommended: JSON arguments recommended for ENTRYPOINT to prevent unintended behavior related to OS signals More info: https://docs.docker.com/go/dockerfile/rule/json-args-recommended/

Check warning on line 123 in Dockerfile

GitHub Actions / Build Docker image

JSON arguments recommended for ENTRYPOINT/CMD to prevent unintended behavior related to OS signals

JSONArgsRecommended: JSON arguments recommended for ENTRYPOINT to prevent unintended behavior related to OS signals More info: https://docs.docker.com/go/dockerfile/rule/json-args-recommended/