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

rpc: verify dcrwallet's have --manualtickets set. #187

Merged
merged 1 commit into from
Sep 25, 2020

Conversation

dajohi
Copy link
Member

@dajohi dajohi commented Sep 17, 2020

No description provided.

rpc/dcrwallet.go Outdated Show resolved Hide resolved
@jholdstock
Copy link
Member

FYI @isuldor

This should be the fix for dcrwallet voting on tickets which have the same voting address but haven't paid a fee. Requires decred/dcrwallet#1833 and decred/dcrwallet#1849

@isuldor
Copy link

isuldor commented Sep 23, 2020

Unless I'm just doing something wrong, ticketbuyer can't seem to make any purchases

2020-09-22 22:21:08.257 [INF] SYNC: Connected block 000000053c978ebec805c259f4d12dc9f784cb2cca7ddffcd2ead7d7f8f211ba, height 518725, 0 wallet transaction(s)
2020-09-22 22:21:08.913 [INF] TKBY: buy: 7, fee: 119627
2020-09-22 22:21:08.913 [INF] TKBY: 203fdd4d3016e2774f19c68613f49ae1839e52e99aa4315a24196f54a5e86959:2
2020-09-22 22:21:08.913 [INF] TKBY: 49f2ad4115441b40ee5e3d51435e569f75059da14d618d01d99d6692aac5b673:2
2020-09-22 22:21:08.913 [INF] TKBY: 909253c30d9eb326935075afc5c8347744b09eb50342211f003f9ccc43fc1f7c:2
2020-09-22 22:21:08.913 [INF] TKBY: b3d7b4b3dbe1e6ad30a66ecfac8b717159746928d57acd3ca8e9dc9ab022ebb0:2
2020-09-22 22:21:08.913 [INF] TKBY: 7a30df0c57f5c5be4f0ab9f40ec5025c606c1e261bd57d87af5346418fdfddb9:2
2020-09-22 22:21:08.913 [INF] TKBY: fad67191d32e7e4fd5858a568179caf8efd7a52e3a4ed2aa7ecbba8fdbaeb6cf:1
2020-09-22 22:21:08.913 [INF] TKBY: daf6de0381be55e4b0e1ebb12c44eef2a0c7e718ba4f9de65e57ce29ffc30ad3:2
2020-09-22 22:21:08.918 [INF] WLLT: Returning address (account=0 branch=1 child=5275)
2020-09-22 22:21:08.918 [INF] WLLT: Returning address (account=0 branch=1 child=5276)
2020-09-22 22:21:08.939 [ERR] TKBY: BUY: wallet.PurchaseTicketsWithResponse: insufficient balance:: txauthor.NewUnsignedTransaction
2020-09-22 22:21:08.939 [INF] TKBY: unlocked unneeded credit 203fdd4d3016e2774f19c68613f49ae1839e52e99aa4315a24196f54a5e86959:2
2020-09-22 22:21:08.939 [INF] TKBY: unlocked unneeded credit 49f2ad4115441b40ee5e3d51435e569f75059da14d618d01d99d6692aac5b673:2
2020-09-22 22:21:08.939 [INF] TKBY: unlocked unneeded credit 909253c30d9eb326935075afc5c8347744b09eb50342211f003f9ccc43fc1f7c:2
2020-09-22 22:21:08.939 [INF] TKBY: unlocked unneeded credit b3d7b4b3dbe1e6ad30a66ecfac8b717159746928d57acd3ca8e9dc9ab022ebb0:2
2020-09-22 22:21:08.939 [INF] TKBY: unlocked unneeded credit 7a30df0c57f5c5be4f0ab9f40ec5025c606c1e261bd57d87af5346418fdfddb9:2
2020-09-22 22:21:08.939 [INF] TKBY: unlocked unneeded credit fad67191d32e7e4fd5858a568179caf8efd7a52e3a4ed2aa7ecbba8fdbaeb6cf:1
2020-09-22 22:21:08.939 [INF] TKBY: unlocked unneeded credit daf6de0381be55e4b0e1ebb12c44eef2a0c7e718ba4f9de65e57ce29ffc30ad3:2
2020-09-22 22:25:25.144 [INF] SYNC: Connected block 000000421d624b168e26840b79ad67243c6207c17a1d7eb90d3bb379382c850f, height 518726, 0 wallet transaction(s)
2020-09-22 22:25:25.797 [INF] TKBY: buy: 7, fee: 119627
2020-09-22 22:25:25.797 [INF] TKBY: 203fdd4d3016e2774f19c68613f49ae1839e52e99aa4315a24196f54a5e86959:2
2020-09-22 22:25:25.797 [INF] TKBY: 49f2ad4115441b40ee5e3d51435e569f75059da14d618d01d99d6692aac5b673:2
2020-09-22 22:25:25.797 [INF] TKBY: 909253c30d9eb326935075afc5c8347744b09eb50342211f003f9ccc43fc1f7c:2
2020-09-22 22:25:25.797 [INF] TKBY: b3d7b4b3dbe1e6ad30a66ecfac8b717159746928d57acd3ca8e9dc9ab022ebb0:2
2020-09-22 22:25:25.797 [INF] TKBY: 7a30df0c57f5c5be4f0ab9f40ec5025c606c1e261bd57d87af5346418fdfddb9:2
2020-09-22 22:25:25.797 [INF] TKBY: fad67191d32e7e4fd5858a568179caf8efd7a52e3a4ed2aa7ecbba8fdbaeb6cf:1
2020-09-22 22:25:25.797 [INF] TKBY: daf6de0381be55e4b0e1ebb12c44eef2a0c7e718ba4f9de65e57ce29ffc30ad3:2
2020-09-22 22:25:25.812 [INF] WLLT: Returning address (account=0 branch=1 child=5277)
2020-09-22 22:25:25.812 [INF] WLLT: Returning address (account=0 branch=1 child=5278)
2020-09-22 22:25:25.827 [ERR] TKBY: BUY: wallet.PurchaseTicketsWithResponse: insufficient balance:: txauthor.NewUnsignedTransaction
2020-09-22 22:25:25.827 [INF] TKBY: unlocked unneeded credit 203fdd4d3016e2774f19c68613f49ae1839e52e99aa4315a24196f54a5e86959:2
2020-09-22 22:25:25.827 [INF] TKBY: unlocked unneeded credit 49f2ad4115441b40ee5e3d51435e569f75059da14d618d01d99d6692aac5b673:2
2020-09-22 22:25:25.827 [INF] TKBY: unlocked unneeded credit 909253c30d9eb326935075afc5c8347744b09eb50342211f003f9ccc43fc1f7c:2
2020-09-22 22:25:25.827 [INF] TKBY: unlocked unneeded credit b3d7b4b3dbe1e6ad30a66ecfac8b717159746928d57acd3ca8e9dc9ab022ebb0:2
2020-09-22 22:25:25.827 [INF] TKBY: unlocked unneeded credit 7a30df0c57f5c5be4f0ab9f40ec5025c606c1e261bd57d87af5346418fdfddb9:2
2020-09-22 22:25:25.827 [INF] TKBY: unlocked unneeded credit fad67191d32e7e4fd5858a568179caf8efd7a52e3a4ed2aa7ecbba8fdbaeb6cf:1
2020-09-22 22:25:25.827 [INF] TKBY: unlocked unneeded credit daf6de0381be55e4b0e1ebb12c44eef2a0c7e718ba4f9de65e57ce29ffc30ad3:2

Running lastest dcrd/dcrwallet master and vspd with this PR merged. Voting nodes have --manualtickets.

@jholdstock
Copy link
Member

Looks unrelated to this PR. Seems strange for ticketbuyer to run into insufficient balance errs, maybe @dajohi or @jrick know whats up. Otherwise it could be worth opening a dcrwallet issue.

@isuldor
Copy link

isuldor commented Sep 23, 2020

okay, I'm using purchaseaccount=default and guessing that everyone else is testing this code using the coinshuffle server. I'm waiting to see if the wallet will work with that and then will open a case. Otherwise I may just be doing something wrong. Will update when enough time passes for epochs and blocks to come in so ticketbuyer will try to purchase some mixed tickets.

Update: Nevermind, looks like this is being addressed already here: decred/dcrwallet#1840

Copy link
Member

@jholdstock jholdstock left a comment

Choose a reason for hiding this comment

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

I've read through the dcrd funcs which have the new isTreasuryEnabled param, and afaict, hardcoding true should have no impact on the operation of vspd. LGTM.

@jholdstock jholdstock merged commit c83a2a0 into decred:master Sep 25, 2020
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.

3 participants