!QCCCSJHEsTIfozrZxz:nixos.org

Nix + Go

228 Members
Go packaging for and with Nixpkgs. | Be excellent to each other.47 Servers

Load older messages


SenderMessageTime
28 Jan 2024
@k900:0upti.meK900 ⚡️Like15:36:49
@k900:0upti.meK900 ⚡️It's doing compile-link-run-compile-link-run etc15:36:59
@k900:0upti.meK900 ⚡️When it could be doing multiple packages in parallel15:37:12
@k900:0upti.meK900 ⚡️Tests in different packages can't talk to each other anyway, right?15:37:22
@k900:0upti.meK900 ⚡️I guess they could be touching ambient state15:37:29
@k900:0upti.meK900 ⚡️But we could have an escape hatch for that15:37:36
@qbit:tapenet.org@qbit:tapenet.org They can call other packages 15:38:16
@k900:0upti.meK900 ⚡️Wew15:38:28
@k900:0upti.meK900 ⚡️/me looks at 40 minutes of grafana tests15:38:41
@qbit:tapenet.org@qbit:tapenet.org try adding -parallel to the check flags? 15:45:16
@k900:0upti.meK900 ⚡️That's not really the problem15:45:55
@k900:0upti.meK900 ⚡️The problem is that it's like 150 packages15:46:05
@k900:0upti.meK900 ⚡️And most of them HAVE NO TESTS15:46:09
@k900:0upti.meK900 ⚡️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:0upti.meK900 ⚡️Or run like one test in like 0.1 seconds15:46:29
@qbit:tapenet.org@qbit:tapenet.orgwell, a number of the tests can be done in parallel: https://github.com/search?q=repo%3Agrafana%2Fgrafana+parallel+path%3Atest&type=code15:59:02
@qbit:tapenet.org@qbit:tapenet.orgso it would probably speed things up by adding it15:59:08
@k900:0upti.meK900 ⚡️You know what16:00:07
@k900:0upti.meK900 ⚡️I have cycles16:00:09
@k900:0upti.meK900 ⚡️I can try it16:00:10
@qbit:tapenet.org@qbit:tapenet.org also maybe try setting subPackages to just grafana stuff 16:00:38
@k900:0upti.meK900 ⚡️It is all grafana stuff16:00:50
@k900:0upti.meK900 ⚡️Is the problem16:00:51
@qbit:tapenet.org@qbit:tapenet.orgthat will exclude tests in the vendor dir (which is dumb to test anyway)16:00:52
@qbit:tapenet.org@qbit:tapenet.orgit has no deps? 16:01:21
@qbit:tapenet.org@qbit:tapenet.orgi highly doubt that16:01:26
@k900:0upti.meK900 ⚡️I don't think it runs tests for anything in vendor at least16:01:45
@qbit:tapenet.org@qbit:tapenet.orgit does in my experience16:02:44
@qbit:tapenet.org@qbit:tapenet.orgwell - builds.. so I assume tests as well16:03:08
@k900:0upti.meK900 ⚡️Hmm16:03:25

Show newer messages


Back to Room ListRoom Version: 9