Skip to content

Commit

Permalink
feat(bartio): Update configs
Browse files Browse the repository at this point in the history
  • Loading branch information
itsdevbear committed Jun 28, 2024
1 parent 930f68d commit b65689d
Show file tree
Hide file tree
Showing 7 changed files with 13 additions and 17 deletions.
6 changes: 3 additions & 3 deletions mod/beacon/blockchain/receive.go
Original file line number Diff line number Diff line change
Expand Up @@ -60,13 +60,13 @@ func (s *Service[
// If the block is nil or a nil pointer, exit early.
if blk.IsNil() {
s.logger.Warn(
"Aborting block verification - beacon block not found in proposal 🚫",
"Aborting block verification - beacon block not found in proposal",
)
return errors.WrapNonFatal(ErrNilBlk)
}

s.logger.Info(
"Received incoming beacon block 📫",
"Received incoming beacon block",
"state_root", blk.GetStateRoot(),
)

Expand Down Expand Up @@ -96,7 +96,7 @@ func (s *Service[
}

s.logger.Info(
"State root verification succeeded - accepting incoming beacon block 🏎️ ",
"State root verification succeeded - accepting incoming beacon block",
"state_root",
blk.GetStateRoot(),
)
Expand Down
6 changes: 1 addition & 5 deletions mod/beacon/validator/block_builder.go
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,6 @@ func (s *Service[
)

defer s.metrics.measureRequestBlockForProposalTime(startTime)
s.logger.Info(
"Requesting beacon block assembly 🙈",
"slot", requestedSlot.Base10(),
)

// The goal here is to acquire a payload whose parent is the previously
// finalized block, such that, if this payload is accepted, it will be
Expand Down Expand Up @@ -121,7 +117,7 @@ func (s *Service[
}

s.logger.Info(
"Beacon block successfully built 🛠️ ",
"Beacon block successfully built",
"slot", requestedSlot.Base10(),
"state_root", blk.GetStateRoot(),
"duration", time.Since(startTime).String(),
Expand Down
6 changes: 3 additions & 3 deletions mod/da/pkg/da/service.go
Original file line number Diff line number Diff line change
Expand Up @@ -191,20 +191,20 @@ func (s *Service[_, _, BlobSidecarsT, _, _]) receiveSidecars(
}

s.logger.Info(
"Received incoming blob sidecars 🚔",
"Received incoming blob sidecars",
)

// Verify the blobs and ensure they match the local state.
if err := s.bp.VerifySidecars(sidecars); err != nil {
s.logger.Error(
"rejecting incoming blob sidecars",
"rejecting incoming blob sidecars",
"reason", err,
)
return err
}

s.logger.Info(
"Blob sidecars verification succeeded - accepting incoming blob sidecars 💦",
"Blob sidecars verification succeeded - accepting incoming blob sidecars",
"num_blobs",
sidecars.Len(),
)
Expand Down
2 changes: 1 addition & 1 deletion mod/node-core/pkg/components/availability_store.go
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ func ProvideAvailibilityStore[
filedb.WithLogger(in.Logger),
),
),
in.Logger.With("service", "beacon-kit.da.store"),
in.Logger.With("service", "da-store"),
in.ChainSpec,
), nil
}
Expand Down
4 changes: 2 additions & 2 deletions mod/payload/pkg/builder/payload.go
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ func (pb *PayloadBuilder[
args = append(args, "num_blobs", len(blobsBundle.GetBlobs()))
}

pb.logger.Info("Payload retrieved from local builder 🏗️ ", args...)
pb.logger.Info("Payload retrieved from local builder", args...)

// If the payload was built by a different builder, something is
// wrong the EL<>CL setup.
Expand Down Expand Up @@ -238,7 +238,7 @@ func (pb *PayloadBuilder[
}

pb.logger.Info(
"Sending startup forkchoice update to execution client 🚀 ",
"Sending startup forkchoice update to execution client",
"head_eth1_hash", lph.GetBlockHash(),
"safe_eth1_hash", lph.GetParentHash(),
"finalized_eth1_hash", lph.GetParentHash(),
Expand Down
2 changes: 1 addition & 1 deletion testing/networks/80084/app.toml
Original file line number Diff line number Diff line change
Expand Up @@ -278,7 +278,7 @@ suggested-fee-recipient = "0x0000000000000000000000000000000000000000"
# The timeout for local build payload. This should match, or be slightly less
# than the configured timeout on your execution client. It also must be less than
# timeout_proposal in the CometBFT configuration.
payload-timeout = "1.1s"
payload-timeout = "1s"

[beacon-kit.validator]
# Graffiti string that will be included in the graffiti field of the beacon block.
Expand Down
4 changes: 2 additions & 2 deletions testing/networks/80084/config.toml
Original file line number Diff line number Diff line change
Expand Up @@ -453,7 +453,7 @@ version = "v0"
wal_file = "data/cs.wal/wal"

# How long we wait for a proposal block before prevoting nil
timeout_propose = "1.5s"
timeout_propose = "1.25s"
# How much timeout_propose increases with each round
timeout_propose_delta = "500ms"
# How long we wait after receiving +2/3 prevotes for “anything” (ie. not a single block or nil)
Expand All @@ -468,7 +468,7 @@ timeout_precommit_delta = "500ms"
# height (this gives us a chance to receive some more precommits, even
# though we already have +2/3).
# Set to 0 if you want to make progress as soon as the node has all the precommits.
timeout_commit = "1s"
timeout_commit = "750ms"

# Deprecated: set `timeout_commit` to 0 instead.
skip_timeout_commit = false
Expand Down

0 comments on commit b65689d

Please sign in to comment.