-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
ci(INFRA-3596): phase4: add workflow_dispatch to build-ios-e2e, update run-e2e-api-specs runs-on #29812
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
ci(INFRA-3596): phase4: add workflow_dispatch to build-ios-e2e, update run-e2e-api-specs runs-on #29812
Changes from 29 commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
374ecd3
ci(actionlint): register Namespace runner labels
jluque0101 534abd2
ci(workflows): add runner_provider workflow_dispatch input on Phase 1…
jluque0101 2167ddc
ci(workflows): add runner_provider workflow_call input on Phase 1-4 r…
jluque0101 87ee641
ci(workflows): forward runner_provider to Phase 1-4 reusable callees
jluque0101 3e2ea7f
ci(workflows): wrap runs-on in runner_provider ternary on Phase 1-4 jobs
jluque0101 9046d5c
ci(workflows): replace Namespace placeholder with canonical profile l…
jluque0101 17615e1
Merge branch 'main' into namespace-runner-trial
jluque0101 82136ac
Merge branch 'main' into namespace-runner-trial
jluque0101 5a68282
Merge branch 'main' into namespace-runner-trial
jluque0101 5a8c041
Merge branch 'main' into namespace-runner-trial
jluque0101 f0b04ee
Merge branch 'main' into namespace-runner-trial
jluque0101 08caa40
Merge branch 'main' into namespace-runner-trial
jluque0101 c7a597e
Merge branch 'main' into namespace-runner-trial
jluque0101 36ffd4d
Merge remote-tracking branch 'origin/main' into namespace-runner-trial
jluque0101 fe5188d
Merge branch 'main' into namespace-runner-trial
jluque0101 734f456
INFRA-3596: phase4: add workflow_dispatch to build-ios-e2e, update ru…
XxdpavelxX 9968d30
fix: quote $HOME in debug step, declare runner_provider input in api-…
XxdpavelxX de2f549
feat: add nscloud-cache-action for iOS build cache (Phase 4 Step 5)
XxdpavelxX 2ab6107
fix: remove ios/build from nscloud cache paths to prevent mount inter…
XxdpavelxX f073446
fix: remove DerivedData from nscloud cache to prevent stale build index
XxdpavelxX bfd2c57
fix(ci): clear Xcode DerivedData before build on Namespace runner
XxdpavelxX 43b2f1f
fix(ci): also clear ios/build before Namespace iOS build
XxdpavelxX b049760
fix(ci): remove ios/Pods from nscloud cache, clear before pod install
XxdpavelxX a8964e7
ci(infra-3596): plumb runner_provider through run-e2e-api-specs.yml
XxdpavelxX 3ada7d1
ci(infra-3596): provider-aware runs-on in update-e2e-fixtures.yml
XxdpavelxX cf1f514
ci(infra-3596): provider-aware runs-on for signed iOS / TestFlight up…
XxdpavelxX ed0bc5c
ci(infra-3596): provider-aware runs-on in build-ios-upload-to-browser…
XxdpavelxX 1f2f633
chore(infra-3596): drop non-Phase-4 file changes from PR
XxdpavelxX e7405cc
Merge remote-tracking branch 'origin/main' into namespace-runner-trial4
XxdpavelxX 2194102
fix(ci): restore id-token permission for build-ios-e2e.yml dispatch
XxdpavelxX fd64763
chore(ci): trim verbose comments in build-ios-e2e.yml Namespace steps
XxdpavelxX d806dc2
test(ci): A/B test — try cocoapods preset, keep clear step
XxdpavelxX 0ec5f25
test(ci): A/B test — preset only, no clear step
XxdpavelxX f37c5cc
chore(ci): adopt cocoapods preset as canonical Namespace iOS cache co…
XxdpavelxX 856c6c5
chore(ci): drop workflow_dispatch from build-ios-e2e.yml
XxdpavelxX File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the reason for we need to add a dispatch?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair point, added it for trial iteration speed during Phase 4 validation. It's useful during trial, not really needed once this lands. Removed