Skip to content

Commit

Permalink
Fix bad merge
Browse files Browse the repository at this point in the history
  • Loading branch information
samcm committed Jul 20, 2023
1 parent d280491 commit d3f2464
Showing 1 changed file with 12 additions and 6 deletions.
18 changes: 12 additions & 6 deletions pkg/sentry/ethereum/services/duties.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,19 +104,25 @@ func (m *DutiesService) Name() Name {
return "duties"
}

func (m *DutiesService) RequiredEpochDuties() []phase0.Epoch {
func (m *DutiesService) RequiredEpochDuties(ctx context.Context) []phase0.Epoch {
now := m.metadata.Wallclock().Epochs().Current()

epochNumber := now.Number()

epochs := []phase0.Epoch{
phase0.Epoch(epochNumber - 3),
phase0.Epoch(epochNumber - 2),
phase0.Epoch(epochNumber - 1),
phase0.Epoch(epochNumber),
phase0.Epoch(epochNumber + 1),
}

// Lodestar does not support fetching beacon committees for older epochs.
if m.metadata.Client(ctx) != string(ClientLodestar) {
epochs = append(epochs,
phase0.Epoch(epochNumber-1),
phase0.Epoch(epochNumber-2),
phase0.Epoch(epochNumber-3),
)
}

final := map[phase0.Epoch]struct{}{}

// Deduplicate in case the current epoch is below epoch 3.
Expand All @@ -133,7 +139,7 @@ func (m *DutiesService) RequiredEpochDuties() []phase0.Epoch {
}

func (m *DutiesService) Ready(ctx context.Context) error {
for _, epoch := range m.RequiredEpochDuties() {
for _, epoch := range m.RequiredEpochDuties(ctx) {
if duties := m.beaconCommittees.Get(epoch); duties == nil {
return fmt.Errorf("duties for epoch %d are not ready", epoch)
}
Expand All @@ -147,7 +153,7 @@ func (m *DutiesService) backFillEpochDuties(ctx context.Context) error {
return fmt.Errorf("metadata service is not ready")
}

for _, epoch := range m.RequiredEpochDuties() {
for _, epoch := range m.RequiredEpochDuties(ctx) {
if duties := m.beaconCommittees.Get(epoch); duties == nil {
if err := m.fetchBeaconCommittee(ctx, epoch); err != nil {
return err
Expand Down

0 comments on commit d3f2464

Please sign in to comment.