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

[Move] Infra Escrow and PledgedAccounts #1236

Closed
wants to merge 116 commits into from
Closed

[Move] Infra Escrow and PledgedAccounts #1236

wants to merge 116 commits into from

Conversation

0o-de-lally
Copy link
Collaborator

@0o-de-lally 0o-de-lally commented Feb 11, 2023 •
edited
Loading

PledgeAccounts

  • struct initialization on beneficiary project, and user accounts
  • deposit and pledge
  • beneficiary withdraw
  • logic for voting on dissolving/revoking pledge
  • use coins from accounts (not integers from scaffold)
  • transactional tests
    • init
    • deposit
    • withdraw
    • dissolve

Infrastructure Escrow

  • refactor epoch boundary
  • refactor validator rewards
  • refactor burn mechanism
  • VM initializes on genesis
  • VM withdraws
  • helper script for user to pledge
  • transactional tests
    • init on genesis
    • withdraw on epoch boundary

@0o-de-lally 0o-de-lally changed the base branch from main to v6 February 11, 2023 22:33
@0o-de-lally
For 0L genesis tools for hard forks, and migrations: create a MoveVm …
1b18a7a 
…session from fake data to be able to apply migrations from Move system contracts.
@0o-de-lally
wip testing fakeexector for account migrations
4c5e0d2
@0o-de-lally
Scaffolding tests for v6 genesis fork. first cleanup deprecated modules.
4e1b6cb
@0o-de-lally
add fixtures for testing recovery genesis
d5b72e1
@0o-de-lally
cleanup fixtures
074636d 
implement functional tests for:
- exporting db backup to json
- creating genesis blob from json
- creating genesis blob in one shot from db backup

TODO: launch test node from from genesis.blob
TODO: e2e tests from a fixture file
@0o-de-lally
integration test using blob from fixture to start a diem-node in test…
8ca37d0 
… mode
@0o-de-lally
patch github actions tests
e1b75c6
@0o-de-lally
full e2e test from db backup to starting a node
757648c
@0o-de-lally
Add diem_logger as per @corythian plus comments.
19bd78f
@0o-de-lally
query nodes to check if onchain state is the same as recovery.
1e5f1c6
@0o-de-lally
- include db reader util to inspect a genesis.blob.
270f569 
- refactor recovery tests
@0o-de-lally
test to check validator count. cleanup test utlis
c5b9fa4
@0o-de-lally
cleanup tests
e7df8b6
@0o-de-lally
stub a "check" feature, to check a json file agains the output genesi…
66c4ffe 
…s.blob
@0o-de-lally
finish balance comparison with error list.
203ac54
@0o-de-lally
clean
7c43037
@0o-de-lally
test to create single validator and check output.
12e6479
@0o-de-lally
skip comparing system zero address.
6bc8809
@0o-de-lally
cargo fix
6156a0b
@0o-de-lally
cargo fmt
6da792c
@0o-de-lally
create function to start a vm executor, and uses a function pointer t…
6a4f3d6 
…o transform legacy data to Move arguments.
@0o-de-lally
experimenting with alternative way of starting a vm genesis session
886bec0
@0o-de-lally
runs
1766888
@0o-de-lally
cleanup
f0b6c3b
@0o-de-lally
need to explicitly set the validators at genesis
6cbe210
@0o-de-lally
test to confirm genesis validator set, is the same as defined set.
395568c
@0o-de-lally
tests for validator set and balance passing
fded61e
@0o-de-lally
change fixtures, make node start from command line
e1c43b0
@0o-de-lally
can start a test node from genesis blob
979e811
@0o-de-lally
add tests for eval_compliance. all transactional tests passing!
c0880b5
@0o-de-lally
rename variable in Globals
0c40d56
@0o-de-lally
patch test, remove deprecated.
d94dc8d
@0o-de-lally
add bid restrictions, and checking upon seating
c88bc6f
@0o-de-lally
genesis initialization of musical chairs struct
e4ddb75
@0o-de-lally
create first stubs of Pledge Accounts, a primitive service for infra …
d40524b 
…escrow.
@0o-de-lally
list pledgers on beneficiary struct
e9838f6
@0o-de-lally
can withdraw from all accounts
133ae03
@0o-de-lally
pledge account feature complete, except for withdrawing of coins
5731916
@0o-de-lally
finalize infraescrow wrapper on pledge accounts
8204fa0
@0o-de-lally
Vm can withdraw funds
ffc3d6a
@0o-de-lally
infra escrow can withdraw
f551d7d
@0o-de-lally
adding tests, patching.
3173254
@0o-de-lally
tests for pledge, withdraw unit, withdraw e2e, revoke
5e94738
@0o-de-lally
change APIs to use coins, not u64
ec0f1c4
@0o-de-lally
now using coins instead of temp values
b1863c4
@0o-de-lally
infra escrow and pledges use coins. add helpers to withdraw coins. pa…
39339ad 
…tch tests to use coins. all tests passing.
@0o-de-lally
make pledge withdrawals proportionate to balances
8b330d2
@0o-de-lally
Refactor paying of fees based on Infra Escrow. Builds. Tests need to …
26bbfbd 
…be ported.
@0o-de-lally
reordering the epoch boundary pipeline.
b0b63fb
@0o-de-lally
refactor burning of excess network fees at boundary
d66450b
@0o-de-lally
handle zero transaction fees cases
b2540f2
@0o-de-lally
update .exp files on tests
5fcbb06
@0o-de-lally
more tests for val subsidy
a1ba41a
@0o-de-lally
patching tests
c562ec7
@0o-de-lally
refactor burn.move patch tests
96cd19f
@0o-de-lally
Copy link
Collaborator Author

@hemulin

Start in PledgeAccounts
https://github.com/0LNetworkCommunity/libra/pull/1236/files#diff-cf584d0dbf812d6fc61acddcdf686c37bcde9d95914f15744f3b83edc62ad0f4

And then InfrastructureEscrow
https://github.com/0LNetworkCommunity/libra/pull/1236/files#diff-72e528c00ee4f636bbbca7cce49625c22feaf3210368226bd41dd0c010595186

@0xzoz
Copy link
Collaborator

0xzoz commented Jul 6, 2023

Closing due to changes included #1261

@0xzoz 0xzoz closed this Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xzoz 0xzoz 0xzoz approved these changes

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

Successfully merging this pull request may close these issues.

None yet
4 participants
@0o-de-lally @nourspace @sirouk @0xzoz