Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

feat(host): Configuration using a single l1 genesis#3205

Merged
Inphi merged 1 commit intomainfrom
inphi/interop-geneses
Dec 18, 2025
Merged

feat(host): Configuration using a single l1 genesis#3205
Inphi merged 1 commit intomainfrom
inphi/interop-geneses

Conversation

@Inphi
Copy link
Collaborator

@Inphi Inphi commented Dec 17, 2025

This patch fixes an oversight in the kona-interop configuration where it accepted an optional set of L1 genesis paths. There's no good usecase to configure a host with multiple L1 genesis. As interop can only occur across chains that have identical L1.

Testing

With this change, the interop action tests now work as it's already compatible with the op-challenger:

export KONA_HOST_PATH=target/debug/kona-host
cd tests/optimism/op-e2e/actions/interop && gotestsum -- -run TestInteropFaultProofs

I'll follow up with another change to reenable the interop action tests in gh ci

Copilot AI review requested due to automatic review settings December 17, 2025 19:09
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Dec 17, 2025

Codecov Report

❌ Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.1%. Comparing base (86910c9) to head (e54da5a).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
bin/host/src/interop/handler.rs 0.0% 7 Missing ⚠️
bin/host/src/interop/cfg.rs 0.0% 5 Missing ⚠️
bin/host/src/interop/local_kv.rs 0.0% 2 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (86910c9) and HEAD (e54da5a). Click for more details.

HEAD has 19 uploads less than BASE
Flag BASE (86910c9) HEAD (e54da5a)
proof 7 0
e2e 11 0
unit 2 1

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

@theochap theochap enabled auto-merge December 17, 2025 20:35
@theochap theochap disabled auto-merge December 17, 2025 20:36
@theochap theochap enabled auto-merge December 17, 2025 20:36
@theochap theochap disabled auto-merge December 18, 2025 16:23
@theochap theochap added this pull request to the merge queue Dec 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 18, 2025
@Inphi Inphi added this pull request to the merge queue Dec 18, 2025
Merged via the queue into main with commit ca04878 Dec 18, 2025
27 of 32 checks passed
@Inphi Inphi deleted the inphi/interop-geneses branch December 18, 2025 17:14
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 15, 2026
This patch fixes an oversight in the kona-interop configuration where it
accepted an optional set of L1 genesis paths. There's no good usecase to
configure a host with multiple L1 genesis. As interop can only occur
across chains that have identical L1.

### Testing

With this change, the interop action tests now work as it's already
compatible with the op-challenger:
```
export KONA_HOST_PATH=target/debug/kona-host
cd tests/optimism/op-e2e/actions/interop && gotestsum -- -run TestInteropFaultProofs
```

I'll follow up with another change to reenable the interop action tests
in gh ci
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ensure that kona interop action tests are not disabled

3 participants