-
Notifications
You must be signed in to change notification settings - Fork 3
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
Other arch #38
base: pbs
Are you sure you want to change the base?
Other arch #38
Conversation
@@ -0,0 +1,187 @@ | |||
package builder |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This comes from op-geth
dockerfile: Dockerfile.builder | ||
args: | ||
BUILDER_IMAGE: ${BUILDER_IMAGE:-flashbots/op-geth:latest} | ||
dockerfile: Dockerfile.l2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vanilla op-geth
@@ -280,6 +280,7 @@ services: | |||
- builder-op-geth | |||
command: > | |||
op-node | |||
--sequencer.enabled |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This flag enables the FCU
|
||
// save the payload, FOR SIMPLIFITY assuem this is the code path for the builder not the producer, later one we only | ||
// do this if there is a builder flag enabled. | ||
savedPayloads[l2head] = payload |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Save the payloads so that builder op-node can return them on the builder API server.
type builderServerImpl struct { | ||
} | ||
|
||
func (b *builderServerImpl) GetPayload(request *builderTypes.BuilderPayloadRequest) (*builderTypes.VersionedBuilderPayloadResponse, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Builder API server returns the cache payloads.
No description provided.