Skip to content

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

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

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.23.0 #6409 first; this PR should be retargeted to main once its base lands.

@antonis antonis added the ready-to-merge Triggers the full CI test suite label Jul 22, 2026
@github-actions

github-actions Bot commented Jul 22, 2026

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.23.0 by github-actions in #6409
  • chore(deps): update Sentry Android Gradle Plugin to v6.16.0 by github-actions in #6508
  • chore(deps): update Android SDK to v8.50.1 by github-actions in #6503

🤖 This preview updates automatically when you update the PR.

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 3857.94 ms 3745.55 ms -112.39 ms
Size 4.98 MiB 6.57 MiB 1.59 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
b0d3373+dirty 3831.75 ms 1227.29 ms -2604.46 ms
5ee78d6+dirty 3847.52 ms 1224.27 ms -2623.25 ms
7436d0f+dirty 3851.58 ms 1232.21 ms -2619.38 ms
a858ac3+dirty 3836.62 ms 1222.56 ms -2614.06 ms
0307fa4+dirty 3829.83 ms 1212.15 ms -2617.68 ms
27d9693+dirty 3831.11 ms 1216.94 ms -2614.17 ms
3a829f0+dirty 3839.76 ms 1214.69 ms -2625.06 ms
57e0069+dirty 3842.29 ms 1212.12 ms -2630.17 ms
d038a14+dirty 3845.71 ms 1228.11 ms -2617.59 ms
bf168a4+dirty 3838.60 ms 1218.98 ms -2619.62 ms

App size

Revision Plain With Sentry Diff
b0d3373+dirty 5.15 MiB 6.68 MiB 1.53 MiB
5ee78d6+dirty 5.15 MiB 6.69 MiB 1.53 MiB
7436d0f+dirty 5.15 MiB 6.70 MiB 1.54 MiB
a858ac3+dirty 4.98 MiB 6.51 MiB 1.53 MiB
0307fa4+dirty 4.98 MiB 6.46 MiB 1.48 MiB
27d9693+dirty 4.98 MiB 6.51 MiB 1.53 MiB
3a829f0+dirty 5.15 MiB 6.70 MiB 1.54 MiB
57e0069+dirty 4.98 MiB 6.50 MiB 1.52 MiB
d038a14+dirty 5.15 MiB 6.67 MiB 1.51 MiB
bf168a4+dirty 4.98 MiB 6.47 MiB 1.49 MiB

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 423.85 ms 449.43 ms 25.58 ms
Size 49.74 MiB 55.36 MiB 5.62 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
a50b33d+dirty 500.81 ms 532.11 ms 31.30 ms
5257d80+dirty 423.37 ms 467.54 ms 44.17 ms
5a23c47+dirty 423.85 ms 466.47 ms 42.62 ms
bfba737+dirty 419.52 ms 462.68 ms 43.16 ms
7ff4d0f+dirty 413.81 ms 450.64 ms 36.83 ms
0b5a379+dirty 472.78 ms 533.16 ms 60.38 ms
853723c+dirty 405.54 ms 440.08 ms 34.54 ms
88735e9+dirty 429.04 ms 484.17 ms 55.13 ms
7d8c8bd+dirty 417.45 ms 462.10 ms 44.65 ms
57e0069+dirty 413.82 ms 453.47 ms 39.65 ms

App size

Revision Plain With Sentry Diff
a50b33d+dirty 43.75 MiB 48.08 MiB 4.33 MiB
5257d80+dirty 48.30 MiB 53.58 MiB 5.28 MiB
5a23c47+dirty 49.74 MiB 54.82 MiB 5.07 MiB
bfba737+dirty 49.74 MiB 55.09 MiB 5.34 MiB
7ff4d0f+dirty 48.30 MiB 53.60 MiB 5.30 MiB
0b5a379+dirty 48.30 MiB 53.58 MiB 5.28 MiB
853723c+dirty 48.30 MiB 53.58 MiB 5.28 MiB
88735e9+dirty 49.74 MiB 54.82 MiB 5.07 MiB
7d8c8bd+dirty 48.30 MiB 53.54 MiB 5.23 MiB
57e0069+dirty 49.74 MiB 54.85 MiB 5.11 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

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 3850.66 ms 3735.39 ms -115.27 ms
Size 4.98 MiB 6.57 MiB 1.59 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
853723c+dirty 3849.33 ms 1221.07 ms -2628.26 ms
c151573+dirty 3835.64 ms 1216.10 ms -2619.53 ms
68ae91b+dirty 3836.36 ms 1222.04 ms -2614.32 ms
f170ec3+dirty 3844.74 ms 1222.67 ms -2622.07 ms
267d3ed+dirty 3860.14 ms 1223.39 ms -2636.76 ms
a858ac3+dirty 3850.72 ms 1226.35 ms -2624.37 ms
27d9693+dirty 3837.40 ms 1227.74 ms -2609.67 ms
c004dae+dirty 3857.82 ms 1224.87 ms -2632.95 ms
57e0069+dirty 3842.23 ms 1210.00 ms -2632.23 ms
bf168a4+dirty 3846.83 ms 1221.33 ms -2625.49 ms

App size

Revision Plain With Sentry Diff
853723c+dirty 5.15 MiB 6.69 MiB 1.53 MiB
c151573+dirty 5.15 MiB 6.68 MiB 1.53 MiB
68ae91b+dirty 4.98 MiB 6.46 MiB 1.48 MiB
f170ec3+dirty 5.15 MiB 6.69 MiB 1.53 MiB
267d3ed+dirty 5.15 MiB 6.69 MiB 1.54 MiB
a858ac3+dirty 4.98 MiB 6.51 MiB 1.53 MiB
27d9693+dirty 4.98 MiB 6.51 MiB 1.53 MiB
c004dae+dirty 5.15 MiB 6.67 MiB 1.51 MiB
57e0069+dirty 4.98 MiB 6.50 MiB 1.52 MiB
bf168a4+dirty 4.98 MiB 6.47 MiB 1.49 MiB

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 534.90 ms 560.19 ms 25.29 ms
Size 49.74 MiB 55.36 MiB 5.62 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
2c735cc+dirty 435.20 ms 459.48 ms 24.28 ms
a5d243c+dirty 499.20 ms 525.62 ms 26.42 ms
5fe1c6c+dirty 365.84 ms 408.62 ms 42.78 ms
4953e94+dirty 398.80 ms 431.81 ms 33.01 ms
5a316ea+dirty 626.41 ms 645.00 ms 18.59 ms
bfba737+dirty 425.19 ms 465.98 ms 40.79 ms
3ce5254+dirty 373.90 ms 427.84 ms 53.94 ms
40c9884+dirty 453.76 ms 478.98 ms 25.22 ms
853723c+dirty 415.82 ms 460.94 ms 45.12 ms
68ae91b+dirty 515.04 ms 578.08 ms 63.04 ms

App size

Revision Plain With Sentry Diff
2c735cc+dirty 43.94 MiB 48.94 MiB 5.00 MiB
a5d243c+dirty 48.30 MiB 53.54 MiB 5.23 MiB
5fe1c6c+dirty 43.94 MiB 49.00 MiB 5.06 MiB
4953e94+dirty 43.94 MiB 48.94 MiB 5.00 MiB
5a316ea+dirty 49.74 MiB 55.09 MiB 5.34 MiB
bfba737+dirty 49.74 MiB 55.09 MiB 5.34 MiB
3ce5254+dirty 43.94 MiB 48.98 MiB 5.04 MiB
40c9884+dirty 49.74 MiB 55.09 MiB 5.34 MiB
853723c+dirty 48.30 MiB 53.58 MiB 5.28 MiB
68ae91b+dirty 49.74 MiB 54.79 MiB 5.05 MiB

@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch 2 times, most recently from 63e7c97 to 932a11b Compare July 23, 2026 04:02
@antonis
antonis force-pushed the chore/remove-cocoapods-ci-job branch from 139f852 to 6e9ff1b Compare July 23, 2026 07:33
@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch from 932a11b to fc8bd37 Compare July 23, 2026 08:08
@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch 3 times, most recently from eca25be to 88771ff Compare July 23, 2026 14:54
@antonis
antonis force-pushed the chore/remove-cocoapods-ci-job branch from d0887e0 to 650469e Compare July 23, 2026 15:10
@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch from 88771ff to 114b52c Compare July 24, 2026 05:12
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 force-pushed the chore/remove-cocoapods-ci-job branch from 650469e to ccfef9e Compare July 24, 2026 05:17
@antonis
antonis marked this pull request as draft July 24, 2026 05:45

@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.

Marking as blocked till the refactoring is reapplied #6507

@antonis antonis added Blocked and removed ready-to-merge Triggers the full CI test suite labels Jul 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor
Fails
🚫 Pull request is not ready for merge, please add the "ready-to-merge" label to the pull request

Generated by 🚫 dangerJS against ccfef9e

@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch from 114b52c to 50ad702 Compare July 24, 2026 05:51
@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch 4 times, most recently from b1c040d to b64a642 Compare July 24, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants