-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Make message wait work, use it for storage miner init #103
Conversation
|
||
// create actors and stuff | ||
// TODO: persist this address in the storage-miner repo | ||
log.Infof("New storage miners address is: %s", addr) |
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.
That's the worker address from params, isn't it? (The one we generate few lines above - https://github.com/filecoin-project/go-lotus/pull/103/files#diff-9e20bebb13ea36c4a8ae7c31d606791bR104-R113)
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.
Its actually going to be the address created for the new miner actor, the worker address gets put inside the actors state, and that actors address gets returned to us.
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.
Oh, right. This feels a bit weird, but I can't think of anything better right now
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.
Just 1 question, rest looks good
27c1b23
to
e59944b
Compare
upgrade lotus and plans to use buildbase:v4
* add json rpc client * support onchain reconfiguration * use file membership by default * add compatibility test
The storage miner init now waits for the message it sends to get mined, and prints out the brand new miner actor address when it does! Excitement!