Commit Graph

21 Commits

Author SHA1 Message Date
Pascal Hartig
3129250a12 Fix CircleCI maven snapshot build (#4257)
Summary:
It somehow broke with 16983deaaf and failed with a fairly unspecific error message: https://app.circleci.com/pipelines/github/facebook/flipper/11548/workflows/5f114b29-edd8-4a12-97c0-97e2b6e2aef3/jobs/6136

This fixes it and also upgrades the build to make use of the new "orb" system Circle CI provides.

Pull Request resolved: https://github.com/facebook/flipper/pull/4257

Reviewed By: mweststrate

Differential Revision: D40641195

Pulled By: passy

fbshipit-source-id: ca99a6063ff4aca97510ea8b65f6d466f86a72fd
2022-10-25 02:57:26 -07:00
Pascal Hartig
7763b80adc Revert D34818577: Remove checkout step
Differential Revision:
D34818577 (16ab0b7148)

Original commit changeset: 1f34ab1d1830

Original Phabricator Diff: D34818577 (16ab0b7148)

fbshipit-source-id: b21c3692e63c309b9ff4a5afcea1260b37d3ff31
2022-03-11 06:02:53 -08:00
Pascal Hartig
16ab0b7148 Remove checkout step (#3519)
Summary:
According to [this](https://circleci.com/docs/2.0/configuration-reference/#checkout), using the checkout step uses SSH but we're fine with HTTPS.

Pull Request resolved: https://github.com/facebook/flipper/pull/3519

Reviewed By: lblasa

Differential Revision: D34818577

Pulled By: passy

fbshipit-source-id: 1f34ab1d183081125607e6d2020bc7a1c7c34685
2022-03-11 05:29:43 -08:00
Pascal Hartig
da9511a5bf Fix CI build for Android (#2836)
Summary:
I've been really sweating about this one. It looks like Google now removed NDK 21 as it's too old. However, we've been struggling with the upgrade because OpenSSL was built against an old version of the NDK/glibc/LLVM/some other stuff.

I've now managed to create an OpenSSL distribution for 1.1.1k (we had 1.1.0h before) that seems to build with this after some small modifications.

This seems to do the trick, but I wouldn't be shocked if we found some more incompatibilities further down the line.

Pull Request resolved: https://github.com/facebook/flipper/pull/2836

Test Plan:
- Locally: `./gradlew :tutorial:installDebug`. Builds, starts up. Cool.
- Public GitHub CI: Happy.
- Circle CI: Only triggers post-land. We'll see. But the setup is simple, so hopefully it should work there, too.
- Internal CI: Waiting for signal.

Reviewed By: fabiomassimo

Differential Revision: D30839209

Pulled By: passy

fbshipit-source-id: efe599f28cc0edfdf2149f905c3483555239edc0
2021-09-09 08:46:59 -07:00
Pascal Hartig
781df776f2 Rename default branch references to main (#2689)
Summary:
I really hope I hit everything and didn't break anything that wasn't
referring to branch names.

Pull Request resolved: https://github.com/facebook/flipper/pull/2689

Test Plan:
_eyes

CI

|[Site Preview: flipper](https://our.intern.facebook.com/intern/staticdocs/eph/D30305789/V2/flipper/)

Reviewed By: timur-valiev

Differential Revision: D30305789

Pulled By: passy

fbshipit-source-id: 5daaa09250bb96bb50be679fc01dae86c666eb73
2021-08-13 15:00:39 -07:00
Pascal Hartig
87d7bcf274 Remove JCenter specifics (#1914)
Summary:
This removes Bintray/JCenter specific code and replaces it with a new plugin recommended by [Chris Banes](https://chris.banes.dev/publishing-to-maven-central/).

Pull Request resolved: https://github.com/facebook/flipper/pull/1914

Test Plan:
Manually uploaded as snapshot and full release, but with this setup still requires all sorts of manual setup, including GPG. Next PR will include automation for this.

CI here should also verify that I didn't split this diff up incorrectly.

Reviewed By: jknoxville

Differential Revision: D26367350

Pulled By: passy

fbshipit-source-id: faa6b488d6c95bc643d2f9328362e29fb4b05ded
2021-02-10 07:16:32 -08:00
Pascal Hartig
b5e613141e Set up automated retry (#1714)
Summary:
Circle occasionally fails with a network error in one of the Android dependencies. This uses a simple retry to alleviate the need to manually restart the tasks.

Pull Request resolved: https://github.com/facebook/flipper/pull/1714

Test Plan: Ran this on my private fork: https://app.circleci.com/pipelines/github/passy/flipper-1/9/workflows/e1b40086-1dce-49eb-a178-81052145ed0f

Reviewed By: nikoant

Differential Revision: D25170790

Pulled By: passy

fbshipit-source-id: 220759fb227855eac807d64f41c78f5781599515
2020-11-27 09:11:19 -08:00
Pascal Hartig
4550726a99 Fix Android build (#1503)
Summary:
This started failing in early August when this was deployed:
https://support.circleci.com/hc/en-us/articles/360048152911-Java-Builds-Fail-With-Unrecognized-VM-option-UseCGroupMemoryLimitForHeap-

Pull Request resolved: https://github.com/facebook/flipper/pull/1503

Test Plan: CI

Reviewed By: priteshrnandgaonkar

Differential Revision: D23373472

Pulled By: passy

fbshipit-source-id: 94c185293f1abf3831e3ea2f2f09e59e434ade93
2020-08-27 09:38:49 -07:00
Pritesh Nandgaonkar
6f9d82117e Publish pods gh (#1083)
Summary:
This PR adds a github action to publish pods on a new tag. It also tries to fix a race condition which happens when we try to publish FlipperKit pod. During the release of FlipperKit pod, we rely on the newly published Flipper Pod, and sometimes the release of Flipper Pod takes a time to get updated in the cocoapods trunk.

This is a known issue by the Cocoapods and they will be shipping a fix to this in Cocoapods 1.10. Till then we can keep retrying for 5 times.

Ref: https://github.com/CocoaPods/CocoaPods/issues/9502#issuecomment-579486258

## Changelog

- Adds Github actions to publish cocoapods.
- Removes the Circle CI config to publish on cocoapods
Pull Request resolved: https://github.com/facebook/flipper/pull/1083

Test Plan: I tested this action by running it on this PR and it worked fine.

Reviewed By: jknoxville

Differential Revision: D21348084

Pulled By: priteshrnandgaonkar

fbshipit-source-id: f33f3cdef43ae032ad42c94db72323212e40ec67
2020-05-01 09:07:02 -07:00
Pascal Hartig
81b2487b8a Attempt to build with newer SDK/NDK (#543)
Summary:
Current builds are failing and I'm not quite sure why.
Pull Request resolved: https://github.com/facebook/flipper/pull/543

Reviewed By: jknoxville

Differential Revision: D17364947

Pulled By: passy

fbshipit-source-id: 07c418e340cabb70806c8f73b4eb867ba7e0fc4b
2019-09-13 08:56:41 -07:00
Pritesh Nandgaonkar
07f327f46b Automate iOS release (#445)
Summary:
This PR sets up Circle CI to publish pods when github release is made. This PR also updates the Flipper-Folly pod as it didn't build on 32 bit architecture, because `thread_local` was not supported.

Automated publish of cocoapods through Circle CI.
Pull Request resolved: https://github.com/facebook/flipper/pull/445

Reviewed By: danielbuechele

Differential Revision: D15333975

Pulled By: priteshrnandgaonkar

fbshipit-source-id: 7db68efeeebbe344213917f10f9f3ce4132e8cbc
2019-05-14 07:24:10 -07:00
pavlospt
b8dbbb0313 Properly fix license issue in CircleCI (#362)
Summary:
Apparently this seems to be the only way to make build work, with the updated licenses. It also follows the same strategy as Travis.

I am using double pipe in `sdkmanager` command execution, to avoid breaking the build, because `sdkmanager` returns exit code 141 in case it meets the requirements requested. (e.g.: `yes | sdkmanager --licenses` returns 141, which in turn breaks the build due to `set -eo pipefail` Bash flags, that CircleCI is using in order to execute any scripts)

For the record, I tried the following solutions, that did not work:
* Add accepted license SHA in Android SDK licenses folder.
* Accept all licenses prior to running the other jobs (with a separate job, working as a dependency to the rest of the Workflow jobs) (`yes | sdkmanager --licenses`)
* Accept all licenses prior to executing any Gradle command (without a separate job) (`yes | sdkmanager --licenses`)
Pull Request resolved: https://github.com/facebook/flipper/pull/362

Differential Revision: D13817430

Pulled By: passy

fbshipit-source-id: d8c95fc5b9202306dd50ea103b6e90142e73ef61
2019-01-25 05:26:31 -08:00
Pavlos-Petros Tournaris
682b9bda57 Typo in CircleCI/config.yml (#361)
Summary:
My bad, forgot the `steps` key.
Pull Request resolved: https://github.com/facebook/flipper/pull/361

Differential Revision: D13760713

Pulled By: passy

fbshipit-source-id: 7650644827dbd9919681cd16a72ccf0140f59446
2019-01-22 02:30:37 -08:00
pavlospt
51c496ddef Add accepted Android SDK license (#359)
Summary:
Android SDK license SHA has been updated as of January 16, 2019 so it needs to be re-accepted. This is only needed for CircleCI since in Travis setup `sdkmanager` is used in order to accept the Licenses.
Pull Request resolved: https://github.com/facebook/flipper/pull/359

Differential Revision: D13751841

Pulled By: passy

fbshipit-source-id: f3d84277f06bedf06b7a766429d9f7e3275bb516
2019-01-21 10:33:39 -08:00
Pascal Hartig
6c3a6c0d66 Upgrade to Circle 2.1 (#289)
Summary:
Use CircleCI 2.1 for common executors and remove the common build step
as this is already done by Travis. Instead, focus on publishing either
snapshots or releases.
Pull Request resolved: https://github.com/facebook/flipper/pull/289

Reviewed By: jknoxville

Differential Revision: D10216413

Pulled By: passy

fbshipit-source-id: cfbc52c43187339ae6cb1c486d81e7c40216e86b
2018-10-05 07:59:19 -07:00
Pascal Hartig
d69fb9e6a1 Separate snapshot publish logic (#288)
Summary:
Snapshots should only ever be published off of master.
Pull Request resolved: https://github.com/facebook/flipper/pull/288

Reviewed By: danielbuechele

Differential Revision: D10161974

Pulled By: passy

fbshipit-source-id: dca8463ff80de9d407758405c24e6511bef534d1
2018-10-04 03:42:26 -07:00
Pascal Hartig
e98d4821fd Attempt to fix config-error (#270)
Summary:
Per title. Not very helpful error here:

https://circleci.com/workflow-run/4df8960b-e74e-4c58-8072-ac9aad94c063
Pull Request resolved: https://github.com/facebook/flipper/pull/270

Reviewed By: jknoxville

Differential Revision: D9861448

Pulled By: passy

fbshipit-source-id: 71f46ce0bc3b9fe98ac9a7abecc6fb9c3a531f59
2018-09-17 09:58:01 -07:00
Pascal Hartig
b342b07a89 Add branch filter for builds (#268)
Summary:
Nothing seems to run now, so let's try "trigger schedules" instead.
Pull Request resolved: https://github.com/facebook/flipper/pull/268

Reviewed By: priteshrnandgaonkar

Differential Revision: D9851086

Pulled By: passy

fbshipit-source-id: 8327e0f7b2a416541e8b531578ea813cc168ff66
2018-09-17 07:12:00 -07:00
Pascal Hartig
474a36c0e2 Set up automatic release publishing (#266)
Summary:
On tags, build and publish artifacts to bintray. The encrypted accounted creds are for a `flipperfbbot` account on Bintray I created for this.
Pull Request resolved: https://github.com/facebook/flipper/pull/266

Reviewed By: priteshrnandgaonkar

Differential Revision: D9848181

Pulled By: passy

fbshipit-source-id: cf9150b2352a9ce972c48533ce15e8e75aa908fe
2018-09-17 01:57:48 -07:00
Pascal Hartig
cf3cb0d08f Use CircleCI for SNAPSHOT deployment (#260)
Summary:
Per title. It actually works!
Pull Request resolved: https://github.com/facebook/flipper/pull/260

Reviewed By: jknoxville

Differential Revision: D9633320

Pulled By: passy

fbshipit-source-id: 10ee856977fa0a337972a472d41eb7a83075c39b
2018-09-04 04:28:20 -07:00
Pascal Hartig
a26cc6db9b Initial setup for CircleCI (#257)
Summary:
This is so much faster! This should allow us to more easily publish snapshots.
Pull Request resolved: https://github.com/facebook/flipper/pull/257

Reviewed By: danielbuechele

Differential Revision: D9630067

Pulled By: passy

fbshipit-source-id: 64c633933a971cc15371d2ee90c184e13b98e06a
2018-09-03 07:43:52 -07:00