Skip to content

Extend iOS workflow timeout#1465

Merged
bmehta001 merged 2 commits into
microsoft:mainfrom
bmehta001:bhamehta/codeql-skip-js-gitlink-only
Jun 5, 2026
Merged

Extend iOS workflow timeout#1465
bmehta001 merged 2 commits into
microsoft:mainfrom
bmehta001:bhamehta/codeql-skip-js-gitlink-only

Conversation

@bmehta001
Copy link
Copy Markdown
Contributor

@bmehta001 bmehta001 commented Jun 5, 2026

Increase the iOS workflow timeout from 30 minutes to 60 minutes so slower simulator startup does not cancel otherwise healthy runs.

Context:

  • Update lib/modules submodule to c637015fb #1461 passed after rerunning the failed JavaScript CodeQL job.
  • The remaining failure pattern is an iOS simulator leg timing out at about 30 minutes.
  • The earlier simulator timing/deadlock issue appears to have been addressed separately, so a longer timeout is now appropriate.

Validation performed locally:

  • git diff --check
  • Workflow sanity check confirmed timeout-minutes: 60 is present and the temporary CodeQL gitlink detector was removed.

@bmehta001 bmehta001 requested a review from a team as a code owner June 5, 2026 03:37
@bmehta001 bmehta001 changed the title Skip JavaScript CodeQL for gitlink-only PRs Extend iOS workflow timeout Jun 5, 2026
Allow the iOS simulator matrix up to one hour so slower simulator startup does not cancel otherwise healthy runs.

Files changed: .github/workflows/build-ios-mac.yml

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bmehta001 bmehta001 force-pushed the bhamehta/codeql-skip-js-gitlink-only branch from ad074ad to 94059d4 Compare June 5, 2026 03:42
@bmehta001 bmehta001 self-assigned this Jun 5, 2026
@bmehta001 bmehta001 merged commit c8df1d1 into microsoft:main Jun 5, 2026
25 of 27 checks passed
@bmehta001 bmehta001 deleted the bhamehta/codeql-skip-js-gitlink-only branch June 5, 2026 16:16
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