03:48:51 <UkoeHB> selsta: 8920 and 7852 are mergeable. 9198 is an easy one that needs to be merged to unblock other PRs.
03:52:19 <UkoeHB> vtnerd: 8203 could use some more review. The seraphis_lib is kind of stuck on top of that commit.
03:57:02 <UkoeHB> 9196 needs cryptography review. Maybe explore using the research funds to hire Aaron for open-ended crypto-review hours.
09:44:33 <selsta> .merge+ 8920 7852
09:44:33 <xmr-pr> Added
09:45:13 <selsta> jeffro256: please approve 9198
11:29:03 <Guest6854>  /msg NickServ IDENTIFY nssy Master2620
12:01:35 <m-relay> <r​ottenwheel:kernal.eu> Oh noes. Lol.
17:54:16 <m-relay> <p​cre:monero.social> shit happens
18:32:52 <m-relay> <j​effro256:monero.social> @selsta: reviewing now
18:33:43 <selsta> UkoeHB: 8920 tests CI failed, do you remember if this was unrelated? the logs are gone now
18:39:48 <UkoeHB> selsta: IIRC it was unrelated (that spurious race condition test). I can force-push to trigger them again if you want
18:40:44 <UkoeHB> That PR's code is independent of unit tests.
18:58:10 <m-relay> <j​effro256:monero.social> UkoeHB: is the reason that we're copying blake2b from randomx is changing the namespace ?
19:05:46 <UkoeHB> jeffro256: It adds the memwipe back in, and avoids excess coupling outside the lib.
19:06:13 <m-relay> <j​effro256:monero.social> wait im stupid i literally just looked at that
19:07:04 <m-relay> <j​effro256:monero.social> but yeah everything matches except for those  uncommented clear_internal_memory() lines
23:00:36 <selsta> UkoeHB: rerunning CI is not necessary for this PR
23:00:46 <selsta> .merge+ 9198
23:00:46 <xmr-pr> Added
23:11:27 <UkoeHB> ok, thanks :)
23:31:21 <selsta> if someone runs a node with Tor / I2P inbound connections and wants to get added as a seed node please add a comment here: https://github.com/monero-project/monero/pull/9316