refactor(beacon): remove sync checks based on sync state and head slot

The checks were removed because they were not reliable and caused
false negatives. The services' readiness is now the only criteria
for determining if the beacon node is synced.

pkg/sentry/ethereum/beacon.go

@@ -177,26 +177,11 @@ func (b *BeaconNode) Synced(ctx context.Context) error {
 		return errors.New("missing beacon status")
 	}
 
-	syncState := status.SyncState()
-	if syncState == nil {
-		return errors.New("missing beacon node status sync state")
-	}
-
-	if syncState.SyncDistance > 3 {
-		return errors.New("beacon node is not synced")
-	}
-
 	wallclock := b.Metadata().Wallclock()
 	if wallclock == nil {
 		return errors.New("missing wallclock")
 	}
 
-	currentSlot := wallclock.Slots().Current()
-
-	if currentSlot.Number()-uint64(syncState.HeadSlot) > 32 {
-		return fmt.Errorf("beacon node is too far behind head, head slot is %d, current slot is %d", syncState.HeadSlot, currentSlot.Number())
-	}
-
 	for _, service := range b.services {
 		if err := service.Ready(ctx); err != nil {
 			return errors.Wrapf(err, "service %s is not ready", service.Name())