-
Notifications
You must be signed in to change notification settings - Fork 18
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
Restaking #367
base: master
Are you sure you want to change the base?
Conversation
* Add chiado network Signed-off-by: cyc60 <[email protected]> * Change GENESIS_FORK_VERSION for chiado Signed-off-by: cyc60 <[email protected]> * Update AVAILABLE_NETWORKS Signed-off-by: cyc60 <[email protected]> * Change gnosis deposit data amount Signed-off-by: cyc60 <[email protected]> * Update deposit data amount for gnosis Signed-off-by: cyc60 <[email protected]> * Revert "Change gnosis deposit data amount" This reverts commit 34581ea * Upd contract addresses --------- Signed-off-by: cyc60 <[email protected]> Co-authored-by: Evgeny Gusarov <[email protected]>
* Add depositDataManager contract Signed-off-by: cyc60 <[email protected]> * Refactored harvest module Signed-off-by: cyc60 <[email protected]> --------- Signed-off-by: cyc60 <[email protected]>
# Conflicts: # src/config/networks.py
* Add VaultV1Contract * Add abi
* Del api server * Replace api host/port with relayer host/port * Del pending_validator_registrations * Add get_validators_from_relayer * Add startup check * Fix proof indexes * Rename vault methods * Fix checker address * Add startup check for auto mode * Review fixes
* Fix proof indexes * Upd packages
* Add swap-xdai * Fix gno * Refactor update-state calls * Consistent update-state calls * Decrease coverage
* Add validators_manager_signature param to register_validators * Fix RelayerClient * Add relayer_endpoint, del host/port
* Upd GENESIS_VALIDATORS_IPFS_HASH * Fix SLOTS_PER_EPOCH * Upd gnosis
* Refactor validators registration * Del getattr * Add address param * Fix proof indexes * Reorder functions * Add _check_swap_xdai * Move register_validators.py * Split get_update_state_calls * Review fixes
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: Dmitri Tsumak <[email protected]>
* Fix get_withdrawable_assets * Add aggregate, del aggregate3
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
@@ -110,6 +112,9 @@ def deposit_datum_dict(self) -> dict[str, bytes]: | |||
datum_dict.update({'fork_version': fork_version}) | |||
datum_dict.update({'network_name': self.network}) | |||
datum_dict.update({'deposit_cli_version': DEPOSIT_CLI_VERSION}) | |||
if self.withdrawal_address: | |||
datum_dict.update({'withdrawal_address': self.withdrawal_address.lower()}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(optional) In deposit-data file all hex strings are without 0x prefix
data = await self.contract.functions.validatorPubkeyToInfo(public_key).call( | ||
block_identifier=block_number | ||
) | ||
return ValidatorInfo(*data) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You annotate ValidatorInfo.status is str. I suspect contract returns status as int. Because solidity enums are integers. Pls check this.
from_block=from_block, | ||
to_block=to_block, | ||
) | ||
return [QueuedWithdrawal(*event['args']['withdrawalRoot']) for event in events] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
event['args']['withdrawalRoot']
is bytes32 as I see for WithdrawalCompleted event. You can not simply wrap in QueuedWithdrawal.
Signed-off-by: cyc60 <[email protected]>
return self | ||
|
||
def __exit__(self, exc_type, exc_val, exc_tb): | ||
for file in self.files: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
handle exc?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe use contextlib.asynccontextmanager
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
Signed-off-by: cyc60 <[email protected]>
No description provided.