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

Remove datastore, fetch latest block #928

Open
wants to merge 73 commits into
base: develop
Choose a base branch
from
Open

Remove datastore, fetch latest block #928

wants to merge 73 commits into from

Conversation

volovyks
Copy link
Collaborator

@volovyks volovyks commented Nov 7, 2024 •
edited
Loading

  • Remove DataStore and all related code
  • Add Redis storage manager for persistent in app data (for now only latest processed block)
  • Do not pass latest block in Terraform, we often forget to update those values, not a good practice
  • Fetch latest block using RPC client and use it or use latest processed block (check the logic for more details)

volovyks and others added 30 commits October 2, 2024 19:48
@volovyks
basic redis setup in the same container
c2fcf30
@volovyks
formatting
f552f24
@volovyks
basic presignature-storage using redis
704ee5f
@volovyks
presignature storage wrapped in arc and ewlock
3aa2ac3
@volovyks
presignature storage redis functions implemented
2f0e5b1
@volovyks
redis presignature storage used in presignature manager
8539fc8
@volovyks
develop
e26a21a
@volovyks
serialization for presignature
feba156
@volovyks
warnings fixed
0102d1f
@volovyks
presignature bad parameters error check
3d5796b
@volovyks
min refactoring
91d25dd
@kmaus-near
added redis config file path to dockerfile, added tf to mount /data v…
05e8394 
…olume
@volovyks
it fix
9d28c3d
@volovyks
rust 2024 requirements
7ca6a5d
@volovyks
start redis in it
6e3b027
@volovyks
pull redis in it
6d6fcbb
@volovyks
lock conflict
eefe68a
@volovyks
lock file update
998c897
@volovyks
redis unit test
e449f8c
@volovyks
develop
19fc21d
@volovyks
comments 1
4f82d8b
@volovyks
comments 2
90d6d8e
@volovyks
triple manager functions renamed to match presiganture manager
3bce87b
@volovyks
serialization simlified
fde93fb
@volovyks
comments 3
9fa82ca
@kmaus-near
added start command to mpc-node and added redis URL format
1edc9b3
@kmaus-near
Merge branch 'serhii/presignature-storage' of https://github.com/near…
992f334 
…/mpc-recovery into serhii/presignature-storage
@volovyks
use account id as a part of redis storage key
077466d
@volovyks
Merge branch 'serhii/presignature-storage' of github.com:near/mpc-rec…
9737f5a 
…overy into serhii/presignature-storage
@kmaus-near
added full redis config, modified dockerfile to use it
99e8f75
volovyks and others added 21 commits October 29, 2024 16:22
@volovyks volovyks marked this pull request as ready for review November 8, 2024 19:26
@volovyks
Copy link
Collaborator Author

volovyks commented Nov 8, 2024

@kmaus-near reverted changes in mpc recovery

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@auto-mausx auto-mausx auto-mausx left review comments

@ChaoticTempest ChaoticTempest Awaiting requested review from ChaoticTempest

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.
4 participants
@volovyks @auto-mausx @kmaus-near @ChaoticTempest