!QCCCSJHEsTIfozrZxz:nixos.org

Nix + Go

233 Members
Go packaging for and with Nixpkgs. | Be excellent to each other.53 Servers

Load older messages


SenderMessageTime
28 Jan 2024
@k900:0upti.meK900Tests in different packages can't talk to each other anyway, right?15:37:22
@k900:0upti.meK900I guess they could be touching ambient state15:37:29
@k900:0upti.meK900But 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.meK900Wew15: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.meK900That's not really the problem15:45:55
@k900:0upti.meK900The problem is that it's like 150 packages15:46:05
@k900:0upti.meK900And most of them HAVE NO TESTS15:46:09
@k900:0upti.meK900But 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.meK900Or 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.meK900You know what16:00:07
@k900:0upti.meK900I have cycles16:00:09
@k900:0upti.meK900I 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.meK900It is all grafana stuff16:00:50
@k900:0upti.meK900Is 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.meK900I 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.meK900Hmm16:03:25
@k900:0upti.meK900I'm rerunning it with -parallel now16:03:37
@k900:0upti.meK900We'll see where that goes16:03:40
@katexochen:matrix.orgPaul Meyer (katexochen)
In reply to @k900:0upti.me
https://github.com/NixOS/nixpkgs/blob/1e5988ee34cb0c0c5d0db9594f51841455730f54/pkgs/build-support/go/module.nix#L276-L278
This is terrible in general, I'm working on a general rework on how packages to build and test are discovered.
16:07:25

Show newer messages


Back to Room ListRoom Version: 9