Skip to content

Reduce number of CircleCI workers to 4#1447

Merged
christopherbunn merged 3 commits into
mainfrom
cb_test_circleci_machine
Nov 24, 2020
Merged

Reduce number of CircleCI workers to 4#1447
christopherbunn merged 3 commits into
mainfrom
cb_test_circleci_machine

Conversation

@christopherbunn

@christopherbunn christopherbunn commented Nov 19, 2020

Copy link
Copy Markdown
Contributor

To reduce memory utilization but retain some speedup, this PR reduces the number of CircleCI workers to 4.

@codecov

codecov Bot commented Nov 19, 2020

Copy link
Copy Markdown

Codecov Report

Merging #1447 (f1173de) into main (a5e17f8) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1447   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files         223      223           
  Lines       14930    14930           
=======================================
  Hits        14923    14923           
  Misses          7        7           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a5e17f8...f1173de. Read the comment docs.

@christopherbunn
christopherbunn force-pushed the cb_test_circleci_machine branch 2 times, most recently from 1448442 to 32453e0 Compare November 23, 2020 16:14
@christopherbunn christopherbunn changed the title Optimizing CircleCI runs Reduced number of CircleCI workers to 4 Nov 23, 2020
@christopherbunn
christopherbunn marked this pull request as ready for review November 23, 2020 16:24
@christopherbunn christopherbunn changed the title Reduced number of CircleCI workers to 4 Reduce number of CircleCI workers to 4 Nov 23, 2020

@dsherry dsherry left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍 yep! 🚢

@christopherbunn
christopherbunn force-pushed the cb_test_circleci_machine branch from e7dced3 to f1173de Compare November 24, 2020 16:09
@christopherbunn
christopherbunn merged commit 67b534f into main Nov 24, 2020
@dsherry dsherry mentioned this pull request Nov 24, 2020
@freddyaboulton
freddyaboulton deleted the cb_test_circleci_machine branch May 13, 2022 14:58
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