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.

Sign up for GitHub

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

Jump to bottom

Develop #15

Merged
merged 19 commits into from
Feb 2, 2024
Merged

Develop #15

merged 19 commits into from
Feb 2, 2024

Conversation

gd-0
Copy link
Collaborator

@gd-0 gd-0 commented Feb 1, 2024 •
edited
Loading

  • fix tests
  • ignore submit header and block v2 submissions for past slots
  • check and set last slot and hash delivered is now atomic
  • use tokio broadcaster to ensure single source of truth
  • other optimistations/ fixes

@gd-0
fix beacon-client crate tests
2ed7e63
@gd-0
postgres and redis cache tests that rely on local instances running n…
d334385 
…ow don't run in cargo test
@gd-0
unit test workflow + get cargo test passing
34309b1
@gd-0
rustfmt fix
eea99e5
@gd-0
Merge pull request #30 from gattaca-com/fix-tests
80ccdc4 
Fix tests
@gd-0
ignore submit_header_v2 submissions for past slots
27a5b99
@gd-0
Merge pull request #32 from gattaca-com/fix-opt-v2-demotions
337ae63 
ignore submit_header_v2 submissions for past slots
@gd-0
improve grpc gossiper "failed to broadcast header" log
44f1c8b
@gd-0
make check_and_set_last_slot_and_hash_delivered sets atomic
b8f4699
@gd-0
remove unecassary known_pub_keys Arc
dfd70a9
@gd-0
use tokio broadcaster for chain events to ensure single source of truth
76bd5b1
@gd-0
fix housekeeper_tests
3d58250
@gd-0
fix checks
3ad1b0f
@gd-0
Merge pull request #33 from gattaca-com/more-audit-fixes
035458d 
More audit fixes
@gd-0
do not exit when fetching payload if get_payload is too late
41f9f76
@gd-0 gd-0 requested review from DomAtGattaca and 0w3n-d and removed request for DomAtGattaca February 1, 2024 20:27
@gd-0 gd-0 merged commit 84d31a6 into main Feb 2, 2024
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0w3n-d 0w3n-d 0w3n-d approved these changes

@DomAtGattaca DomAtGattaca Awaiting requested review from DomAtGattaca

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gd-0 @0w3n-d