Skip to content

chore: remove ev-grpc - #3380

Merged
tac0turtle merged 1 commit into
mainfrom
chore/remove-ev-grpc
Jul 10, 2026
Merged

chore: remove ev-grpc#3380
tac0turtle merged 1 commit into
mainfrom
chore/remove-ev-grpc

Conversation

@tac0turtle

@tac0turtletac0turtle commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Overview

Ev-grpc isnt needed anymore, im going to do some removal of code we dont need anymore to make maintenance easier

Summary by CodeRabbit

  • Removed
    • Removed the obsolete gRPC execution service and single-sequencer application.
    • Removed related build, Docker, deployment, protocol-generation, and release configuration.
    • Removed the gRPC executor API and associated generated client/server interfaces.
  • Documentation
    • Updated release notes to record the breaking removal.
    • Added a Storage Ownership page to the ev-reth documentation navigation and overview.

@github-actions

Copy link
Copy Markdown
Contributor

The latest Buf updates on your PR. Results from workflow CI / buf-check (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed⏩ skipped✅ passed⏩ skippedJul 10, 2026, 2:02 PM

@coderabbitai

coderabbitaiBot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8d5e6b43-605f-467d-8b7a-3dc33164a054

📥 Commits

Reviewing files that changed from the base of the PR and between fdc2b50 and 8bb2c46.

⛔ Files ignored due to path filters (3)
  • apps/grpc/go.sum is excluded by !**/*.sum
  • execution/grpc/go.sum is excluded by !**/*.sum
  • execution/grpc/types/pb/evnode/v1/execution.pb.go is excluded by !**/*.pb.go
📒 Files selected for processing (34)
  • .github/workflows/ci.yml
  • .just/build.just
  • .just/proto.just
  • CHANGELOG.md
  • RELEASE.md
  • apps/grpc/Dockerfile
  • apps/grpc/README.md
  • apps/grpc/cmd/init.go
  • apps/grpc/cmd/run.go
  • apps/grpc/docker-compose.yml
  • apps/grpc/go.mod
  • apps/grpc/main.go
  • buf.gen.grpc.yaml
  • buf.yaml
  • client/crates/types/src/proto/evnode.v1.messages.rs
  • client/crates/types/src/proto/evnode.v1.services.rs
  • docs/.vitepress/config.ts
  • docs/ev-reth/overview.md
  • execution/grpc/README.md
  • execution/grpc/client.go
  • execution/grpc/client_test.go
  • execution/grpc/go.mod
  • execution/grpc/handler.go
  • execution/grpc/otel_propagation.go
  • execution/grpc/otel_propagation_test.go
  • execution/grpc/proto/evnode/v1/execution.proto
  • execution/grpc/server.go
  • execution/grpc/server_test.go
  • execution/grpc/tx_batch.go
  • execution/grpc/tx_batch_test.go
  • execution/grpc/types/pb/evnode/v1/v1connect/execution.connect.go
  • execution/grpc/unix.go
  • execution/grpc/unix_test.go
  • go.work.example
💤 Files with no reviewable changes (31)
  • execution/grpc/README.md
  • apps/grpc/README.md
  • execution/grpc/unix_test.go
  • apps/grpc/cmd/run.go
  • buf.gen.grpc.yaml
  • execution/grpc/otel_propagation_test.go
  • apps/grpc/main.go
  • go.work.example
  • execution/grpc/tx_batch.go
  • .just/proto.just
  • execution/grpc/handler.go
  • execution/grpc/unix.go
  • execution/grpc/go.mod
  • apps/grpc/cmd/init.go
  • buf.yaml
  • .github/workflows/ci.yml
  • apps/grpc/docker-compose.yml
  • apps/grpc/go.mod
  • execution/grpc/tx_batch_test.go
  • execution/grpc/client.go
  • execution/grpc/types/pb/evnode/v1/v1connect/execution.connect.go
  • execution/grpc/proto/evnode/v1/execution.proto
  • execution/grpc/server.go
  • .just/build.just
  • RELEASE.md
  • execution/grpc/otel_propagation.go
  • execution/grpc/server_test.go
  • execution/grpc/client_test.go
  • client/crates/types/src/proto/evnode.v1.services.rs
  • client/crates/types/src/proto/evnode.v1.messages.rs
  • apps/grpc/Dockerfile

📝 Walkthrough

Walkthrough

The obsolete evgrpc application and execution gRPC transport are removed, along with their protocol definitions, generated bindings, build integrations, and release references. Buf now targets the main proto directory, and ev-reth documentation links to Storage Ownership.

Changes

Legacy gRPC removal

Layer / File(s)Summary
Executor protocol contracts
execution/grpc/proto/..., client/crates/types/src/proto/..., execution/grpc/types/pb/...
The ExecutorService schema, messages, enum, and generated client/server bindings are removed.
Transport and application removal
execution/grpc/*, apps/grpc/*
The gRPC client, server, handlers, Unix socket helpers, transaction batching, tests, and evgrpc application are deleted.
Build and repository integration
.github/workflows/ci.yml, .just/*, buf.yaml, go.work.example, CHANGELOG.md, RELEASE.md
Docker, build, protobuf generation, workspace, changelog, and release references are updated for the removed transport.

ev-reth documentation navigation

Layer / File(s)Summary
Storage Ownership links
docs/.vitepress/config.ts, docs/ev-reth/overview.md
The sidebar and overview next steps now link to Storage Ownership.

Estimated code review effort: 4 (Complex) | ~45 minutes

Possibly related PRs

Suggested labels:T:code-hygiene

Suggested reviewers:chatton, randygrok, julienrbrt, pthmas

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly matches the main change: removing ev-grpc and its related code.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-ev-grpc

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actionsBot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-10 14:08 UTC

@tac0turtle
tac0turtle merged commit 34b278a into mainJul 10, 2026
38 checks passed
@tac0turtle
tac0turtle deleted the chore/remove-ev-grpc branch July 10, 2026 14:07
@codecov

codecovBot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.26%. Comparing base (fdc2b50) to head (8bb2c46).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #3380 +/- ##
==========================================
+ Coverage 60.66% 62.26% +1.59% 
==========================================
Files 129 121 -8 Lines 14213 13439 -774 ==========================================
- Hits 8623 8368 -255 + Misses 4635 4132 -503 + Partials 955 939 -16 
FlagCoverage Δ
combined62.26% <ø> (+1.59%)⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tac0turtletac0turtle mentioned this pull request Jul 20, 2026
Sign up for freeto 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.

1 participant

@tac0turtle