-
Bobtette
moneromooo, Now there are a lot of tx when i do print_pool_sh, looks like those are all the tx in the network still need to get confirmed?
-
moneromooo
Yes.
-
Bobtette
Any way i can force this pending tx to get failed status so i know it will not try to push this tx again and again?
-
moneromooo
Does it show pending or failed in monero-wallet-cli ?
-
Bobtette
In GUI it shows pending. I´ll check CLI
-
xmr-pr
selsta opened pull request #2950: LeftPanel: simpifly color binding
-
xmr-pr
-
Bobtette
moneromooo, In CLI it shows as failed.
-
selsta
Possible that the GUI can’t display failed transactions.
-
Bobtette
So i assume its safe to re-send the transaction.
-
selsta
any way to simulate a failed transaction?
-
Bobtette
No idea.
-
selsta
If I manage to simulate a failed tx I can add GUI support for it :P
-
moneromooo
For a failed tx: start the daemon with --offline, send one, then flush_txpool_, then let the wallet refresh a couple time.
-
xmr-pr
rating89us opened issue #2951: Transfer: amount field should also accept comma as decimal separator
-
xmr-pr
-
xmr-pr
rating89us opened issue #2952: Hardware wallet: request to plug in the device only when sending trans...
-
xmr-pr
-
selsta
moneromooo: E: failed to get output distribution, request = Daemon reports suspicious number of rct outputs
-
selsta
when trying to send with --offline
-
selsta
ok managed to do it
-
moneromooo
Do you know why then ?
-
selsta
yep my daemon was like 1 day behind
-
selsta
so I synced up and restarted with --offline
-
selsta
now I have the problem that is_failed always evaluates to true
-
selsta
-
selsta
does ^ look correct?
-
moneromooo
Failed it set if the tx was not seen in the chain nor the pool for two refreshes.
-
moneromooo
Does it work in monero-wallet-cli ?
-
selsta
yes
-
moneromooo
I assume you copied the logic off monero-wallet-cli ?
-
selsta
the logic did already exist
-
selsta
the one I linked before
-
moneromooo
Then the most likely seems that the refresh calls are wrong.
-
moneromooo
Was that particular tx mined ?
-
selsta
nope
-
selsta
directly after sending both pending and failed are both true in wallet2_api
-
selsta
(without getting mined yet)
-
moneromooo
And it's in the txpool ?
-
selsta
yes
-
selsta
I can see it on xmrchain.net
-
moneromooo
Then you possibly refreshed the chain but not the txpool, so as far as it's concerned, it sees it in neither.
-
moneromooo
Bump log level in monerod to see which RPC are being called.
-
moneromooo
*rpc*:DEBUG ought to do
-
selsta
interesting
-
selsta
using my local node it works
-
selsta
but my remote node (restricted) does not
-
selsta
I tested CLI with my local node
-
moneromooo
Ah, maybe restricted mode hides the tx.
-
moneromooo
(If received by RPC rather than P2P)
-
moneromooo
Yes, it must do. You wouldn't want people to see what you send.
-
selsta
CLI correctly sets it to false even when using a restricted remote node
-
selsta
s/false/failed/
-
moneromooo
You said it was always setting to failed, so maybe it's accidentially setting it correctly.
-
selsta
will test once my balance is unlocked
-
selsta
`pending out`
-
selsta
looks good
-
selsta
in CLI
-
selsta
oh well will maybe ask xiphon to take a look at it, starting to get confused
-
selsta
if I create the tx with the GUI + restricted node -> open same wallet in CLI it will say failed while still pending
-
selsta
if I do the same in CLI + restricted node it will say correctly pending
-
selsta
guess that’s the problem
-
moneromooo
The GUI probably set it wrong before the wallet has a chance to do anything.
-
selsta
with local node everything is ok in both CLI / GUI
-
selsta
-
selsta
here is_failed doesn’t even get set
-
selsta
so maybe it defaults to true?
-
moneromooo
No idea without looking.
-
moneromooo
I looked. Looks like pending when created.
-
selsta
yep the pending flag is correct
-
selsta
the failed one is just the problem
-
moneromooo
Flag ? It's an enum. It's one or the other.
-
moneromooo
(or the third)
-
selsta
on the GUI side it’s a flag
-
selsta
see line 207
-
moneromooo
OK, I see it, but it's initialized from the enum AFAICT.
-
selsta
yes
-
xmr-pr
selsta opened pull request #2953: [do not merge] History: add failed transaction state
-
xmr-pr