Skip to content

[CI] Cache CocoaPods specs-repo#25095

Closed
hramos wants to merge 2 commits into
facebook:masterfrom
hramos:cocoapods-caching
Closed

[CI] Cache CocoaPods specs-repo#25095
hramos wants to merge 2 commits into
facebook:masterfrom
hramos:cocoapods-caching

Conversation

@hramos
Copy link
Copy Markdown
Contributor

@hramos hramos commented May 30, 2019

Summary

The test_end_to_end job has been timing out due CocoaPods taking too long to check out the Specs repo. This repo is stored at ~/.cocoapods, therefore by caching this folder we can keep the individual e2e test run from exceeding 10 minutes w/o output.

Changelog

[Internal] [Added] - Cache CocoaPods

Test Plan

CI: https://circleci.com/gh/hramos/react-native/6155

With empty cache:

Screen Shot 2019-05-30 at 12 54 04 PM

After cache is ready:

Screen Shot 2019-05-30 at 1 30 16 PM

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner RN Team labels May 30, 2019
Comment thread .circleci/config.yml Outdated
@react-native-bot react-native-bot added the Type: Enhancement A new feature or enhancement of an existing feature. label May 30, 2019
@hramos hramos force-pushed the cocoapods-caching branch from 2d9e48f to cd227af Compare May 30, 2019 22:44
@hramos hramos force-pushed the cocoapods-caching branch 15 times, most recently from 8a44e4b to b125ea4 Compare May 31, 2019 22:07
@hramos hramos force-pushed the cocoapods-caching branch from b125ea4 to a24f98c Compare May 31, 2019 22:20
Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @hramos in 4a6676d.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label May 31, 2019
@hramos hramos deleted the cocoapods-caching branch June 6, 2019 20:41
M-i-k-e-l pushed a commit to M-i-k-e-l/react-native that referenced this pull request Mar 10, 2020
Summary:
The `test_end_to_end` job has been timing out due CocoaPods taking too long to check out the Specs repo. This repo is stored at `~/.cocoapods`, therefore by caching this folder we can keep the individual e2e test run from exceeding 10 minutes w/o output.

## Changelog

[Internal] [Added] - Cache CocoaPods
Pull Request resolved: facebook#25095

Differential Revision: D15587702

Pulled By: hramos

fbshipit-source-id: 6669ff09a5021f012ac8a829db70442d8f7f07e8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. p: Facebook Partner: Facebook Partner RN Team Type: Enhancement A new feature or enhancement of an existing feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants