00:07:48 <moneromooo> wfaressuissia[m]: will you want to review https://github.com/monero-project/monero/pull/7345 ? It's going to be right in your area.
03:07:02 <slbawfalm> What kind of person steals from own community? shorturl.at/jmDM4 Your own leaders are laughing at how stupid you are falling for thier 'Magical Crypto Friendship'.
05:22:27 <selsta> .merges
05:22:27 -xmr-pr- 6810 7005 7274 7332 7349 7366 7368 7373 7394 7398 7400 7401 7402 7412 7413 7417 7418 7419 7422
08:14:36 <mj-xmr> Please mention once the merges are done, so I can run a report generation. Thanks.
08:54:58 <selsta> mj-xmr: please also add https://github.com/monero-project/monero/pull/7401 to adaptive mining
08:55:14 <selsta> so that the test gets skipped if python dependencies are not installed
08:55:38 <selsta> afterwards it's good from my side
08:58:51 <mj-xmr> selsta, on it. Thanks for coordination.
08:59:13 <mj-xmr> I'll just rebase.
09:04:18 <mj-xmr> Rebased. Waiting for the tests. https://github.com/monero-project/monero/pull/7387
09:08:22 <mj-xmr> Oops. I need to add there the other missing requirements.
09:14:40 <selsta> yep that's what I meant
09:19:40 <mj-xmr> https://github.com/monero-project/monero/pull/7396/files
09:21:54 <selsta> commit name looks wrong
09:23:55 <mj-xmr> https://github.com/monero-project/monero/pull/7387/files#diff-37f9fb9259464460050873b25e78a0f5c3cf4fd2e2d92a15be4b0595addb83ea
09:24:27 <mj-xmr> <selsta> commit name looks wrong <--- because it's a different branch.
09:24:39 <mj-xmr> Can it be like this, in order to avoid conflicts?
09:25:57 <selsta> https://github.com/monero-project/monero/pull/7396/commits
09:26:20 <selsta> PR title and commit title don't match
09:26:31 <selsta> 7387 looks ok now
09:27:55 <mj-xmr> Ah damn. Thanks
09:33:23 <mj-xmr> selsta, Done. Sorry working under pressure, because I don't want to miss TehMergeTrain™
10:32:46 <anxstgnnb> Why does the Saviour of NASA take a group achievement award and present it as a proof of individual glory? https://twitter.com/hyc_symas/status/1203709575226183683
12:09:57 <mj-xmr> selsta, there's a testing problem with the "Adaptive". I can reproduce it locally.
12:16:06 <mj-xmr> selsta, it's a problem on Master
12:16:15 <mj-xmr> I can have a look after lunch.
12:17:27 <mj-xmr> If anybody wants to pick it up before this time (so in the coming 2 hours), please let us know, not to double the work time.
12:24:33 <selsta> https://github.com/monero-project/monero/pull/7366 this seems to make functional tests fail
12:24:42 <selsta> vtnerd: ^
13:13:54 <mj-xmr> I guess everybody thought, that it was just that ever-failing mining test, that I'm about to fix.
15:36:13 <mj-xmr> I still need to do some house cleaning and in about an hour I can look into current issues until the end of the day.
16:49:25 <mj-xmr> OK. I finally have time and will have a look at master now.
17:01:52 <mj-xmr> vtnerd, have you already picked up 7366?
17:14:41 <mj-xmr> I might have quick fixed 7366 by temporarily disabling the feature.
17:15:15 <mj-xmr> check_missing_rpc_methods is still failing. selsta should these quick fixes be in separate branches?
17:19:42 <moneromooo> What's missing ?
17:22:56 <kpyvnthcdf> Did you know that all witdraw-buyer-seller-depoist chains are trackable in Monero? No? You should have read Breaking Monero. How many people are you endangering with your 'privacy' coin?
17:24:14 <mj-xmr> moneromooo, Error: wallet API method scan_tx does not have a matching function
17:24:27 <mj-xmr> I'm lost on this one TBH
17:24:39 <moneromooo> Ah, new RPC. I'll add it.
17:24:58 <mj-xmr> Thanks. Then I'll PR my "fix" and vtnerd can fix it up later on.
17:30:12 -xmr-pr- mj-xmr opened pull request #7617: Hotfix: temporarily disabled SSL certificate storing from PR 7366
17:30:12 -xmr-pr- > https://github.com/monero-project/monero/pull/7617
17:45:12 -xmr-pr- moneromooo-monero opened pull request #7618: python-rpc: add function for new scan_tx RPC
17:45:12 -xmr-pr- > https://github.com/monero-project/monero/pull/7618
17:58:28 <mj-xmr> moneromooo, I don't know where you prefer discussing, but my rationale is, that a feature, which breaks tests, also prevents other features from being tested with the same test.
17:58:45 <mj-xmr> Which is why I'm overreacting here.
17:59:27 <mj-xmr> and as usual, it concerns my ill-fated Adaptive Mining tests, which for this reason can't be merged (X_X)
18:09:47 <mj-xmr> I just wish, that one day we can have a policy: "Failing tests -> no merge"
18:10:04 <childofthecorn[m> Does disabling the test make it so the other feature can be tested with the (now disabled) same test? Seems like either way the new feature cant use that test. Might as well leave it enabled and failing, as an indication that someone needs to come fix the test
18:11:48 <mj-xmr> childofthecorn[m, The proper, honest and responsible reaction would actually be to REVERT the change which introduced the regression. I'm trying hard not to be an asshole to demand it.
18:12:48 <childofthecorn[m> <mj-xmr "childofthecorn, The proper, hone"> I've been that asshole in other projects. Features shouldn't merge with broken tests, IMO.
18:13:08 <mj-xmr> Exactly
18:13:12 <childofthecorn[m> But also, broken tests with loud errors are better than disabled tests with no errors
18:13:22 <mj-xmr> This is not a broken test.
18:13:43 <childofthecorn[m> Ah, I misread. I'll see myself out 😅
18:14:14 <childofthecorn[m> s/broken/failing/
18:14:57 <mj-xmr> OK so what I did was disabling the feature, which broke the test.
18:15:12 <mj-xmr> And I did it in a polite way, by not reverting vtnerd's changes.
18:15:33 <mj-xmr> So he can have his due weekend and return on Monday to provide a proper fix.
18:17:16 <mj-xmr> And also, when his feature breaks the test, it disables testing of other ~20 features, which opens the door for regressions even wider.
18:18:56 <mj-xmr> The academic term of this situation is a fucking mess.
18:30:08 <mj-xmr> childofthecorn[m, I know why you misread. Because I said it so earlier as if the test was broken. The test tests many things. One of them is mining, which fails with about 50% probability. I fixed this and was very close for this to be merged. Then this breaking change came and my PR can't me merged anymore.
18:30:22 <mj-xmr> So I hope you see the irony.
18:30:51 <mj-xmr> Devs are by now used to ignore the test, since they think it's that bloody mining test failing again.
18:31:11 <mj-xmr> So there you have it.
18:32:59 <childofthecorn[m> Ahhh, I see now. I should have remained a fly on the wall anyways :)
18:33:00 <childofthecorn[m> Thanks for the clarification
18:34:05 <mj-xmr> "I should have remained a fly on the wall anyways :" <--- well you did confirm a rigid methodology.
18:34:27 <mj-xmr> i.e. failing tests -> no merge
18:38:27 <vtnerd> "The academic term of this situation is a fucking mess." -> a bit hyperbolic, but will fix test
18:40:52 <mj-xmr> I'm signing off for today.
18:40:52 <vtnerd> some of these tests were/are brittle iirc, but hopefully that has been sorted out