-
goin2mars
thanks ill try that
-
hyc
what autodetector?
-
gingeropolous
hyc, if you run 'start_mining xmr_ADDRESS auto' in the daemon, it will cycle through a number of threads until it finds the optimal # to get the highest hashrate
-
gingeropolous
optimal # of threads
-
WhatDo_
I got a quick question about one of the unit tests for bulletproofs
-
WhatDo_
the "invalid_gamma_0" test seems to also use an invalid amount (from the previous test). why? what are we testing?
-
hyc
gingeropolous: ah cool. a bit futile in the daemon though, since "optimal" will be impacted by other daemon activity
-
hyc
every other node and wallet interaction will consume CPU+memory, you really can't expect anything like optimal mining efficiency in an active daemon
-
KnifeOfPi_
hello
-
KnifeOfPi_
we’re considering making an auto-output-splitter in cake wallet - is there any danger to using transactions larger than 1in/2out?
-
KnifeOfPi_
we want to avoid an “early 2018 xwallet” situation where certain transactions stick out, but also don’t want to clog the blockchain with churn transactions
-
rbrunner
skrech: No default location for CLI config file. I write small shell scripts and batchfiles for such cases to avoid too much typing
-
rbrunner
Ups, wrong channel
-
ErCiccione[m]
Could somebody give me a screenshot of the CLI making a stagenet/testnet transaction? From transfer... to the message of success.
-
ErCiccione[m]
I tried to do it myself. I made the testnet transaction from the faucet and i mined it , but now i have to wait 58 blocks to unlock the balance and seems like i'm the only one transacting on stagenet atm
-
rbrunner
-
rbrunner
ErCiccione[m]: Like so?
-
ErCiccione[m]
rbrunner: yes, something like that, but with included the messages you get when you open the CLI
-
rbrunner
Is this for some kind of manual then?
-
ErCiccione[m]
it's for the download page of getmonero
-
ErCiccione[m]
this is what i made, but it's ugly AF:
-
ErCiccione[m]
-
ErCiccione[m]
would be cool to have something like that, but with a transfer at the bottom
-
rbrunner
Hmmm, if it's *that* public maybe put some effort into it ...
-
rbrunner
Also a simple paste would not be nice because not preserving color
-
ErCiccione[m]
? That is something i just made to show what i would like to have.
-
ErCiccione[m]
it shouldn't be a paste, but a screenshot
-
rbrunner
Yeah, that's what I mean
-
rbrunner
And I meant *me* to put some effort into it ...
-
ErCiccione[m]
ah ok. I misunderstood :)
-
sech1
moneromooo gingeropolous xmrchain.net is stuck and can't sync, this is how it's seen from other nodes:
pastebin.com/raw/0dZikAZF
-
rbrunner
-
Snipa
2019-12-01 10:51:55.563 [P2P3] ERROR verify src/cryptonote_core/blockchain.cpp:1849 Block recognized as orphaned and rejected, id = <8f3ca1541334c185029047fc57d710fea4f7c9ba6da3c4512c05ff66246656b1>, height 1979078, parent in alt 0, parent in main 0 (parent <0d9fbbfb4db65c4b8978ce007b3ef5f34cfc28b288d795378902874536e4d1b4>, current top <25ee250bb678088722562c1fdf6f0014e62f8ae891b741769620f1ef7f666e61>, chain height 1979077)
-
Snipa
Is the main issue it appears.
-
ErCiccione[m]
That's perfect. Thanks a lot rbrunner
-
rbrunner
The only detail that is not correct is the exact version number of the daemon, it's not proper release ...
-
Snipa
There's a number of nodes with the same error.
-
ErCiccione[m]
rbrunner: not a big deal. IMO the point is to show how the cli and a transaction look like
-
rbrunner
Ok then. Glad to be of help.
-
ErCiccione[m]
jsut one detail rbrunner. Could you take another screenshot, but square this time? This one is rectangular and i would have to manipulate it a bit to show properly.
-
ErCiccione[m]
To clarfy: The GUI screenshot is 888x878. Your CLI screenshot is 1018x630
-
Snipa
Looks like:
pastebin.com/raw/9T0SyymW is the source of the auctual issue.
-
Snipa
Daemon version: 0.15.0.1-6def88ad4
-
Snipa
The remote listed as IP:PORT has blocks available out to 1979169, so it's clearly validated in their daemon, but the daemon on xmrchain isn't accepting it.
-
rbrunner
ErCiccione[m]: Ok will do. Perhaps in half an hour or so
-
hyc
may need to resync. would be nice to know how long ago it went bad
-
Snipa
block 77
-
Snipa
Well, 1979077 is the dead block.
-
ErCiccione[m]
rbrunner: thanks!
-
Snipa
Popping off 500+ blocks hasn't helped so far.
-
Snipa
Also, there's almost 2 dozen nodes effected already by whatever the heck this is.
-
Snipa
Same block on blox. using the same monero version according to the reporter:
blox.minexmr.com/search?value=0d9fb…5f34cfc28b288d795378902874536e4d1b4
-
hyc
hm.not 15.0.1-release?
-
Snipa
Apparently not.
-
hyc
the commit ID is the same as the 15.0.1 tag though
-
hyc
so I can't imagine it's missing any fixes
-
Snipa
Unlikely would be my suspicion in that case. It was clearly compiled, I've found the traces from Ginger's build script.
-
Snipa
Theres 22 nodes effected at the moment based on the logs I'm seeing.
-
hyc
MDB_BAD_TXN is a runtime / memory error, shouldn't persist after a restart
-
Snipa
That's the 7th restart so far.
-
hyc
but it means there was an earlier error that monerod didn't respond to
-
hyc
i.e., there was an earlier error and monerod ought ot have aborted the TXN at that point but kept on using the DB txn instead
-
sech1
xmrchain.net runs 0.15.0.1-6def88ad4, so not a release binary
-
Snipa
Wellp, I'll start packaging an export from a known-good node.
-
sech1
maybe something borked during compilation and broke now
-
Snipa
Interesting. There's a segfault during block popping.
-
hyc
but 6def88ad4 is the same as the v0.15.0.1 tag, so any difference from -release must be in its dependencies and not monero code itself
-
Snipa
Block ID: 1973151 causes the pop-blocks to brick on that version.
-
hyc
can you let me get a copy of the DB to examine?
-
Snipa
Yeah, I'll export it here in a sec.
-
hyc
blockchain-export may crash too if it's actually corrupted
-
Snipa
Well, once I find somewhere to stick it.
-
Snipa
Yeah, I'm just going to gzip the lmdb file
-
hyc
ok
-
rbrunner
-
rbrunner
ErCiccione[m]: Square, 882x882
-
ErCiccione[m]
Awesome. Thanks again rbrunner
-
rbrunner
You are welcome :)
-
rbrunner
Testnet without a new block since 2.5 hours. 10 blocks in the last 10 hours. Looks to me that somebody drove up difficulty (or held it up) and then left, about 16 hours ago.
-
Snipa
I mean, people were on testnet testing miners, then the fork happened.
-
hyc
yeah, why waste power on testnet coins after you test successfully and mainnet switched
-
moneromooo
Snipa: "set_log 1", flush_cache bad-txs", then paste the resulting errors on verification.
-
Snipa
I know the sxmr testnet pool had 300+kh/s pre-fork.
-
sech1
it's just everyone used testnet for testing (what a surprise), so it's stuck at 1+ MH/s now
-
Snipa
moneromooo - I'll hand it off to ginger with those notes, it's 4 AM here, and I'm already up way too late.
-
rbrunner
Makes sense. Not nice anyway. Will take a while until becoming usable again
-
Snipa
Just throw a couple miners on there and let em chug, not much worse than a normal network fork.
-
rbrunner
Yeah, my old notebook with 1/1000th of that 1 MH/s :) That will show it.
-
rbrunner
No real problem of course.
-
rbrunner
Make that 1/10,000 ...
-
tevador
no problem, diff will adjust in about 10 years :P
-
dEBRUYNE
tevador: As a side note, do you have a current estimated hashrate for mainnet?
-
tevador
I think sech1 has a script to estimate it, I'd say 800 MH/s
-
dEBRUYNE
Guess it is going to take a few days before more hash joins in
-
dEBRUYNE
Because at 800 MH/s it should be quite profitable no?
-
sech1
XMR network hashrate 60 : 672.212 MH/s
-
sech1
XMR network hashrate 120: 661.608 MH/s
-
sech1
XMR network hashrate 360: 678.075 MH/s
-
sech1
right now
-
sech1
so it went down a bit
-
gingeropolous
ok, im here. what can i do?
-
sech1
fix xmrchain.net?
-
sech1
it doesn't use official release binaries and got stuck today
-
gingeropolous
well yeah i compiled the tag. it looks like they were trying to debug it
-
gingeropolous
moneromooo, hyc , did Snipa get you everything you need?
-
moneromooo
No. Said they'd leave you to it :)
-
moneromooo
"set_log 1", flush_cache bad-txs", then paste the resulting errors on verification.
-
gingeropolous
ok, well what do you need? :) oh
-
gingeropolous
well data.mdb is packed up in a .gz now....
-
gingeropolous
i guess i could unzip it and try and run it.
-
gingeropolous
well, lemme get that file to a server for dload
-
iDunk
Builds on some systems do not display -release since commit hash in version was expanded to 9 digits, instead they display the 9 digit hash even though it's a tagged build.
-
gingeropolous
ermagerd this things on a spinny. everythings gonna take forever
-
gingeropolous
ok, the sha256sum monerod bin in /bin/ matches the bin in /monero/build/release/bin . So its possible its on master and not release
-
gingeropolous
this is worse than watching paint dry
-
hyc
what are you watching?
-
hyc
is that data.mdb downloadable yet?
-
gingeropolous
moving this file around .
-
gingeropolous
question - is the whole mdb needed for this kind of debugging? or could one slice it up somehow? this sort of slicing could be useful for debugging in the future when data.mdb is 9 gajillion terrabytes
-
hyc
well, if there are corrupted pages in the file then yes, we need to see the entire file to trace the corruption.
-
hyc
if the file was normal/non-corrupted, then yes, it's possible to use mdb_copy -c to only copy the most recent data.
-
gingeropolous
gotcha
-
gingeropolous
so the file is downloadable, but moneromooo wants me to try and run the daemon on it with flush_cache. so, i'm copying the file to a different location on snipas server. hopefully i can prop a temporary http so it canbe downloaded straight from that server. i might need to transfer it to my server if that doesn't work.
-
hyc
ok
-
hyc
you're moving the .gz now?
-
gingeropolous
yeah. copying.
-
gingeropolous
hyc, i pmd you the dload server
-
gingeropolous
moneromooo, u as well.
-
Inge-
sech1: looks like it has plateaued for now?
-
sech1
Estimated hashrate is 730-750 MH/s now
-
gingeropolous
well i can't unzip this data.mdb until i get the other copy off this server
-
gingeropolous
damn you limited HD space
-
gingeropolous
anyone downloading that file? ima kill that server
-
KnifeOfPi_
hi - we’re considering making an auto-output-splitter in cake wallet (to mitigate the 20minute lock concern) - is there any danger to using transactions larger than 1in/2out? we want to improve UX while avoiding an “early 2018 xwallet” situation where certain transactions stick out. but we don’t want to clog the blockchain with churn transactions (which could happen if we only use 1/2)
-
moneromooo
No danger per se. Maybe best to make a x->16 tx to blend in with pools ?
-
moneromooo
Is "early 2018 xwallet" the people wanting to always have 3 outputs ?
-
gingeropolous
ok if anyone wants that mdb its on
66.85.74.134:8964 now
-
rbrunner
Yes. They paid some wallet usage fee to themselves with the third output, also to gain wallet use statistics
-
rbrunner
One of the worst confidence-damaging moves I ever saw in Monero wallet space
-
gingeropolous
hyc, ^
-
gingeropolous
well, about four 3 ^
-
gingeropolous
ok here goes nothing. all we need is "set_log 1", flush_cache bad-txs" right?
-
moneromooo
Yes.
-
gingeropolous
well it seems snipa tried to pop blocks?
-
moneromooo
< Snipa> Popping off 500+ blocks hasn't helped so far.
-
gingeropolous
well set_log 1 is going bonkers. couldn't see output of flush_cache bad-txs. tried to run it from other screen
-
moneromooo
It won't output anything IIRC. It'll just allow the tx to be verified again.
-
gingeropolous
ok
-
rehrar
hi. Sorry for being late. Did someone start a meeting?
-
moneromooo
No.
-
gingeropolous
ok, ima swith to the 15.0.1 i compiled
-
rehrar
Can we have one? It won't be long. :)
-
rehrar
Or are you guys in the middle of something?
-
moneromooo
Sure, go ahead if you want.
-
rehrar
Ok. Well just a few things to discuss, I guess.
-
hyc
hey, on the road atm. will check in again soon
-
rehrar
1. How'd you think the release went? Anything of importance to discuss about it?
-
gingeropolous
apparently some exchanges didn't update in time... ?
-
moneromooo
Snipa reported a couple dozen stuck nodes. I assume they're all his. Not sure whether it's related yet.
-
gingeropolous
xmrchain.net node went down, but its syncronizing fine now
-
rehrar
well, this is talking about the fork itself. Talking about the release.
-
rehrar
But if there are things to note about the fork, we can discuss those too.
-
rehrar
Nothing of note. That's good. :D
-
rehrar
2. What's planned for the immediate future?
-
moneromooo
Merging stuff to master again :)
-
rehrar
I guess some people have been batting back and forth the idea of moving to yearly releases from here on out.
-
gingeropolous
trying to break rpc-payments is high on my list
-
rehrar
But that needs to be discussed with the community at large.
-
selsta
GUI will do a point release in the next days. I guess CLI does not need one?
-
moneromooo
Not unless gingeropolous finds something that needs fixing with this sync issue.
-
rehrar
selsta what does the point release do?
-
selsta
bug fixes
-
selsta
also compile Linux GUI with Trezor support
-
fluffypony
I'm definitely keen on annual releases
-
fluffypony
I think it's time
-
selsta
annual forks or releases?
-
rehrar
fluffypony?
-
ErCiccione[m]
already?
-
selsta
I think we can aim for annual network upgrades in the future but let’s say we have CLSAG ready in 6 months I’m not sure if we should wait 6 months to deploy it.
-
selsta
But (hopefully) no more PoW changes.
-
ErCiccione[m]
I mean, there are a lot of big upgrades being worked on.
-
rehrar
perhaps an "as needed" makes sense?
-
sech1
0.15 actually took 9 months since 0.14
-
rehrar
it doesn't HAVE to have a set time frame, yeah?
-
sech1
maybe 9 months until the next release?
-
rehrar
ErCiccione[m] what other big upgrades are being worked on?
-
selsta
9 months sounds good as a target IMO.
-
moneromooo
CLSAG. Sublinear signatures.
-
rehrar
Ok. So this is something that can obviously be discussed further, but if there are any other opinions on this topic please feel free to speak up.
-
moneromooo
If you mean also non fork related: dandelion
-
moneromooo
Well, my opinion is that yearly feels long, but I'm also not the one doing the work so I'm trying to shut up.
-
rehrar
moneromooo is there any way to enforce dandelion on a protocol level? Liek is that even possible?
-
moneromooo
None that come to mind.
-
moneromooo
CLSAG's been ready for a while, waiting for reviews. If it misses a release, waiting one more year feels harsh.
-
rehrar
Alright. Anything else to discuss surrounding the release/fork?
-
rehrar
You guys think everything went pretty smoothly this time, all things considered?
-
selsta
fork went smooth, no last minute binaries and also no issue with slow blocks this time
-
rehrar
Yeah. A lot of issues from the past went fine this time around.
-
rehrar
Alright, I won't hold you guys any longer. Thanks! We can break here. Just wanted to do an after-fork check in.
-
ErCiccione
(test)
-
ErCiccione[m]
test
-
moneromooo
-=test=-
-
ErCiccione
sorry. I think my matrix handle was relaying the irc chat in a messy order and late. No idea what's happening. Is the meeting over?
-
moneromooo
I think so.
-
moneromooo
Did you need a paste of what got said after your last line ?
-
ErCiccione
Alright, then i lost all the conversation and i have to wait my matrix user to catch up -_-
-
ErCiccione
that would be appreciated, yes
-
ErCiccione
rehrar: the upgrades i was talking about were the ones mentioned by mooo. I too think would be a bit a pain to wait that long before they go live
-
luigi1111
I think annual fork is good. I guess that coincides with major releases but I don't know that it would have to
-
moneromooo
It has to, if people have to have code that's ready for the fork. Unless you're planning forks a year ahead, but that seems very unlikely.
-
rehrar
luigi1111 not a fan of the 9 mo proporsal?
-
luigi1111
I was thinking every major release
-
luigi1111
like there could be non forking ones
-
luigi1111
9 mo is ok as a stepping stone I suppose
-
fluffypony
sorry meant annual forks
-
fluffypony
not annual releases lol
-
fluffypony
selsta: we definitely SHOULD wait 6 months to deploy it
-
fluffypony
it needs extensive testing on testnet + an audit or 2 or 3
-
selsta
right that’s not what I meant :P
-
selsta
ready = tested and audited
-
fluffypony
also some of this stuff can be deployed as soft forks
-
fluffypony
not CLSAG per se, but other things
-
fluffypony
selsta: we've traditionally taken a very short route with a lot of this stuff, I'd personally like to encourage a little bit more maturity of something like that before it goes live
-
selsta
I’m not even against 1 year forks
-
selsta
I think the added 1-2 month delay this fork was really important
-
gingeropolous
whats the bang for buck for clsag?
-
gingeropolous
hrm, xmrchain node is stuck on 1979077/1979413 again
-
gingeropolous
ok moneromooo , im trying flush_cache bad-txs again
-
sarang
CLSAG saves maybe 25% space on average transactions, and 15-20% verification time
-
gingeropolous
is the plan to bump up ringsize to effectively nullify, or we stickin with 11?
-
sarang
Depends what the consensus is
-
sarang
Would be a very small increase
-
gingeropolous
is the 15.0.1 bind only available from getmonero.org?
-
moneromooo
I'd rather bump the ring size only if there's a convincing argument that it increases privacy more than epsilon. And I feel weird saying that given I like privacy all around...
-
selsta
gingeropolous: binary? yes
-
dEBRUYNE
Perhaps it would be better to bump it in conjunction with a switch to a sublinear ring signature scheme (when that becomes feasible to implement)
-
gingeropolous
ok moneromooo , ima try just using the release bins on the explorer node, unless you can think of something else i should try
-
moneromooo
Well, I just want to see the verification logs when it verifies the tx (because it's 99% going to be a tx that fails to verify).
-
gingeropolous
well i ran that command. is it in the log files ... yeah.. ok i guess i gotta grep
-
gingeropolous
any what the string would be/?
-
moneromooo
Yes. "verify" category, most likely.
-
moneromooo
Might be other interesting stuff a bit before/after too.
-
iDunk
Silly question, but you didn't write bad-txs verbatim, right ?
-
moneromooo
I did.
-
dEBRUYNE
I am personally in favor of 9-12 month schedule too fwiw
-
moneromooo
There's only one cache atm, but I left it open to have multiple caches that you'd want to differentiate.
-
dEBRUYNE
With respect to CLSAG, I think it's better to fit the fork to CLSAG than the other way around
-
dEBRUYNE
fluffypony: In a scenario where we would have annual HFs, would you still advocate for having a major release every 6 months?
-
dEBRUYNE
To avoid confusion I guess we shouldn't bump the number then
-
iDunk
Ah, I misread flush_cache for flush_txpool.
-
gingeropolous
moneromooo its on xmrchain.net:8964
-
gingeropolous
its 1e6 lines of log
-
moneromooo
ty
-
moneromooo
At what time did you run flush_cache ?
-
gingeropolous
13:15:56 according to irc logs
-
moneromooo
Oh, there's a log showing it, nvm.
-
gingeropolous
around there
-
gingeropolous
u got the file? ima kill server
-
gingeropolous
must've got it
-
fluffypony
dEBRUYNE: sure, or even more often
-
moneromooo
Yes, I have it, thanks.
-
moneromooo
And the tx doesn't get re-verified. Hrm.
-
dEBRUYNE
<fluffypony> selsta: we definitely SHOULD wait 6 months to deploy it <= Transaction format also changes as far as I know, which means that third party wallets also have to perform work to ensure their wallets remain compatible
-
dEBRUYNE
So we need to accomodate some time for that
-
fluffypony
yeah
-
selsta
good points :P
-
gingeropolous
and how far out are the super-mega-awesome protocols that'll get us 1200 ringsize at 1 kb transactions?
-
dEBRUYNE
Both Triptych and RCT 3.0's numbers looked quite feasible (as far as I could see) for ring size 128
-
gingeropolous
which may also require ecosystem-wide tx format updated etc?
-
luigi1111
release should be whenever there's enough new stuff to make sense. some of the time that would include a fork. at least that's how i see it
-
dEBRUYNE
Afaik, multisig has not been figured out though
-
gingeropolous
i guess my point is that if triptych/rct3 is gonna happen, is it worth doing this twice in such close succession
-
» moneromooo agrees with luigi1111
-
sarang
Multisig is indeed unsolved with the sublinear ideas
-
sarang
Due to a key inversion step
-
gingeropolous
moneromooo, any ideas for xmrchain node? or should i just try the release binary?
-
moneromooo
I don't see a reason for the failure so far.
-
gingeropolous
ok. well the data.mdb is copied for further research, so ima try release bins
-
gingeropolous
well that didn't do anything
-
moneromooo
Looks like it's missing a block between your current top and the one it's trying to add.
-
moneromooo
So it's a tx verification problem then. Would a level "2,*throttle*:ERROR" log be possible ? ^_^
-
moneromooo
it's *not* a tx verification problem
-
sarang
Fwiw the Triptych preprint is nearly complete
-
sarang
Finishing up the intro narrative and citations
-
gingeropolous
moneromooo, files up on same port
-
moneromooo
Got it.
-
moneromooo
It's skipping some block because it was reported as invalid earlier. So I'd need a log with "2,*throttle*:ERROR" from the start.
-
moneromooo
ie, monerod --log-level "2,*throttle*:ERROR" --whatever-else-you-use
-
moneromooo
And I'll add a flush_cache for that block cache too :)
-
gingeropolous
ah ok
-
gingeropolous
ok its running with the new log level
-
gingeropolous
i flushed and its up on the same port
-
gingeropolous
moneromooo, ^
-
moneromooo
Got it, thanks.
-
gingeropolous
no, thank YOU!!!!!!!!!!!!!!!!
-
moneromooo
Aha: Failed to query m_blocks: MDB_BAD_TXN: Transaction must abort, has a child, or is invalid
-
moneromooo
Looks like we get that one a lot these days.
-
gingeropolous
bad txn. bad bad
-
Snipa
Yeah, I linked the blockID it appeared to be in last night before I passed out.
-
Snipa
As it broke the pop-blocks tool.
-
Snipa
Causes a real pretty segfault when you hit it.
-
moneromooo
Oh, I might want to look at the stack trace for that.
-
Snipa
On the MDB file, assuming you snagged a copy, block 1973151 causes the segfault.
-
Snipa
So any pop-blocks that tries to get past that block segfaults immedately
-
gingeropolous
so what should i do to get this explorer working for the people again
-
selsta
resync
-
Snipa
Or the more sensible answer: I'll copy in a clean MDB from a known good server.
-
Snipa
Because that's way faster.
-
Snipa
As we can't remove the bad block, we'd be resyncing from scratch.
-
gingeropolous
any more debug etc needed from this mdb moneromooo hyc ? as Snipa mentioned, the data.mdb can be get gotten from my backup
-
moneromooo
How large ?
-
hyc
I'm still uncompressing the file
-
gingeropolous
ok, u gonna take over Snipa with moving a new data.mdb?
-
Snipa
I'm going to start the copy from one of sxmr's nodes in a sec, they're in the same DC so it should be fast.
-
gingeropolous
nice.
-
gingeropolous
i'll stop doin stuff on the server
-
moneromooo
paste.debian.net/hidden/a92477d4 will tell me where that error is coming from. It'll log on stderr.
-
gingeropolous
moneromooo, u want us to run that on the existing data.mdb?
-
moneromooo
Yes.
-
gingeropolous
Snipa, u seein this?
-
Snipa
Yeah, it's a .patch that needs to be applied to the source I'd assume and monerod recompiled.
-
Snipa
I'm not entirely here at the moment, so not super useful, just moving data for the most part.
-
gingeropolous
yea yeah, i just wanna make sure you don't delete data.mdb
-
Snipa
Nah, I'm copying to my home dir, then we can move it over again.
-
gingeropolous
cause i don't think the server has enough room for ... ok
-
Snipa
There's /just/ enough room for a second copy right now.
-
gingeropolous
ok moneromooo what should i do with this patched bin?
-
moneromooo
Run it till it fails to add this block, and paste stderr where it should tell you the line at which it returned MDB_BAD_TXN.
-
gingeropolous
ah, no special log level
-
gingeropolous
and ther eit is
-
gingeropolous
-
gingeropolous
moneromooo, ^
-
moneromooo
Got it. I'll have another patch to add more info where it reported.
-
hyc
so far not looking like there's any corruption in the DB
-
moneromooo
-
manifest
can someone throw an estimate of the removal of integrated payment IDs from monero, if they are going to be removed at all?
-
manifest
i guess i would prefer them not to be removed since i have a project revolving around them but if there is a legit reason for their removal i obviously don't mind
-
moneromooo
There are no plans set for removal yet.
-
manifest
good enough for me
-
gingeropolous
wow moneromooo , that errord out immediately
-
gingeropolous
-
hyc
still scanning the data.mdb file here, but still no errors found
-
moneromooo
Thanks. Seems unlikely it's any race if it fails at startup.
-
moneromooo
I'll have to leave it to hyc now, since what sets the error flag needs knowing lmdb internals.
-
hyc
ok lessee where that tripped...
-
gingeropolous
hang on moneromooo , i was running that patched bin on snipas new copied database
-
gingeropolous
-
hyc
that's a bit weird since it's saying MDB_TXN_ERROR is set
-
hyc
but that only gets set in write txns
-
hyc
and line 10758 is an mdb_stat call, which is most likely in a read txn
-
hyc
probably would be more useful to get a stacktrace when that event occurs
-
moneromooo
I think it's a height() call.
-
moneromooo
But the one I put logs around doesn't show so I'm not sure.
-
hyc
might as well just use an assert or set a gdb breakpoint
-
hyc
anyway, that most likely means the MDB_TXN_ERROR flag was set before the height() call
-
hyc
so we're seeing the result of an ignored error, long after the fact
-
moneromooo
Right. But looking at what sets the error flag in mdb.c, I'm out of my depth :)
-
moneromooo
Alloc is unlikely, after that it's page juggling.
-
hyc
well, we could set breakpoints on each of the instances, there aren't many
-
moneromooo
gingeropolous: feeling up for it ? :)
-
gingeropolous
sure!
-
hyc
ok
-
gingeropolous
im always hungry for some copypasta
-
moneromooo
Run with gdb, but before the run command, put breakpoints. Let me go find the syntax again.
-
gingeropolous
1800gosyntax
-
hyc
hmm, lemme apply that patch so my line numbers match yours
-
hyc
first: "list mdb_page_malloc"
-
moneromooo
Hrm, Should be: break mdb.c:1111 but it's not liking it.
-
moneromooo
Even with the path.
-
hyc
do the "list mdb_page_malloc" first to give gdb some file contet
-
hyc
theb "break 2031"
-
gingeropolous
ok, so start gdb ./monerod , then put in list mdb_page_malloc run ?
-
gingeropolous
or r
-
hyc
star gdb
-
hyc
then use list command
-
hyc
then use break command
-
hyc
don't run yet
-
hyc
because we want a few more breakpoints too
-
hyc
then "list mdb_page_loose"
-
hyc
then "break 2149"
-
hyc
then "list mdb_page_spill"
-
hyc
then "break 2369"
-
gingeropolous
not a good start: Function "mdb_page_malloc" not defined.
-
hyc
then "list mdb_page_alloc"
-
hyc
then "break 2617"
-
hyc
and this only works on a debug build
-
hyc
?
-
moneromooo
Function "mdb_page_malloc" not defined. Works with mdb_env_sync0 (not static).
-
gingeropolous
oh, well then i gotta recompile with debug
-
gingeropolous
ok.
-
gingeropolous
No symbol table is loaded. Use the "file" command.
-
moneromooo
It does that before it's loaded the process.
-
gingeropolous
hyc, halp. need multipass
-
moneromooo
Try "run", then ^C after it's started monerod.
-
gingeropolous
im an idiot
-
gingeropolous
ok, still got Function "mdb_page_malloc" not defined.
-
gingeropolous
but screw it, ima plow forward
-
hyc
how about any of the others?
-
gingeropolous
well, i got No line 2031 in the current file.
-
hyc
ok, mdb_dump -a scanned the entire data file and had no errors
-
hyc
yeah the line number is no good if the list command failed
-
moneromooo
Alternatively, do the same thing I did for MDB_BAD_TXN but with MDB_ERROR, and add assert(0); after the printf. Then enable core dumps, run, load core post mortem.
-
hyc
yeah that'd work
-
hyc
MDB_TXN_ERROR
-
hyc
but I'm puzzled. my debug monerod has all of the symbols present.
-
hyc
gdb monerod
-
hyc
list mdb_page_malloc
-
hyc
^ works fine
-
gingeropolous
hyc, do u just want access to the box?
-
hyc
that might be easier
-
gingeropolous
ok, lets move to pm
-
hyc
fyi - becuase this debug build uses shared libs for the internal libs, you have to start the process before the symbols are present
-
hyc
so "break main"
-
hyc
"run"
-
hyc
then everything else shows up
-
gingeropolous
ah
-
gingeropolous
yeah i was surprised it didn't error out in compile. i usually have to edit the cmake.whatever file to get debug to build
-
hyc
ok took a few seconds, but hit a break at 6399
-
hyc
-
hyc
now to figure out why this happened
-
gingeropolous
and your running off of the old db at --data-dir /home/monerodaemon/.bitmonero.bak
-
gingeropolous
correct hyc ?
-
hyc
yes
-
hyc
we're still getting stderr / stdout directed into the DB file
-
hyc
-
selsta
wasn’t mine also unbound? I forgot
-
hyc
I think it was, yes
-
gingeropolous
y did this crop up now? i feel like i was running master for awhile
-
selsta
this is happening for over a month now but I think it triggers really unlikely
-
selsta
it happened twice on my DB
-
selsta
didn’t happen to others
-
hyc
snipa was saying this happened on 22 of his nodes?
-
Snipa
Yup, it's happened to multiple so far.
-
hyc
I think we'll have to look into libunbound and see what it's doing
-
Snipa
I've just been blowing em away and resyncing them.
-
iDunk
Are you using your system's libinbound or the embedded one ?
-
gingeropolous
how come poppin blocks doesn't fix it?
-
iDunk
libunbound*
-
gingeropolous
i dunno, hyc could tell yah :)
-
iDunk
ldd monerod | grep libunbound
-
hyc
info shared says system libunbound
-
iDunk
Hmm
-
hyc
selsta which are you using?
-
selsta
> Found libunbound shared library
-
selsta
so I guess system?
-
hyc
so maybe try another build using the bundled one instead?
-
hyc
I think our bundled version is more up to date ?
-
selsta
more up to date? Mine is 1.9.5
-
iDunk
v1.7.3, I think.
-
iDunk
So not really.
-
hyc
ok
-
selsta
but I think it was 1.9.4 the last time i had problems
-
hyc
it's definitely looking like an unbound bug
-
selsta
it still writes `sendto:` when exiting the daemon
-
selsta
-
selsta
lots of fixes recently including Fix Out of Bounds Write
-
selsta
not saying that it’s related :P
-
hyc
dang, that's a ton of out of bounds bugs
-
iDunk
The one in Ubuntu 18.04 is v1.6.7, but I'm sure security updates (at least) are backported to it.
-
gingeropolous
ok. hyc , are u still hacking around there? i'll try removing the system libunound and recompiling
-
gingeropolous
and just fyi, i started another monerod process on different port bindings to resync using release bins into the default data dir