Skip to content
This repository has been archived by the owner on Jun 6, 2023. It is now read-only.

Remove miners with test sector types from state #1474

Open
ZenGround0 opened this issue Aug 25, 2021 · 1 comment
Open

Remove miners with test sector types from state #1474

ZenGround0 opened this issue Aug 25, 2021 · 1 comment
Assignees

Comments

@ZenGround0
Copy link
Contributor

From an earlier issue raised by @arajasek

I noticed that this PR made possible creating miners with 2k / 8k / 512Mb post types. This is probably fine, since these miners can't add any storage, but it does have some impact on state -- the Power State's MinerAboveMinPowerCount for instance includes all 2 KiB miners since their min power is 0.

List of such miners:

f0466399
f0480175
f0480253
f0431713
f0431953
f0432178
f0480490
f0440705
f0481497
f0480393
f0486159
f0461500
f0431712
f0431939
f0431663
f0431716
f0453913
f0463860
f0463597
f0463917
f0481525
f0431711
f0480068
f0432231
f0466392
f0481115
f0481379
f0466398
f0431733
f0431725
f0431679
f0463769
f0480322
f0441531
f0481888
f0432245

Type: 5  Count:  13
Type: 6  Count:  3
Type: 7  Count:  20

We blocked further creation of these miners in v5 actors. For v6 migration we should remove these miner actors from the state, returning any funds sent to them to their owner address.

@ZenGround0
Copy link
Contributor Author

Migration testing criteria:

  • Miners with bad proof types need to go away
  • Signature of miners with bad proof types in power actor need to change
    — claim is removed
    — total miners miners over conseusns …. is removed
  • Market actor escrow table removes these bad miners
  • Owner address of these bad miners is given the funds belonging to the miner actor
  • Owner address also gets funds put in the market actor escrow table

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants