Skip to content

ci: post scheduled scene-routing E2E failures to #squad-mobile-alerts #679

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.

Sign up for GitHub

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

Merged
mahmoud-elmorabea merged 3 commits into from
Oct 23, 2026
Merged

ci: post scheduled scene-routing E2E failures to #squad-mobile-alerts #679

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3d6a3f3
ci: post scheduled scene-routing E2E failures to #squad-mobile-alerts
mahmoud-elmorabea Oct 22, 2026
da560dc
ci: reserve time for scene E2E failure alerts
mahmoud-elmorabea Oct 23, 2026
05a7fda
Merge remote-tracking branch 'origin/main' into HEAD
mahmoud-elmorabea Oct 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions .github/workflows/react-native-scene-e2e.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,30 +23,37 @@ jobs:
route-notifications:
name: RN scene notification routing on Xcode 27
runs-on: xcode-27
# Step budgets total 57 minutes, leaving room before the job cancellation.
# A step timeout fails the job while evidence and notification steps can still run.
timeout-minutes: 60

steps:
- name: Check out React Native SDK
timeout-minutes: 2
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2

- name: Verify Xcode 27 toolchain
timeout-minutes: 1
uses: customerio/mobile-ci-tools/github-actions/ios/report-toolchain/v1@main
with:
expected-xcode-major: '27'
expected-ios-sdk-major: '27'

- name: Set up Node
timeout-minutes: 3
uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
with:
node-version: '24'
cache: npm

- name: Set up Ruby
timeout-minutes: 3
uses: ruby/setup-ruby@95ef2b042f9d7a56d8268cba8559e2842e2ad01b # v1.321.0
with:
ruby-version: '3.4'

- name: Install Maestro
timeout-minutes: 3
shell: bash
run: |
set -euo pipefail
Expand All @@ -61,16 +68,26 @@ jobs:
echo "$maestro_dir/maestro/bin" >> "$GITHUB_PATH"

- name: Test warm and terminated notification routing
timeout-minutes: 40
shell: bash
env:
E2E_IOS_RUNTIME_MAJOR: '27'
run: .maestro/run_scene_push.sh

- name: Upload Maestro diagnostics
timeout-minutes: 3
if: always()
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
with:
name: react-native-scene-maestro-diagnostics
path: ${{ runner.temp }}/react-native-scene-maestro-*
if-no-files-found: ignore
retention-days: 7

- name: Notify #squad-mobile-alerts of a scheduled failure
timeout-minutes: 2
if: (failure() || cancelled()) && github.event_name == 'schedule'

@cursor cursor Bot Oct 23, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Notify on non-test failures

Medium Severity

The Slack notify step uses failure(), which is true when any earlier step failed. On a scheduled run, Maestro diagnostics upload runs with if: always() after the test step. If the E2E test succeeds but upload fails, the workflow still posts a scheduled scene-routing failure to #squad-mobile-alerts.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 05a7fda. Configure here.

uses: customerio/mobile-ci-tools/github-actions/slack-notify-scheduled-failure/v1@main
with:
slack_webhook_url: ${{ secrets.SLACK_WEBHOOK_URL }}
label: scene notification routing
Loading