Skip to content

Conversation

aykevl
Copy link
Member

@aykevl aykevl commented Apr 6, 2021

This job is causing OOM errors on CircleCI so limit it to just two jobs
(which should be fine on a 2CPU executor). Hopefully this fixes the
errors in CI that have occured recently.

This job is causing OOM errors on CircleCI so limit it to just two jobs
(which should be fine on a 2CPU executor). Hopefully this fixes the
errors in CI that have occured recently.
@aykevl aykevl requested a review from deadprogram April 6, 2021 10:21
@deadprogram
Copy link
Member

Looks good, now merging.

Thanks @aykevl!

@deadprogram deadprogram merged commit f880950 into dev Apr 6, 2021
@deadprogram deadprogram deleted the ci-limit-jobs branch April 6, 2021 10:57
aykevl added a commit that referenced this pull request Apr 6, 2021
Instead of the regular build, it's the `make test` line that fails due
to OOM. This is because testing means that a lot of test binaries need
to be built while the regular build only needs to link one binary.

This improves #1774 and should
hopefully actually fix the OOM errors.
aykevl added a commit that referenced this pull request Apr 6, 2021
Instead of the regular build, it's the `make test` line that fails due
to OOM. This is because testing means that a lot of test binaries need
to be built while the regular build only needs to link one binary.

This improves #1774 and should
hopefully actually fix the OOM errors.
deadprogram pushed a commit that referenced this pull request Apr 7, 2021
Instead of the regular build, it's the `make test` line that fails due
to OOM. This is because testing means that a lot of test binaries need
to be built while the regular build only needs to link one binary.

This improves #1774 and should
hopefully actually fix the OOM errors.
deadprogram pushed a commit that referenced this pull request Apr 9, 2021
Instead of the regular build, it's the `make test` line that fails due
to OOM. This is because testing means that a lot of test binaries need
to be built while the regular build only needs to link one binary.

This improves #1774 and should
hopefully actually fix the OOM errors.
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.

2 participants