Skip to content

fix(build): Run integration bundle builds sequentially #4740

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 21, 2022

Conversation

lobsterkatie
Copy link
Member

@lobsterkatie lobsterkatie commented Mar 21, 2022

When run in CI, the parallelized integration bundle builds are being flaky, and sometimes seeming to timeout before they complete. (They don't output a timeout message, but the ones that succeed take an extraordinarily long time, and others just log as killed. Because of the parallelization, of course, it's impossible to tell which builds are killed, but the end result is that not all bundles get build and included in the artifact upload.) I can only speculate that this is because the GHA machine is trying to do too many things at once and runs out of memory.

This therefore removes the parallelization, by not backgrounding each build job as it starts, but instead running each one in turn, which should result in all bundles getting built.

Note: This significantly increases build time in CI. In a future PR, the integration bundle builds can get split over a number of different GHA jobs, which will restore the parallelization, without asking any one job to handle more than it's capable of.

@github-actions
Copy link
Contributor

size-limit report

Path Base Size (ebc9da5) Current Size Change
@sentry/browser - ES5 CDN Bundle (gzipped + minified) 19.49 KB 19.49 KB +0.02% 🔺
@sentry/browser - ES5 CDN Bundle (minified) 62.17 KB 62.17 KB 0%
@sentry/browser - ES6 CDN Bundle (gzipped + minified) 18.11 KB 18.11 KB +0.01% 🔺
@sentry/browser - ES6 CDN Bundle (minified) 55.5 KB 55.5 KB 0%
@sentry/browser - Webpack (gzipped + minified) 22.6 KB 22.6 KB 0%
@sentry/browser - Webpack (minified) 79.21 KB 79.21 KB 0%
@sentry/react - Webpack (gzipped + minified) 22.63 KB 22.63 KB 0%
@sentry/nextjs Client - Webpack (gzipped + minified) 47.6 KB 47.6 KB 0%
@sentry/browser + @sentry/tracing - ES5 CDN Bundle (gzipped + minified) 25.36 KB 25.36 KB +0.01% 🔺
@sentry/browser + @sentry/tracing - ES6 CDN Bundle (gzipped + minified) 23.72 KB 23.72 KB +0.01% 🔺

Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for the time being

@AbhiPrasad AbhiPrasad merged commit cb0f97d into master Mar 21, 2022
@AbhiPrasad AbhiPrasad deleted the kmclb-unparallelize-building-integration-bundles branch March 21, 2022 15:06
@AbhiPrasad AbhiPrasad added this to the Pre 7.0.0 Work milestone Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants