Skip to content

Refactor [no-ticket] Bitrise stack downsize in AddNightlyGroupToTestFlightBeta and Firebase_Performance_Test#33920

Open
clarmso wants to merge 1 commit into
mainfrom
cs/bitrise-workflow-runner-bump-May-2026
Open

Refactor [no-ticket] Bitrise stack downsize in AddNightlyGroupToTestFlightBeta and Firebase_Performance_Test#33920
clarmso wants to merge 1 commit into
mainfrom
cs/bitrise-workflow-runner-bump-May-2026

Conversation

@clarmso
Copy link
Copy Markdown
Collaborator

@clarmso clarmso commented May 20, 2026

📜 Tickets

💡 Description

Bitrise suggested that AddNightlyGroupToTestFlightBeta and Firebase_Performance_Test could use a smaller machine for the workflow without any performance lost.
Screenshot 2026-05-20 at 10-38-49 Builds (firefox-ios) - Bitrise CI (Mozilla) - Bitrise

This PR tests and shows what Bitrise will apply automatically in a week.

Firebase_Performance_Test`: https://app.bitrise.io/app/6c06d3a40422d10f/build/2123cbce-089d-472a-a0eb-5a165ada93e5
(The workflow hasn't been passing because the lack of Firebase support in newer Xcode, but the job uses "Xcode 16.2 • macOS Large" so ok.)

TODO: How do I test AddNightlyGroupToTestFlightBeta?

🎥 Demos

Before After
Demo

📝 Checklist

  • I filled in the ticket numbers and a description of my work
  • I updated the PR name to follow our PR naming guidelines
  • I ensured unit tests pass and wrote tests for new code
  • If working on UI, I checked and implemented accessibility (Dynamic Text and VoiceOver)
  • If adding telemetry, I read the data stewardship requirements and will request a data review
  • If adding or modifying strings, I read the guidelines and will request a string review from l10n
  • If needed, I updated documentation and added comments to complex code

…ase_Performance_Test runners

Apply Bitrise optimizer recommendations to downsize two workflows
from g2.mac.4large to g2.mac.large with no expected speed change:
- AddNightlyGroupToTestFlightBeta: pure Python script, oversized
- Firebase_Performance_Test: not CPU-bound enough to saturate XL

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@clarmso clarmso requested review from Eijebong, hneiva and isabelrios May 20, 2026 14:49
@clarmso clarmso requested review from a team as code owners May 20, 2026 14:49
@mobiletest-ci-bot
Copy link
Copy Markdown

🧹 Tidy commit

Just 1 file(s) touched. Thanks for keeping it clean and review-friendly!

💬 Description craftsman

Great PR description! Reviewers salute you 🫡

✅ New file code coverage

No new file detected so code coverage gate wasn't ran.

✅ Existing file code coverage

No modified file detected so code coverage gate wasn't ran.

Generated by 🚫 Danger Swift against 747ca2c

@clarmso
Copy link
Copy Markdown
Collaborator Author

clarmso commented May 20, 2026

😆 There's no better way to AddNightlyGroupToTestFlightBeta than to test in production.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants