Skip to content

Conversation

samoht9277
Copy link
Collaborator

Description
Deploy Ethrex automatically, for servers

@samoht9277 samoht9277 requested a review from a team as a code owner June 13, 2025 21:32
@samoht9277 samoht9277 marked this pull request as draft June 13, 2025 21:32
Copy link

Lines of code report

Total lines added: 27
Total lines removed: 140
Total lines changed: 167

Detailed view
+------------------------------------------------------------------------+-------+------+
| File                                                                   | Lines | Diff |
+------------------------------------------------------------------------+-------+------+
| ethrex/cmd/ef_tests/state/report.rs                                    | 821   | -10  |
+------------------------------------------------------------------------+-------+------+
| ethrex/cmd/ef_tests/state/runner/mod.rs                                | 238   | -11  |
+------------------------------------------------------------------------+-------+------+
| ethrex/cmd/ef_tests/state/runner/revm_runner.rs                        | 531   | -23  |
+------------------------------------------------------------------------+-------+------+
| ethrex/cmd/ethrex/cli.rs                                               | 444   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/cmd/ethrex/initializers.rs                                      | 337   | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/cmd/ethrex/l2/command.rs                                        | 368   | +2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/blockchain/blockchain.rs                                 | 673   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/blockchain/dev/block_producer.rs                         | 115   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/blockchain/payload.rs                                    | 565   | +3   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/common/serde_utils.rs                                    | 487   | +2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/common/trie/node.rs                                      | 241   | -5   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/common/trie/node/leaf.rs                                 | 264   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/common/trie/rlp.rs                                       | 121   | -1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/common/types/block.rs                                    | 753   | +4   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/common/src/withdrawals.rs                             | 113   | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/contracts/bin/deployer/main.rs                        | 596   | -4   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/contracts/bin/system_contracts_updater/main.rs        | 39    | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/sdk/src/calldata.rs                                   | 366   | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/sequencer/block_producer.rs                           | 175   | -1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/sequencer/errors.rs                                   | 254   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/sequencer/l1_committer.rs                             | 499   | -3   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/sequencer/l1_proof_verifier.rs                        | 162   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/sequencer/mod.rs                                      | 117   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/sequencer/setup.rs                                    | 98    | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/storage/src/store_db/libmdbx.rs                       | 262   | -1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/l2/utils/helpers.rs                                      | 27    | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/discv4/server.rs                          | 844   | -6   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/kademlia.rs                               | 609   | -4   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/network.rs                                | 188   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/peer_handler.rs                           | 668   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/rlpx/connection.rs                        | 597   | -1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/rlpx/frame.rs                             | 207   | +3   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/rlpx/handshake.rs                         | 447   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/rlpx/snap.rs                              | 405   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/sync/state_sync.rs                        | 242   | -6   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/rpc/clients/eth/mod.rs                        | 1198  | -14  |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/rpc/engine/fork_choice.rs                     | 354   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/rpc/eth/gas_tip_estimator.rs                  | 130   | -2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/rpc/l2/transaction.rs                         | 223   | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/rpc/types/block.rs                            | 181   | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/rpc/types/payload.rs                          | 251   | +2   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/networking/rpc/types/transaction.rs                      | 95    | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/levm/mod.rs                                  | 507   | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/revm/helpers.rs                              | 79    | +1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/revm/mod.rs                                  | 546   | -10  |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/vm/levm/src/lib.rs                                       | 16    | -1   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/vm/levm/src/opcode_handlers/stack_memory_storage_flow.rs | 270   | -3   |
+------------------------------------------------------------------------+-------+------+
| ethrex/crates/vm/levm/src/vm.rs                                        | 179   | -7   |
+------------------------------------------------------------------------+-------+------+
| ethrex/tooling/load_test/src/main.rs                                   | 362   | -3   |
+------------------------------------------------------------------------+-------+------+

@samoht9277 samoht9277 self-assigned this Jun 24, 2025
@ilitteri ilitteri added the L2 Rollup client label Jul 3, 2025
@ilitteri ilitteri moved this to In Progress in ethrex_l2 Jul 3, 2025
@@ -0,0 +1,3 @@
[l1]
35.163.105.41 ansible_python_interpreter=/usr/bin/python3
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be here? Maybe the whole file should be gitignored?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, i'll add the file to gitignore.

Comment on lines 138 to 144
- name: Clone Explorer repository
git:
repo: "{{ explorer_url }}"
dest: "{{ explorer_dir }}"
version: stable
vars:
ansible_ssh_user: app
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fails due to lack of permissions. Maybe explorer_url is expanded with the first user (admin)?

--eth.maximum-allowed-max-fee-per-gas {{ eth_max_allowed_fee_per_gas }} \
--eth.maximum-allowed-max-fee-per-blob-gas {{ eth_max_allowed_fee_per_blob_gas }} \
--committer.commit-time {{ committer_commit_time }} \
--eth.rpc-url {{ eth_testnet_rpc }} https://sepolia.infura.io/v3/{{ infura_api_key }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid hardcoding the network name

Comment on lines 59 to 64
- name: Clone Explorer repository
git:
repo: "{{ explorer_url }}"
dest: "{{ explorer_dir }}"
version: stable
force: yes
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fails with:

FAILED! => {"changed": false, "msg": "Failed to checkout branch stable", "rc": 1, "stderr": "Already on 'stable'\n\nThis repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting '.git/hooks/post-checkout'.\n\n", "stderr_lines": ["Already on 'stable'", "", "This repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting '.git/hooks/post-checkout'.", ""], "stdout": "Your branch is up to date with 'origin/stable'.\n", "stdout_lines": ["Your branch is up to date with 'origin/stable'."]}

Comment on lines 10 to 13
local: "{{ playbook_dir }}/../crates/blockchain/metrics/provisioning/grafana_provisioning/"
remote: "/etc/grafana/provisioning/"
alerts:
local: "{{ playbook_dir }}/../crates/blockchain/metrics/provisioning/grafana_provisioning/alerting"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The metrics folder was moved to the root directory.

Comment on lines 5 to 7
[Service]
ExecStart=/usr/local/bin/yarn start

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fails because no working directory is set.

- name: Reset connection
meta: reset_connection

- name: Check if Grafana is installed
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can fail if the ssh server hasn't started yet

@github-project-automation github-project-automation bot moved this from In Progress to Requires Changes in ethrex_l2 Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L2 Rollup client
Projects
Status: Requires Changes
Development

Successfully merging this pull request may close these issues.

4 participants