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.

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

CircleCI todos #15855

Closed
ide opened this issue Sep 7, 2017 · 10 comments
Closed

CircleCI todos #15855

ide opened this issue Sep 7, 2017 · 10 comments
Labels
Ran Commands One of our bots successfully processed a command. Resolution: Locked This issue was locked by the bot. 🔩Test Infrastructure Test infrastructure and continuous integration. Type: Discussion Long running discussion.

Comments

@ide
Copy link
Contributor

ide commented Sep 7, 2017

Just making a note of a few things to look at after we get fully running on CircleCI 2.

@hramos
Copy link
Contributor

hramos commented Sep 8, 2017

Currently I'm leaning towards not caching the NDK, as you said it's slower than downloading and extracting the archives.

To add to your list:

  • Look into using a custom image with various dependencies set up ahead of time

The custom image could be set up ahead of time with the Android SDKs and NDK, but as you pointed out in the PR keeping the image up to date may be more trouble than it's worth. Some work has already gone into setting up such an image, see the ContainerShip/ folder in this repo.

@hramos
Copy link
Contributor

hramos commented Sep 9, 2017

Once the test APK is built and native libraries are compiled, we may be able to fan out into three separate jobs that run unit tests, integration tests, and end to end tests in parallel, if that speeds up things. For these three, we may be able to use the basic android docker images, installing node on the last two as needed.

@sibelius
Copy link

resource_class: large

it is working fine on our projects

@pull-bot
Copy link

@facebook-github-bot no-template

@facebook-github-bot
Copy link
Contributor

Hey, thanks for reporting this issue! It looks like your description is missing some necessary information, or the list of reproduction steps is not complete. Can you please add all the details specified in the Issue Template? This is necessary for people to be able to understand and reproduce the issue being reported. I am going to close this, but feel free to open a new issue with the additional information provided. Thanks! See "What to Expect from Maintainers" to learn more.

@facebook-github-bot facebook-github-bot added the Ran Commands One of our bots successfully processed a command. label Oct 10, 2017
@hramos
Copy link
Contributor

hramos commented Oct 10, 2017

Re-opening as this is a discussion task.

@hramos hramos reopened this Oct 10, 2017
@hramos hramos added Type: Discussion Long running discussion. and removed Needs more information labels Oct 10, 2017
@hramos
Copy link
Contributor

hramos commented Oct 13, 2017

@grabbou just opened a PR to move over iOS/tvOS tests from Travis to Circle: #16354

@douglowder
Copy link
Contributor

@hramos is Circle CI robust enough that we can actually run the unit and integration tests on tvOS? At the moment I believe we're just testing that tvOS compiles....

@grabbou
Copy link
Contributor

grabbou commented Jan 4, 2018

@dlowder-salesforce I think yeah, there's full OS X machine with different simulators, so no reason why it wouldn't be possible.

I think it's even advised as we are hitting few tvOS bugs from time to time :)

@grabbou
Copy link
Contributor

grabbou commented Jan 4, 2018

@hramos what's the status of this issue? Anything left to get this closed?

@hramos hramos added the 🔩Test Infrastructure Test infrastructure and continuous integration. label Apr 9, 2018
@hramos hramos closed this as completed Jan 25, 2019
@facebook facebook locked as resolved and limited conversation to collaborators Jan 25, 2020
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Jan 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Ran Commands One of our bots successfully processed a command. Resolution: Locked This issue was locked by the bot. 🔩Test Infrastructure Test infrastructure and continuous integration. Type: Discussion Long running discussion.
Projects
None yet
Development

No branches or pull requests

8 participants