Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add op-besu #58

Merged
merged 9 commits into from
Aug 27, 2024
Merged

feat: add op-besu #58

merged 9 commits into from
Aug 27, 2024

Conversation

barnabasbusa
Copy link
Contributor

No description provided.

@barnabasbusa barnabasbusa linked an issue Aug 9, 2024 that may be closed by this pull request
@barnabasbusa
Copy link
Contributor Author

Currently failing with:

2024-08-09 11:26:07.291+00:00 | main | ERROR | Besu | Failed to start Besu
  java.io.FileNotFoundException: /data/besu/execution-data/VERSION_METADATA.json (No such file or directory)
        at java.base/java.io.FileOutputStream.open0(Native Method)
        at java.base/java.io.FileOutputStream.open(FileOutputStream.java:289)
        at java.base/java.io.FileOutputStream.<init>(FileOutputStream.java:230)
        at java.base/java.io.FileOutputStream.<init>(FileOutputStream.java:179)
        at com.fasterxml.jackson.core.TokenStreamFactory._fileOutputStream(TokenStreamFactory.java:334)
        at com.fasterxml.jackson.core.JsonFactory.createGenerator(JsonFactory.java:1547)
        at com.fasterxml.jackson.databind.ObjectMapper.createGenerator(ObjectMapper.java:1257)
        at com.fasterxml.jackson.databind.ObjectMapper.writeValue(ObjectMapper.java:3980)
        at org.hyperledger.besu.ethereum.core.VersionMetadata.writeToDirectory(VersionMetadata.java:71)
        at org.hyperledger.besu.ethereum.core.VersionMetadata.versionCompatibilityChecks(VersionMetadata.java:119)
        at org.hyperledger.besu.cli.BesuCommand.run(BesuCommand.java:1173)
        at picocli.CommandLine.executeUserObject(CommandLine.java:2026)
        at picocli.CommandLine.access$1500(CommandLine.java:148)
        at picocli.CommandLine$RunLast.executeUserObjectOfLastSubcommandWithSameParent(CommandLine.java:2461)
        at picocli.CommandLine$RunLast.handle(CommandLine.java:2453)
        at picocli.CommandLine$RunLast.handle(CommandLine.java:2415)
        at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:2273)
        at picocli.CommandLine$RunLast.execute(CommandLine.java:2417)
        at picocli.CommandLine.execute(CommandLine.java:2170)
        at org.hyperledger.besu.cli.BesuCommand.lambda$createExecuteTask$1(BesuCommand.java:1100)
        at picocli.CommandLine.execute(CommandLine.java:2170)
        at org.hyperledger.besu.cli.BesuCommand.lambda$createPluginRegistrationTask$2(BesuCommand.java:1110)
        at picocli.CommandLine.execute(CommandLine.java:2170)
        at org.hyperledger.besu.cli.util.
ConfigDefaultValueProviderStrategy.execute(ConfigDefaultValueProviderStrategy.java:58)
        at picocli.CommandLine.execute(CommandLine.java:2170)
        at org.hyperledger.besu.cli.BesuCommand.executeCommandLine(BesuCommand.java:1137)
        at org.hyperledger.besu.cli.BesuCommand.parse(BesuCommand.java:1079)
        at org.hyperledger.besu.Besu.main(Besu.java:41)

cc: @GrapeBaBa

@thinkAfCod
Copy link

The startup parameters should also include --version-compatibility-protection=false to avoid this error.

@barnabasbusa
Copy link
Contributor Author

@thinkAfCod how can pass the rollup's sequencer's IP:port to op-besu? --rollup.sequencerhttp= doesn't seem to be a valid flag.

@barnabasbusa
Copy link
Contributor Author

even with --version-compatibility-protection=false added, I'm still getting the above listed error about the metadata json file missing.

@thinkAfCod
Copy link

The default user for Besu's Docker container is 'besu,' not 'root,' which imposes significant permission restrictions. You can try specifying the data-path to the /opt/besu directory.

@GrapeBaBa
Copy link

The default user for Besu's Docker container is 'besu,' not 'root,' which imposes significant permission restrictions. You can try specifying the data-path to the /opt/besu directory.

@barnabasbusa does it make sense? This is inherited from the origin besu.

@barnabasbusa
Copy link
Contributor Author

@thinkAfCod seems like besu now started up, but seem to be unable to build blocks with op-node:

t=2024-08-26T11:50:12+0000 lvl=error msg="Sequencer temporarily could not seal block" payloadID=0x1dbaadc0b59c8495 timestamp=1724672521 err="failed to seal execution payload (ID: 0x1dbaadc0b59c8495): unrecognized rpc error: Method not enabled"

l1.rpckind is set to standard on op-node.

@barnabasbusa barnabasbusa merged commit 08e7548 into main Aug 27, 2024
5 checks passed
@barnabasbusa barnabasbusa deleted the bbusa/op-besu branch August 27, 2024 12:34
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.

Add op-besu
4 participants