Skip to content

ci(e2e): remove obsolete CocoaPods sample build job#6500

Open
antonis wants to merge 2 commits into
deps/scripts/update-cocoa.shfrom
chore/remove-cocoapods-ci-job
Open

ci(e2e): remove obsolete CocoaPods sample build job#6500
antonis wants to merge 2 commits into
deps/scripts/update-cocoa.shfrom
chore/remove-cocoapods-ci-job

Conversation

@antonis

@antonis antonis commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Removes the dedicated CocoaPods sample build job (sentry-consumption: cocoapods) from sample-application.yml, along with the now single-valued sentry-consumption matrix dimension and its references (job name, SENTRY_USE_XCFRAMEWORK=0 export, archive/upload/log-artifact conditions and names).

Stacked on top of #6409 (Cocoa bump to 9.22.0).

💡 Motivation and Context

sentry-cocoa dropped CocoaPods (trunk) support after 9.19.19.20.0, 9.21.0, and 9.22.0 are published only as GitHub Release xcframeworks and are not on the CocoaPods CDN.

The cocoapods job exports SENTRY_USE_XCFRAMEWORK=0 to exercise the source-build fallback (s.dependency 'Sentry', sentry_cocoa_version), which resolves Sentry from trunk. On #6409 that job fails because Sentry (= 9.22.0) isn't on trunk:

None of your spec sources contain a spec satisfying the dependency: `Sentry (= 9.22.0)`.

The job only passed on main because main is still pinned to 9.19.1. Since the fallback can no longer resolve any Cocoa version past 9.19.1, the regression job is obsolete and blocks every future Cocoa bump.

💚 How did you test it?

  • Validated the workflow YAML parses cleanly.
  • CI on this PR runs the remaining xcframework iOS sample builds.

📝 Checklist

  • I added tests to verify changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.
  • No breaking changes.

🔮 Next steps

  • The SENTRY_USE_XCFRAMEWORK=0 fallback path in RNSentry.podspec is now effectively dead for versions > 9.19.1. Consider removing it (and the deprecated SENTRY_USE_SPM alias) in a follow-up, unless we intend to keep it for offline/proxy builds pinned to old Cocoa versions.
  • Merge chore(deps): update Cocoa SDK to v9.22.0 #6409 first; this PR should be retargeted to main once its base lands.

web-flow and others added 2 commits July 22, 2026 09:09
sentry-cocoa dropped CocoaPods (trunk) support after 9.19.1, so the
source-build fallback (SENTRY_USE_XCFRAMEWORK=0) can no longer resolve
newer versions. The dedicated CocoaPods regression job now fails on any
Cocoa bump past 9.19.1, so remove it and the now single-valued
sentry-consumption matrix dimension.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@antonis antonis added the ready-to-merge Triggers the full CI test suite label Jul 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


  • ci(e2e): remove obsolete CocoaPods sample build job by antonis in #6500
  • chore(deps): update Cocoa SDK to v9.22.0 by github-actions in #6409
  • docs: Update 8.19.0 changelog by antonis in #6498
  • chore(deps): bump fast-uri from 3.1.2 to 3.1.4 by dependabot in #6496
  • chore(deps): pin glob 10.x to ^10.5.0 (dev-only advisory) by antonis in #6489
  • chore(deps): bump brace-expansion resolutions to ^2.1.2 (dev-only advisory) by antonis in #6488
  • chore(deps): add cross-spawn ^7.0.6 resolution (dev-only advisory) by antonis in #6487
  • chore(deps): bump js-yaml resolution to ^4.3.0 (dev-only advisory) by antonis in #6486
  • chore(deps): update CLI to v3.6.1 by github-actions in #6493
  • chore(deps): update JavaScript SDK to v10.67.0 by github-actions in #6494
  • chore(deps): update JavaScript SDK to v10.66.0 by github-actions in #6471
  • chore(deps): update Maestro to v2.7.0 by github-actions in #6481
  • chore(ios): Bump iOS binary size limit to 1650 KiB by antonis in #6485
  • chore(deps): bump tar from 7.5.16 to 7.5.20 by dependabot in #6483
  • chore(deps): bump shell-quote from 1.8.4 to 1.10.0 by dependabot in #6484
  • chore(deps): bump the codeql-action group with 3 updates by dependabot in #6475
  • chore(deps): bump brace-expansion from 1.1.13 to 1.1.16 by dependabot in #6482
  • chore(deps): bump axios from 1.16.0 to 1.18.1 by dependabot in #6479
  • chore(deps): bump actions/setup-java from 5.5.0 to 5.6.0 by dependabot in #6476

🤖 This preview updates automatically when you update the PR.

@github-actions

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 3814.02 ms 1217.43 ms -2596.59 ms
Size 4.98 MiB 6.58 MiB 1.60 MiB

Baseline results on branch: deps/scripts/update-cocoa.sh

Startup times

Revision Plain With Sentry Diff
08c933b+dirty 1224.53 ms 1226.59 ms 2.06 ms
cb5d89e+dirty 3851.80 ms 1225.70 ms -2626.10 ms
ebc61a9+dirty 3844.58 ms 1218.91 ms -2625.67 ms
58dacb0+dirty 3849.82 ms 1228.28 ms -2621.54 ms

App size

Revision Plain With Sentry Diff
08c933b+dirty 3.38 MiB 4.78 MiB 1.40 MiB
cb5d89e+dirty 5.15 MiB 6.70 MiB 1.54 MiB
ebc61a9+dirty 4.98 MiB 6.51 MiB 1.53 MiB
58dacb0+dirty 5.15 MiB 6.67 MiB 1.51 MiB

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 515.98 ms 554.46 ms 38.48 ms
Size 49.74 MiB 55.36 MiB 5.61 MiB

@antonis
antonis marked this pull request as ready for review July 22, 2026 11:37

@antonis antonis left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The failed E2E tests are related to #6497

@github-actions

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 3831.96 ms 1220.65 ms -2611.31 ms
Size 4.98 MiB 6.58 MiB 1.60 MiB

Baseline results on branch: deps/scripts/update-cocoa.sh

Startup times

Revision Plain With Sentry Diff
08c933b+dirty 1204.21 ms 1217.09 ms 12.88 ms
cb5d89e+dirty 3854.04 ms 1228.56 ms -2625.48 ms
ebc61a9+dirty 3822.19 ms 1201.34 ms -2620.85 ms
58dacb0+dirty 3839.13 ms 1211.04 ms -2628.09 ms

App size

Revision Plain With Sentry Diff
08c933b+dirty 3.38 MiB 4.78 MiB 1.40 MiB
cb5d89e+dirty 5.15 MiB 6.70 MiB 1.54 MiB
ebc61a9+dirty 4.98 MiB 6.51 MiB 1.53 MiB
58dacb0+dirty 5.15 MiB 6.67 MiB 1.51 MiB

@github-actions

Copy link
Copy Markdown
Contributor

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 428.09 ms 477.25 ms 49.16 ms
Size 49.74 MiB 55.36 MiB 5.61 MiB

@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch from b42eea9 to 63e7c97 Compare July 22, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Triggers the full CI test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants