search

GenderedIntelligence / the-trans-dimension

The Trans Dimension is an online community hub connecting trans communities in London.
http://transdimension.uk
Other
3 stars 1 forks source link

[Bug]: CI is building inconsistently #404

Open kimadactyl opened 2 weeks ago

kimadactyl commented 2 weeks ago

Description

Site is failing about half the time due to an Elm compilation error. This is significantly reducing our quality of life as our notifications are throwing a lot of false positives, and information updates are inconsistent.

Currently it is as below, but I think the package name has shifted a few times.

Some idea of what might be causing it and a suggested fix is here: https://discourse.elm-lang.org/t/problem-building-dependencies-error-in-pipeline-solved/8755

09:10:12.800 | -- PROBLEM BUILDING DEPENDENCIES -----------------------------------------------
-- | --
09:10:12.800 |  
09:10:12.801 | I ran into a compilation error when trying to build the following package:
09:10:12.801 |  
09:10:12.801 | elm/regex 1.0.0
09:10:12.801 |  
09:10:12.801 | This probably means it has package constraints that are too wide. It may be
09:10:12.802 | possible to tweak your elm.json to avoid the root problem as a stopgap. Head
09:10:12.802 | over to https://elm-lang.org/community to get help figuring out how to take this
09:10:12.802 | path!
09:10:12.802 |  
09:10:12.802 | Note: To help with the root problem, please report this to the package author
09:10:12.802 | along with the following information:
09:10:12.802 |  
09:10:12.802 | elm/core 1.0.5
09:10:12.802 |  
09:10:12.802 | If you want to help out even more, try building the package locally. That should
09:10:12.803 | give you much more specific information about why this package is failing to
09:10:12.803 | build, which will in turn make it easier for the package author to fix it!
09:10:12.803 |  
09:10:12.803 | Compilation failed.
09:10:12.803 |  
09:10:12.818 | Failed: build command exited with code: 1
09:10:13.724 | Failed: error occurred while running build command

Steps to reproduce

View Cloudflare build logs on fail

kimadactyl commented 2 weeks ago

@katjam - link in issue ticket suggests a solution but it's out of my depth

katjam commented 2 weeks ago

Thanks. I did used to use that Netlify cache thing to solve the problem there, but it did stop working a couple times / changed server spec.

For this I am wondering if we pull back to building twice a day. At the moment it's every 3 hours which shouldn't be overlapping - but maybe it overlaps with someone else on same machine? I have no idea how cloudflare allocates these build servers but the problem does seem a lot more frequent than it was on netlify.

kimadactyl commented 2 weeks ago

I think it's just that there's 3 projects in elm-stuff that are clashing, no? thats what the link seemed to suggest to me, anyway, that they are all fighting each other

katjam commented 2 weeks ago

I think it's just that there's 3 projects in elm-stuff that are clashing, no? that's what the link seemed to suggest to me, anyway, that they are all fighting each other

Ah yes, I see. It's because :

The elm-pages build command is running two parallel compilation tasks for the client and worker apps creating a race condition.

Dillon says it's been fixed but not sure if that means in v2 or v3. We also need to cut back on event window because there was a recent timeout fail on the whole build.

katjam commented 2 weeks ago

Looks like that update happened in v3.0 - so for now I'll fix the event window import #391 and we should do an experiment with upgrading to v3.0 when we integrate Manchester because it seems like it'll address a couple things for us.