10BC0 [Infra] Limit concurrnet matrix jobs in spm.yml by ncooke3 · Pull Request #13677 · firebase/firebase-ios-sdk · GitHub
[go: up one dir, main page]

Skip to content

Conversation

ncooke3
Copy link
Member
@ncooke3 ncooke3 commented Sep 19, 2024

Appears to fix the issue in the nightlies

#no-changelog
Fix #13670

Copy link
Member
@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

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

Good find!

The other approach is firebase/data-connect-ios-sdk#11 which is harder to maintain, but has the advantage of reusing checkouts and downloads.

Did you try any higher numbers? Does 2 work?

@paulb777
Copy link
Member

Looks like 2 may only sometimes work - https://github.com/firebase/data-connect-ios-sdk/actions/runs/10949237520/job/30402029979 :(

@ncooke3
Copy link
Member Author
ncooke3 commented Sep 19, 2024

Looking good apart from a simulator crash in the catalyst job:

Testing failed:
	xctest (12656) encountered an error (Early unexpected exit, operation never finished bootstrapping - no restart will be attempted. (Underlying Error: Crash: xctest (12656) <external symbol>: (terminated at launch; ignore backtrace)))

Restarting that one... In any case, I don't think it is related to the GHA issue this PR is addressing.

@ncooke3 ncooke3 merged commit c4e1088 into main Sep 19, 2024
32 of 33 checks passed
@ncooke3 ncooke3 deleted the nc/limit-matrix branch September 19, 2024 22:30
@firebase firebase locked and limited conversation to collaborators Oct 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nightly Testing Report
2 participants
0