Skip to content

ci: remove OTA tag creation step#30813

Merged
weitingsun merged 4 commits into
mainfrom
wsun/remove-ota-production-tag-creation
May 29, 2026
Merged

ci: remove OTA tag creation step#30813
weitingsun merged 4 commits into
mainfrom
wsun/remove-ota-production-tag-creation

Conversation

@weitingsun

@weitingsun weitingsun commented May 29, 2026

Copy link
Copy Markdown
Contributor

Description

Remove OTA publishing and release-tag creation from the automated/Runway CI flows. OTA updates are now published and tagged manually in Runway, so CI only needs to detect OTA bumps (to skip native builds) and otherwise build.

Changes

auto-rc-ota-build-core.yml (Auto RC build/OTA core)

  • Removed the trigger-ota job (no longer pushes EAS OTA updates via push-eas-update.yml).
  • Removed the validate-ota-pr job (only existed to guard the OTA push).
  • Removed the create-ota-production-tag job and the create_production_ota_tag + ota_channel inputs.
  • Kept OTA detection: resolve-context still runs and trigger-build remains gated on ota_bump != 'true', so OTA-only branches skip the native build instead of building.
  • Tightened permissions from contents: write + actions: write + pull-requests: read + id-token: write down to contents: read + pull-requests: read + id-token: write.

runway-ota-production.yml (Runway OTA Production)

  • Removed the create-ota-production-tag job — the v<OTA_VERSION> tag is now created manually in Runway.
  • Tightened permissions contents: write → contents: read (write was only needed to push the tag).
  • Updated header comment + source_branch input description.

runway-create-ota-production-tag.yml

Deleted — it was the reusable tag-creation workflow and is no longer referenced by any caller after the changes above.

Changelog

CHANGELOG entry:null

Related issues

Fixes:

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

Screenshots/Recordings

Before

After

Pre-merge author checklist

Performance checks (if applicable)

  • I've tested on Android
    • Ideally on a mid-range device; emulator is acceptable
  • I've tested with a power user scenario
    • Use these power-user SRPs to import wallets with many accounts and tokens
  • I've instrumented key operations with Sentry traces for production performance metrics

For performance guidelines and tooling, see the Performance Guide.

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Note

Medium Risk
Changes release automation boundaries—OTA-only RC pushes will skip native builds without CI publishing OTA, so Runway/manual steps must stay in sync or releases can stall.

Overview
CI no longer publishes EAS OTA updates or creates v<OTA_VERSION> release tags; those steps move to manual Runway handling.

auto-rc-ota-build-core.yml drops trigger-ota, validate-ota-pr, and create-ota-production-tag, plus inputs ota_channel and create_production_ota_tag. It still runs resolve-context and only runs native build.yml (and TestFlight on iOS) when ota_bump != 'true', so OTA-only commits skip binaries. Permissions are reduced to contents: read (no tag push or workflow dispatch for OTA).

runway-ota-production.yml still pushes production OTA via push-eas-update.yml but no longer creates the production release tag. runway-create-ota-production-tag.yml is deleted.

Reviewed by Cursor Bugbot for commit 0a209ff. Bugbot is set up for automated code reviews on this repo. Configure here.

@weitingsun weitingsun requested a review from a team as a code owner May 29, 2026 16:50
@weitingsun weitingsun self-assigned this May 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbotv2 metamaskbotv2 Bot added the team-mobile-platform Mobile Platform team label May 29, 2026
@github-actions github-actions Bot added pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. size-M risk:high AI analysis: high risk labels May 29, 2026
@weitingsun weitingsun enabled auto-merge May 29, 2026 22:28
@weitingsun weitingsun added this pull request to the merge queue May 29, 2026
Merged via the queue into main with commit c6cc0b9 May 29, 2026
102 of 105 checks passed
@weitingsun weitingsun deleted the wsun/remove-ota-production-tag-creation branch May 29, 2026 22:50
@github-actions github-actions Bot locked and limited conversation to collaborators May 29, 2026
@metamaskbotv2 metamaskbotv2 Bot added the release-7.81.0 Issue or pull request that will be included in release 7.81.0 label May 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. release-7.81.0 Issue or pull request that will be included in release 7.81.0 risk:high AI analysis: high risk size-M team-mobile-platform Mobile Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants