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

Fix validator checkpoint syncer latest index #2858

Merged
merged 6 commits into from
Nov 2, 2023
Merged

Conversation

yorhodes
Copy link
Member

@yorhodes yorhodes commented Oct 26, 2023

Description

  • Ensure validator writes latest index to checkpoint syncer

Drive-by changes

  • Adds write_latest_index to checkpoint_syncer trait

Related issues

Backward compatibility

Yes

Testing

E2E

@yorhodes yorhodes changed the title latest index test Fix validator checkpoint syncer latest index Oct 26, 2023
@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #2858 (13812eb) into v3 (c44fd29) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##               v3    #2858   +/-   ##
=======================================
  Coverage   62.53%   62.53%           
=======================================
  Files          99       99           
  Lines        1017     1017           
  Branches      106      106           
=======================================
  Hits          636      636           
  Misses        344      344           
  Partials       37       37           
Components Coverage Δ
core 50.00% <ø> (ø)
hooks 69.28% <ø> (ø)
isms 69.78% <ø> (ø)
token 55.44% <ø> (ø)
middlewares 73.17% <ø> (ø)

@yorhodes yorhodes enabled auto-merge (squash) November 2, 2023 18:36
@yorhodes yorhodes requested a review from tkporter November 2, 2023 18:55
rust/agents/validator/src/submit.rs Show resolved Hide resolved
@yorhodes yorhodes merged commit 136e7c8 into v3 Nov 2, 2023
19 checks passed
@yorhodes yorhodes deleted the latest-index-test branch November 2, 2023 20:41
@yorhodes yorhodes mentioned this pull request Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants