Skip to content

ci: cleanup disk space for iOS e2e tests#1382

Merged
kirillzyusko merged 6 commits intomainfrom
ci/cleanup-disk-space-for-ios-e2e-tests
Mar 20, 2026
Merged

ci: cleanup disk space for iOS e2e tests#1382
kirillzyusko merged 6 commits intomainfrom
ci/cleanup-disk-space-for-ios-e2e-tests

Conversation

@kirillzyusko
Copy link
Copy Markdown
Owner

@kirillzyusko kirillzyusko commented Mar 19, 2026

📜 Description

Fixed iOS 26 flakiness.

💡 Motivation and Context

It seems like simultaneous video + screenshot causes simulator to hang. Additionally I remove simulator service:

sudo killall -9 com.apple.CoreSimulator.CoreSimulatorService || true

To be sure that there is no zombie simulators.

📢 Changelog

CI

  • kill sim service before spinning up simulator;

E2E

  • don't capture video + screenshots together;

🤔 How Has This Been Tested?

Tested via this PR. 5/5 attempts are successful.

📸 Screenshots (if appropriate):

image

📝 Checklist

  • CI successfully passed
  • I added new mocks and corresponding unit-tests if library API was changed

@kirillzyusko kirillzyusko self-assigned this Mar 19, 2026
@kirillzyusko kirillzyusko added 🍎 iOS iOS specific CI CI (github actions) e2e Anything about E2E tests labels Mar 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 19, 2026

📊 Package size report

Current size Target Size Difference
306910 bytes 306910 bytes 0 bytes 📉

@kirillzyusko kirillzyusko marked this pull request as ready for review March 20, 2026 15:15
@kirillzyusko kirillzyusko merged commit 7efe1e7 into main Mar 20, 2026
40 checks passed
@kirillzyusko kirillzyusko deleted the ci/cleanup-disk-space-for-ios-e2e-tests branch March 20, 2026 15:16
kirillzyusko added a commit that referenced this pull request Mar 23, 2026
## 📜 Description

Use `macos-15` instead of `macos-26`.

## 💡 Motivation and Context

Looks like `macos-26` + iOS 26 are not stable yet at the moment:
actions/runner-images#13830

So in this PR I'm switching to `macos-15` instead. Also reverting
#1382
as it looks like it doesn't improve situation at all.

## 📢 Changelog

<!-- High level overview of important changes -->
<!-- For example: fixed status bar manipulation; added new types
declarations; -->
<!-- If your changes don't affect one of platform/language below - then
remove this platform/language -->

### CI

- removed changes introduced in
#1382
- switch to `macos-15` instead of `macos-26`;

## 🤔 How Has This Been Tested?

Tested via this PR (2x run).

## 📸 Screenshots (if appropriate):

<img width="664" height="176" alt="image"
src="https://github.com/user-attachments/assets/9d49733a-320e-41d4-8e21-105b58623c5b"
/>

## 📝 Checklist

- [x] CI successfully passed
- [x] I added new mocks and corresponding unit-tests if library API was
changed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI CI (github actions) e2e Anything about E2E tests 🍎 iOS iOS specific

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant