Skip to content

build(deps): upgrade go-ethereum to 1.17.5 and adapt to its api changes - #4043

Open
rickstaa wants to merge 1 commit into
masterfrom
build/go-ethereum-1.17
Open

build(deps): upgrade go-ethereum to 1.17.5 and adapt to its api changes#4043
rickstaa wants to merge 1 commit into
masterfrom
build/go-ethereum-1.17

Conversation

@rickstaa

@rickstaa rickstaa commented Sep 7, 2026

Copy link
Copy Markdown
Member

Upgrades go-ethereum from 1.13.5 to 1.17.5 together with gnark-crypto, which does not build against geth 1.13, and google.golang.org/api, which does not compile against the grpc version geth pulls in. Rebased onto master after the Go 1.27 upgrade.

Code changes are limited to what the new API requires: livepeer_cli now configures the slog based logger, and the eth tests drop the removed plaintext keystore and implement the new receipts subscription on the stub. Build, all test packages, and the eth, pm, livepeer_cli, core and server suites pass locally.

Supersedes Dependabot's #4055. Worth a run against a real network before merge since this covers four minor releases of the client used for signing and sending transactions.

@github-actions github-actions Bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Sep 7, 2026
@rickstaa rickstaa changed the title build(deps): upgrade go-ethereum to 1.17 and adapt to its api changes build(deps): upgrade go-ethereum to 1.17.5 and adapt to its api changes Sep 8, 2026
@rickstaa
rickstaa force-pushed the build/go-ethereum-1.17 branch from 0e8a962 to 2a2e61f Compare September 8, 2026 21:51
@rickstaa
rickstaa changed the base branch from build/security-bumps to master September 8, 2026 21:51
go-ethereum 1.17 replaced its log package with slog, removed the
plaintext keystore and added SubscribeTransactionReceipts to
TransactionReader. livepeer_cli and the eth test stubs are updated.
gnark-crypto moves with it since 0.18 does not build against geth 1.13,
and google.golang.org/api moves so it compiles against the grpc that
geth pulls in.
@rickstaa
rickstaa force-pushed the build/go-ethereum-1.17 branch from 2a2e61f to 7885984 Compare September 9, 2026 05:34
@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 39.84980%. Comparing base (68f7fe7) to head (7885984).

Files with missing lines Patch % Lines
cmd/livepeer_cli/livepeer_cli.go 0.00000% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##              master       #4043         +/-   ##
===================================================
- Coverage   39.88785%   39.84980%   -0.03805%     
===================================================
  Files            174         174                 
  Lines          39413       39413                 
===================================================
- Hits           15721       15706         -15     
- Misses         23692       23707         +15     
Files with missing lines Coverage Δ
cmd/livepeer_cli/livepeer_cli.go 0.00000% <0.00000%> (ø)

... and 5 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 68f7fe7...7885984. Read the comment docs.

Files with missing lines Coverage Δ
cmd/livepeer_cli/livepeer_cli.go 0.00000% <0.00000%> (ø)

... and 5 files with indirect coverage changes

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

@rickstaa

rickstaa commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

@j0sh, builds, test pass. E2E for transcoding, live-video-to-video and live-runner verified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant