| [22:22:34] | * heartburn has quit (Ping timeout: 252 seconds) |
| [22:23:41] | * heartburn has joined #farmos |
| [02:38:08] | * heartburn has quit (Ping timeout: 248 seconds) |
| [02:46:58] | * heartburn has joined #farmos |
| [04:00:08] | * jrballesteros05[ has quit (Quit: You have been kicked for being idle) |
| [10:01:12] | <EricLarese[m]> | Hi all, it looks like the PR pipeline (https://github.com/farmOS/farmOS/actions/runs/4018524067/jobs/6908774258) is failing now in the PHPUnit mariadb tests with 95 errors on Failed to connect to your database server -- any idea what causes that ? |
| [10:01:34] | <mstenta[m]> | Eric Larese: yea that's an ongoing issue, unfortunately |
| [10:01:55] | <mstenta[m]> | Sometimes I have to repeatedly re-run MariaDB tests until they pass |
| [10:02:19] | <mstenta[m]> | It didn't always happen, and sometimes it seems worse than other times - so it's been hard to debug |
| [10:02:34] | <mstenta[m]> | It probably relates to the fact that we use paratest to run tests in parallel |
| [10:02:58] | <mstenta[m]> | And maybe there's some kind of race condition or locking occuring - I'm not sure |
| [10:03:08] | <EricLarese[m]> | ah ok, do I have authority to re-run ? I don't see any sort of restart option. |
| [10:03:25] | <mstenta[m]> | PS I've been slammed with meetings this week, so apologies for the slow action on your PR |
| [10:03:37] | <mstenta[m]> | Oh hmm well... if you run it in your own fork you have full control I think |
| [10:03:50] | <mstenta[m]> | Maybe just the PR runs are limited |
| [10:04:03] | <mstenta[m]> | But if you look in your fork's actions, it probably DID run |
| [10:05:08] | <EricLarese[m]> | ok, no worried on the meetings, this defect is low priority and I'm mostly using it to learn about the processes and such. Also I come from a corporate background where everything moves at a glacial pace, so this feels normal :) |
| [10:05:21] | <EricLarese[m]> | s/worried/worries/ |
| [10:05:39] | <mstenta[m]> | haha great! |
| [10:05:52] | <mstenta[m]> | yea this stuff is still volunteer on my part too |
| [10:06:11] | <mstenta[m]> | so i'm balancing it with "real work" too :-) |
| [14:52:12] | <mstenta[m]> | Eric Larese: I re-ran the PR tests and they passed :-) |
| [15:19:52] | <mstenta[m]> | I also started the Log module change, but need to update submit logic too it seems |
| [15:20:16] | <mstenta[m]> | Will try to get back to it after I pick up the kids |
| [15:20:54] | <mstenta[m]> | My general plan is to make the change in Log, tag a new version, then merge your PR |