[openstreetmap/openstreetmap-website] Reduce test run time, current improvement: 9.5m to 4.9m (PR #4708)

[Josh Thompson]

I'd sort of like to get this PR merged as is - I've got some other ideas that I'm exploring, but these two changes seem appropriately low-risk and high-reward. The test improvement speed is good enough. The total CI build time has now changed, too, but I'm not quite sure in exactly what ways. 

For what its worth, I wonder if at some times of day the tests run faster than at other times of day. I found one PR where the tests 'ran fast' on first blush, with a `successful in 5 m` message visible - I misplaced it and am looking again, but it also was in an errored state, so I'm not sure it's a valid data point anyway. 

Generally, the `docker/docker (pull request)` task has been posting as 'successful' in 12-14 minutes, with a bunch of tests taking 9 minutes.

When my tests are taking 5 minutes, that same task shows completing in 10 minutes, so this might represent a total CI run time savings of 20-40% (12->10 or 14-> 10)

Not bad at all. 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4708#issuecomment-2064684960
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4708/c2064684960 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240418/51cbd81f/attachment-0001.htm>