Skip to content

Conversation

@akundaz
Copy link

@akundaz akundaz commented Dec 3, 2025

No description provided.

@akundaz akundaz self-assigned this Dec 3, 2025
@ferranbt
Copy link
Collaborator

ferranbt commented Dec 9, 2025

Is this ready for review?

@akundaz
Copy link
Author

akundaz commented Dec 11, 2025

Is this ready for review?

@julio4 is this working at the moment? If so then we should merge in main and push this change

@julio4 julio4 force-pushed the jovian-fix branch 2 times, most recently from 8fc17c3 to 0c0d13b Compare December 13, 2025 13:16
akundaz and others added 4 commits December 15, 2025 16:26
@julio4
Copy link
Member

julio4 commented Dec 15, 2025

Do we have to keep support of enableLatestFork?

@akundaz
Copy link
Author

akundaz commented Dec 16, 2025

Do we have to keep support of enableLatestFork?

It looks like it would be useful for testing hard forks in the future. Do you want to remove it?

@julio4
Copy link
Member

julio4 commented Dec 16, 2025

Do we have to keep support of enableLatestFork?

It looks like it would be useful for testing hard forks in the future. Do you want to remove it?

Agree but I'm not sure how to correctly test activation of jovian here. Both op-node and op-geth should activate jovian at the same timestamp but get Invalid payload attributes err

Copy link
Author

akundaz commented Dec 16, 2025

So the option to set the block number at which the hard fork activates doesn't work?

@julio4
Copy link
Member

julio4 commented Dec 17, 2025

So the option to set the block number at which the hard fork activates doesn't work?

It sets correctly the fork time for op-geth and op-node but when reaching it the activation is failing. I didn't spend much time debugging it as it works by activating by default at 0

Sign up for free to 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.

3 participants