Skip to content

Commit

Permalink
chore(ci): check e2e main commits against Android API level 27 (#6031)
Browse files Browse the repository at this point in the history
### Description

We want CI to check the app against the highest and lowest supported
Android API levels. The highest supported API level is being checked in
the PR CI. The lowest level should be checked after merging the PR into
a main branch.

Currently, the lowest API level we support is 24, but the levels 24, 25,
and 26 are failing the CI for some reason.

Setting Android API level 27 as the lowest one for now.

* 24 is failing due to Let's Encrypt root certificate expiration
* 25 is failing (RET-1274 to investigate)
* 26 is failing (RET-1275 to investigate)

### Test plan

CI succeeds, like in this workflow:
76f5859

### Related issues

- Fixes RET-1194

### Backwards compatibility

NA

### Network scalability

NA

---------

Co-authored-by: mpgaarciaa <[email protected]>
Co-authored-by: Kathy Luo <[email protected]>
Co-authored-by: Finnian Jacobson-Schulte <[email protected]>
Co-authored-by: valora-bot-crowdin <[email protected]>
Co-authored-by: Valora Bot <[email protected]>
Co-authored-by: Anka <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Jacob Waterman <[email protected]>
Co-authored-by: Myroslav Sviderok <[email protected]>
Co-authored-by: valora-bot <[email protected]>
Co-authored-by: Jean Regisser <[email protected]>
Co-authored-by: Anka <[email protected]>
Co-authored-by: Satish Ravi <[email protected]>
  • Loading branch information
14 people authored Dec 12, 2024
1 parent 33558ee commit b40e2ef
Showing 1 changed file with 15 additions and 18 deletions.
33 changes: 15 additions & 18 deletions .github/workflows/e2e-main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,24 +22,21 @@ concurrency:
cancel-in-progress: true

jobs:
# TODO: enable once we have at least one android-api-level below
# android:
# name: Android
# strategy:
# max-parallel: 2
# fail-fast: false
# matrix:
# # TODO if/when more CI machines: enable Android sdk 21, 23, 29 and 31
# # 21 is failing 9 specs
# # 23 is failing ? spec(s)
# # 31 is failing ? spec(s)
# # 24 is currently failing all the time, disabling it for now
# # 30 is not included as it runs on the merge queue
# android-api-level: []
# uses: ./.github/workflows/e2e-android.yml
# with:
# android-api-level: ${{ matrix.android-api-level }}
# secrets: inherit
android:
name: Android
strategy:
max-parallel: 2
fail-fast: false
matrix:
# min supported API level is 24
# 24 is failing due to Let's Encrypt root certificate expiration
# 25 is failing (RET-1274)
# 26 is failing (RET-1275)
android-api-level: ['27']
uses: ./.github/workflows/e2e-android.yml
with:
android-api-level: ${{ matrix.android-api-level }}
secrets: inherit
ios:
name: iOS
strategy:
Expand Down

0 comments on commit b40e2ef

Please sign in to comment.