You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've made quite a few changes and updates to CI and it's been a bit of a pain a couple of times. Different jobs but for the same platforms keep interfering, i.e.
The job that compiles the Android app fails. This causes the iOS device tests to not run because they depend on the job to finish successfully.
Package Validation. You're making changes to the package contents and CI won't run because Package Validation is be base for all jobs.
The Proposal
Instead, and to get rid of all the if: ${{ matrix.platform == 'Android' }} and similar checks, we could split the jobs into something like the following chart
See what can be parallelized. Some jobs do quite a bit of setup before doing any actual work, while depending on the output of previous jobs. We could utilize @vaind's download action like we do here to pause.
Merge the Android build back into one job.
The main pain-point is all the manual work we have to do to keep the lights on.
Without changing much, we could merge the compilation back into the build step. This way we could still keep the benefit of minimal Unity Editor time (helps with the build license pressure) but still rely on the local Android SDK & NDK setup the Unity installation comes with.
Misc Info
Build Licenses
Build-License overlap would still exist but is already prevalent due to our use of matrix to target multiple platforms with one job.
Additional context
The average times for the individual jobs are:
Build Unity SDK: 5 min
Create Smoke Test: 2 min
Building
Android: 10 min
iOS: 6 min
WebGL: 13 min
Linux: 11 min
Windows: 30 min
Compiling SmokeTest
Android: 5 min
iOS: 4 min
Running Device Test (mobile)
Android: 5 min
iOS: 5 min
The text was updated successfully, but these errors were encountered:
The Problem
I've made quite a few changes and updates to CI and it's been a bit of a pain a couple of times. Different jobs but for the same platforms keep interfering, i.e.
The Proposal
Instead, and to get rid of all the
if: ${{ matrix.platform == 'Android' }}
and similar checks, we could split the jobs into something like the following chartCurrent setup
Proposed setup
The Plan
ci.yml
into reusable workflows, depending on their respective target-platform. feat: Separate CI jobs into workflows #19892022.3.55f1
#1957 (review)sentry-unity/.github/workflows/ci.yml
Line 481 in 123b270
Misc Info
Build Licenses
Build-License overlap would still exist but is already prevalent due to our use of
matrix
to target multiple platforms with one job.Additional context
The average times for the individual jobs are:
The text was updated successfully, but these errors were encountered: