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.

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

[BUG] sectors extend --v1-sectors bugs #6027

Closed
3 tasks
benjaminh83 opened this issue Apr 13, 2021 · 5 comments
Closed
3 tasks

[BUG] sectors extend --v1-sectors bugs #6027

benjaminh83 opened this issue Apr 13, 2021 · 5 comments
Assignees
Labels
kind/bug Kind: Bug P2 P2: Should be resolved

Comments

@benjaminh83
Copy link

benjaminh83 commented Apr 13, 2021

Edit (@jennijuju)

List of edge cases to handle in

Name: "v1-sectors",
:

  • If the sector is expired by the the time the extension message lands on chain, the message will fail. Sectors that are legally allowed to be extended should be extended successfully, which is not the case right now as all sectors will fail extension. example message bafy2bzaceawk4q2q52gr5ewouszkzl5plhp5lxkax6kk3axi4wkskqs24ricg
  • For sectors that have deals and lifetime is over 540 days, we should not try to extend them.
  • Is trying to extend the sector with an illegal extension duration
    lotus-miner sectors extend --v1-sectors` exec with error below ERROR: mpool push message: GasEstimateMessageGas error: estimating gas used: message execution failed: exit 16, reason: invalid expiration 1801153, total sector lifetime (1556194) cannot exceed 1555200 after activation 244959 (RetCode=16)

Describe the bug
I was trying to run the command to extend my v1 sectors, as the network v11 was rolled out. I did the command:

~$ lotus-miner sectors extend --v1-sectors
bafy2bzacear3hoscrxlkf3l6bfkahzijxwee2o7npgi3bdkuavghzpiv5h6gc

This did not get on chain when executed. 7h later, I restarted lotus-daemon and I also pushed some messages stuck in local mpool. It seems I now got the message on chain:
https://filfox.info/en/message/bafy2bzaceawk4q2q52gr5ewouszkzl5plhp5lxkax6kk3axi4wkskqs24ricg

Two problems:
1. One sector expired in the meantime, before the message landed on chain, causing the whole batch message to be rejected.
Status: ErrForbidden
Error Message: cannot extend expiration for expired sector 1131, expired at 665992, now 666145 (RetCode=18)
2. Some kind of overestimation happened on this batch deal, causing it to burn 2 FIL!

Version (run lotus version):

Daemon:  1.6.0+mainnet+git.3fc23a785.dirty+api1.0.1
Local: lotus-miner version 1.6.0+mainnet+git.3fc23a785.dirty

Dirty because of GPU2...

@flyworker
Copy link

lotus-miner sectors extend --v1-sectors
ERROR: mpool push message: GasEstimateMessageGas error: estimating gas used: message execution failed: exit 16, reason: cannot reduce sector 1482's expiration to 1755634 from 1759443 (RetCode=16)

@jennijuju jennijuju changed the title [BUG] ExtendSectorExpiration message ended up in Status: ErrForbidden, but burned 2 FIL [BUG] sectors extend --v1-sectors bugs Apr 13, 2021
@arajasek
Copy link
Contributor

@flyworker What is your miner ID?

@flyworker
Copy link

f02387

@jennijuju jennijuju added P1 P1: Must be resolved P2 P2: Should be resolved and removed hint/needs-triaging P1 P1: Must be resolved labels Apr 13, 2021
@TerenceClark
Copy link

#6033, this helpful

@jennijuju jennijuju linked a pull request May 24, 2021 that will close this issue
@rjan90
Copy link
Contributor

rjan90 commented Nov 25, 2021

I think this issue about extending-bugs with V1-sectors has been fixed, and can be closed now? #rengjøring

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Kind: Bug P2 P2: Should be resolved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants