| 16 Jan 2024 |
| zealot joined the room. | 12:16:49 |
| zealot left the room. | 12:17:04 |
| Saturn set a profile picture. | 17:55:57 |
| 17 Jan 2024 |
| Sumner Evans changed their profile picture. | 05:26:12 |
| garionion changed their profile picture. | 09:36:17 |
| 23 Jan 2024 |
| adamcstephens 🐝 joined the room. | 00:43:27 |
| 28 Jan 2024 |
K900 ⚡️ | Hey folks | 15:18:40 |
K900 ⚡️ | https://github.com/NixOS/nixpkgs/blob/1e5988ee34cb0c0c5d0db9594f51841455730f54/pkgs/build-support/go/module.nix#L276-L278 | 15:18:41 |
K900 ⚡️ | Is there a reason this is serialized? | 15:18:47 |
K900 ⚡️ | Except "no one bothered to make it fast" | 15:18:52 |
@qbit:tapenet.org | My guess would be that you can’t assume the tests can be parallelized- there is a specific test call to allow for it. | 15:36:08 |
@qbit:tapenet.org | Tests are concurrent by default thi | 15:36:20 |
@qbit:tapenet.org | Tho | 15:36:22 |
K900 ⚡️ | But it's doing one package at a time | 15:36:42 |
K900 ⚡️ | Like | 15:36:49 |
K900 ⚡️ | It's doing compile-link-run-compile-link-run etc | 15:36:59 |
K900 ⚡️ | When it could be doing multiple packages in parallel | 15:37:12 |
K900 ⚡️ | Tests in different packages can't talk to each other anyway, right? | 15:37:22 |
K900 ⚡️ | I guess they could be touching ambient state | 15:37:29 |
K900 ⚡️ | But we could have an escape hatch for that | 15:37:36 |
@qbit:tapenet.org | They can call other packages | 15:38:16 |
K900 ⚡️ | Wew | 15:38:28 |
K900 ⚡️ | /me looks at 40 minutes of grafana tests | 15:38:41 |
@qbit:tapenet.org | try adding -parallel to the check flags? | 15:45:16 |
K900 ⚡️ | That's not really the problem | 15:45:55 |
K900 ⚡️ | The problem is that it's like 150 packages | 15:46:05 |
K900 ⚡️ | And most of them HAVE NO TESTS | 15:46:09 |
K900 ⚡️ | But it still has to build them, link them and run the resulting binary just for it to print "oops no tests lol" | 15:46:24 |
K900 ⚡️ | Or run like one test in like 0.1 seconds | 15:46:29 |
@qbit:tapenet.org | well, a number of the tests can be done in parallel: https://github.com/search?q=repo%3Agrafana%2Fgrafana+parallel+path%3Atest&type=code | 15:59:02 |