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

add support for early terminate secotrs manually #4433

Closed
wants to merge 2 commits into from

Conversation

ozhtdong
Copy link

No description provided.

@Kubuxu Kubuxu changed the title add support for early teminate secotrs manually add support for early terminate secotrs manually Oct 19, 2020
Copy link
Contributor

@RobQuistNL RobQuistNL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The current way to call it is;

lotus-miner proving terminate --sectors="437" --really-do-it 1

The current other sector edit commands are as follows:

lotus-miner sectors remove --really-do-it 437

I think you might want to move the command to the sectors endpoint instead of proving, and replace the sectors input. This would also be the only command that allows multiple sectors to be given in a , seperated way - so I'd either remove that option, or add it to all other.

Then there's also the required --sectors flag - whereas in other we can just use --really-do-it

Also, --really-do-it shouldn't be visible in the API output, but only visible through source code (enforcing people to know a bit of programming before they use it ;) )

I also heard that it might be better to put this entire step into the FSM as well (just like the other states).

Last but not least - Thank you! Trying to terminate 1 broken sector early now :)

},
&cli.StringFlag{
Name: "sectors",
Usage: "specied sectors[xx,xx,xx]",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo? :)

@RobQuistNL
Copy link
Contributor

jennijuju pushed a commit that referenced this pull request Oct 20, 2020
…hed:

-  Moving the cmd to lotus-shed so that miners can use this right away without waiting for fsm changes.- Note: this should only be used when the miner is 100% sure that they want to terminate the sector on chain and they will be lose the power and pay a big one-time termination penalty for the sectors.
- Only live sectors can be terminated.
- To use, run `./lotus-shed sectors terminate --really-do-it=true sectorNum1 sectorNum2`.
- A message will be sent and after the message is landed on chain, miner can run `lotus-miner sectors status --on-chain-info <terminated sector number>`, and check Expiration Info at the bottom. Both `OnTime` and `Early` are 0 indicates the sector is terminated.
@arajasek
Copy link
Contributor

arajasek commented Oct 20, 2020

Superseded by #4507 (guessing this can be closed)

@jennijuju
Copy link
Member

@ozhtdong Thank your for the PR first of all. Just want to let you know that we moved your implementation to lotus-shed with some ui changes(to make things more consistent with lotus-miner sectors cli as @RobQuistNL suggested). We moved it to lotus-shed so that this can be used as a utility cli for miners who wants to terminate the sectors on chain immediately. And we cant merge your pr yet b/c we will have to modify things in fsm.go for upgrading the sector status too.

@ozhtdong
Copy link
Author

ozhtdong commented Oct 21, 2020

@RobQuistNL all right, I agree with those and thank you for your modify, you can close this pr.

@magik6k magik6k closed this Oct 22, 2020
magik6k pushed a commit that referenced this pull request Oct 27, 2020
…hed:

-  Moving the cmd to lotus-shed so that miners can use this right away without waiting for fsm changes.- Note: this should only be used when the miner is 100% sure that they want to terminate the sector on chain and they will be lose the power and pay a big one-time termination penalty for the sectors.
- Only live sectors can be terminated.
- To use, run `./lotus-shed sectors terminate --really-do-it=true sectorNum1 sectorNum2`.
- A message will be sent and after the message is landed on chain, miner can run `lotus-miner sectors status --on-chain-info <terminated sector number>`, and check Expiration Info at the bottom. Both `OnTime` and `Early` are 0 indicates the sector is terminated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants