Closed absoludity closed 1 year ago
One down: the memory issue for our dashboard lint was simply because the number of js/ts files was increasing. The simple solution was to stop linting generated code, so in #6031 I added an --ignore-pattern='src/gen/'
which has fixed the issue (reducing the memory requirement for now). I'm not sure why we're linting/prettifying generated code in the first place, let me know if I missed an important reason :)
For reasons, the javascript heap error is happening much more frequently the last day or two. I'll try to investigate tomorrow.
After some investigation, JS error fixed (for now) with #6459
Summary Currently there are two spurious failures that appear to be getting regular enough that we should fix them, as they can result in our Main Pipeline badge being red.
There is also a failure on the full integration pipeline, because it tries to create a PR for the latest upstream chart changes sync, but one already exists (ie. because we haven't yet merged it), this step fails, causing the full integration pipeline to fail.
Background and rationale We don't want red failing badges on our main README.
Description
The four failures I'm aware of, 3 spurious, one not, are:
Javascript Heap out of memory while running linter
Example
Error rolling out flux-system during flux e2e tests
Example
From the output that follows, the deployment has gone ahead, but the liveness and readiness checks are failing:
Error rolling out e2e-runner
Example
Oddly, I can't see anything wrong with the output (all successful)
sync-chart-changes already exists
Example
If we're not quick to merge the generated sync charts, the full integration pipeline fails with:
Dex service fails when setting up multicluster
Saw another one today:
Acceptance criteria A formalized list of conditions that ensure that the issue can be considered as finished.
Additional context Add any other context or screenshots about the issue here.