Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

AR-1689 Fix crashing chromatic builds #179

Merged
merged 1 commit into from
Jun 4, 2020
Merged

Conversation

justinanastos
Copy link
Contributor

@justinanastos justinanastos commented Jun 4, 2020

Chromatic started crashing in CI. My instincts tell me it's a memory issue since we've seen this in the past. Use a larger container class and notify node that it can use 6GB of memory instead of 4.

Resolves AR-1689


This is a stopgap solution until we update to Storybook v6 which supposedly resolves the bug causing such high memory usage storybookjs/storybook#7743. I spent an hour unsuccessfully trying to upgrade to the latest beta. This will have to hold us over.

@justinanastos justinanastos force-pushed the justin/fix-chromatic branch from bad283d to 324d4fb Compare June 4, 2020 17:10
@justinanastos justinanastos added the internal Changes only affect the internal API label Jun 4, 2020
@justinanastos justinanastos changed the title Allocate 6gb to chromatic build AR-1689 Fix crashing chromatic builds Jun 4, 2020
@justinanastos justinanastos force-pushed the justin/fix-chromatic branch from 324d4fb to ab75f82 Compare June 4, 2020 17:20
@cheapsteak
Copy link
Member

cheapsteak commented Jun 4, 2020

can we disable minification instead? should fix the issue for netlify as well nvm, re-upping node size works :)

@justinanastos
Copy link
Contributor Author

can we disable minification instead? should fix the issue for netlify as well

@cheapsteak

I don't believe the issue is minification @cheapsteak . This issue is coming from react-docgen-typescript-loader where it just uses incredible amounts of memory.

@justinanastos justinanastos merged commit b89facc into master Jun 4, 2020
@justinanastos justinanastos deleted the justin/fix-chromatic branch June 4, 2020 17:29
@apollo-bot2
Copy link
Collaborator

🚀 PR was released in v5.2.0 🚀

@apollo-bot2 apollo-bot2 added the released This issue/pull request has been released. label Jun 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
internal Changes only affect the internal API released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants