Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CI: Touch up CircleCI configuration #2764

Merged
merged 2 commits into from
Apr 20, 2022
Merged

CI: Touch up CircleCI configuration #2764

merged 2 commits into from
Apr 20, 2022

Conversation

mgxd
Copy link
Collaborator

@mgxd mgxd commented Apr 19, 2022 •
edited
Loading

Slightly rework the circleci config to:

  • Use YAML anchors to avoid obvious duplication
  • Use large machines (4 vCPUs, 15GB RAM) (we could also consider using xlarge - 8/32GB)
  • Leverage docker_layer_caching (decent speedup!)

Closes #2759

@mgxd mgxd marked this pull request as ready for review April 19, 2022 20:48
@mgxd
Copy link
Collaborator Author

mgxd commented Apr 19, 2022

upgrading to large machines has significantly decreased the dsXXX runtimes, so we can punt that refactor for a later time.

@effigies effigies merged commit 7f68f59 into master Apr 20, 2022
@effigies effigies deleted the ci/update branch April 20, 2022 11:23
@effigies effigies added this to the 22.0.0 milestone May 17, 2022
@mgxd mgxd mentioned this pull request Jul 28, 2022
Closed
@eilidhmacnicol eilidhmacnicol mentioned this pull request May 11, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@effigies effigies effigies approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
22.0.0
Development

Successfully merging this pull request may close these issues.

Clean up testing suite
2 participants
@mgxd @effigies