-
iDunk
6/22 Test #6: functional_tests_rpc ............. Passed 540.66 sec
-
iDunk
unit_tests always fail like this here
paste.debian.net/hidden/3b499324
-
selsta
ok thanks for testing, guess we can add 7387 then
-
selsta
.merge+ 7387
-
xmr-pr
Added
-
iDunk
-
luigi1111w
.merges
-
xmr-pr
7088 7238 7310 7326 7332 7349 7358 7373 7387 7391 7459 7460 7520 7538 7542 7549 7615 7620 7621 7622 7623 7631 7632 7636 7637 7638
-
luigi1111w
plz hold, an operator will be with you shortly
-
luigi1111w
7373 has a comment
-
luigi1111w
hyc or moneromooo (or anyone)
-
selsta
luigi1111w: no big deal but wanted to test 7396 first before merging
-
selsta
but it's a minor PR so should not matter
-
luigi1111w
oh, blame moneromooo
-
selsta
:D
-
luigi1111w
:P
-
hyc
7373? will take a look
-
hyc
what do you suggest? not like there's a monerod user manual
-
hyc
the bug in question was that RANDOMX_UMASK didn't provide a way to control the LARGE_PAGES flag. that bug is certainly fixed
-
selsta
hyc: can you take out the "fixes #7372" part of the PR description? then we can merge it
-
hyc
ok
-
selsta
seems like the issue creator specifically wants documentation which can be done separately
-
hyc
done
-
selsta
luigi1111w: ^^
-
luigi1111w
great
-
selsta
.merge+ 7430 7435 7439 7476
-
xmr-pr
Added
-
selsta
.merges
-
xmr-pr
7332 7349 7387 7430 7435 7439 7476 7637 7638
-
luigi1111w
moneromooo 7332 rebase please
-
mj-xmr
Thanks for working overtime.
-
mj-xmr
luigi1111w, 7387 rebased.
-
mj-xmr
The test will take about 2.5 hours, if you want to call it a day already.
-
mj-xmr
<moneromooo> Looking at it it's a bit weird it cals pi rather than, say, CN, as a "let's use up CPU" function :) <-- I hear you, but it's just a module, that can be replaced in future.
-
mj-xmr
The review was exhausting enough. Though productive.
-
opypgfjnp
Why does the Saviour of NASA take a group achievement award and present it as a proof of individual glory?
twitter.com/hyc_symas/status/1203709575226183683
-
mj-xmr
luigi1111w, 7387 rebased.
-
mj-xmr
luigi1111w, 7387 is green.
-
moneromooo
luigi1111w: 7332 done
-
selsta
we would ideally include
tevador/RandomX #202 in a release
-
selsta
I assume only tevador can do RandomX merges?
-
sech1
yes
-
selsta
any way to contact him?
-
selsta
maybe ping on github is enough
-
sech1
pinged
-
moneronero
what is the use of this levin protocon inside monero?
-
moneromooo
P2P comms.
-
moneromooo
ie, transactions, blocks, mostly.
-
moneronero
ok
-
mj-xmr
-
mj-xmr
Ccache documentation PR
-
gingeropolous
v0.17.0.0-dcba757dd uptime 0d 22h 20m 45s on xmrchain. yay.
-
selsta
mj-xmr: seems like the mining test also stopped failing by itself
github.com/monero-project/monero/commits/master
-
selsta
weird maybe github updated something
-
mj-xmr
selsta, they may as well downgrade it later. I'm a fan of defensive programming. Leads to fewer surprises.
-
mj-xmr
Which is normally shunned, because people prefer to play heroes by fixing problems that become visible. But that's just the amateur sociologist in me.
-
mj-xmr
In our case we indeed have enough real problems to deal with already. With my adaptive mining I just wanted to make it easier to quickly filter PRs which are not really ready for a review.
-
selsta
sure I'm all for less failing CI :P
-
mj-xmr
Perhaps that's why I stayed here :)
-
mj-xmr
Ah, I didn't notice that the mining branch was just merged. Thanks!
-
mj-xmr
I will now run a cyclical/scheduled test (like before) of the parallel testing.
-
mj-xmr
-
selsta
mj-xmr: I guess it makes sense to take the average run time without parallel tests and compare with
-
selsta
to see how much speed benefit there is
-
mj-xmr
Sure. There are plenty of runs of this. It's about 1.5 hour.
-
luigi1111w
.merges
-
xmr-pr
7349
-
xfjzmtbrurd
Will you HODL XMR or dump it for Tari when it comes out? Will others?