@GrahamcOfBorg test acme

nixpkgs-review pr 80542 -p nixosTests.acme passed ok. Change LGTM.

Thanks but as noted in the original PR #77578 (comment)

this is a bit of a hack and not a sustainable fix. We shouldn't be adding dependencies to the test that aren't present in the actual code (Except for the .target probes )

Was the test actually failing on Hydra for a.example.com or did you copy it over because it was there for b.example.com too ?

Anyhow lets merge this for now. but lets open an issue that we should stabilise either the tests, or fix the root cause in the acme module.

cc @flokli

Was the test actually failing on Hydra for a.example.com or did you copy it over because it was there for b.example.com too ?

It was failing for me locally without this change. There was a commit that specifically added this for b.example.com, so I also added it for a.example.com.