-
Notifications
You must be signed in to change notification settings - Fork 153
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
Vsp changes #1840
Vsp changes #1840
Conversation
internal/rpc/rpcserver/server.go
Outdated
@@ -2440,6 +2440,8 @@ func (t *ticketbuyerV2Server) RunTicketBuyer(req *pb.RunTicketBuyerRequest, svr | |||
// new vspd request | |||
var vspHost string | |||
var vspPubKey string | |||
var vspServer *vsp.VSP | |||
isVspdReq := false |
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.
remove isVspdReq, and just use if vspServer != nil
1eba0c9
to
2fe6894
Compare
vsp/vsp.go
Outdated
@@ -126,7 +126,7 @@ func New(hostname, pubKeyStr string, purchaseAccount, changeAccount uint32, dial | |||
log.Infof("VSP has successfully confirmed the fee tx for %v", txHash) | |||
case "error": | |||
log.Warnf("VSP failed to broadcast feetx for %v -- restarting process", txHash) | |||
v.Queue(ctx, *txHash, nil) | |||
v.Queue(ctx, *txHash, feeTx) |
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.
I think this is wrong; this will retry with the same feeTx
. The nil retries by generating a new feetx.
0fb7201
to
5c7f83d
Compare
On this PR I am spliting changes which are related to vsp from #1819 to this PR.
I will rebase #1819 to keep only changes related to db.