Skip to content
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

Upgrade Crashlytics dep versions #6083

Merged
merged 1 commit into from
Jul 8, 2024
Merged

Upgrade Crashlytics dep versions #6083

merged 1 commit into from
Jul 8, 2024

Conversation

mrober
Copy link
Contributor

@mrober mrober commented Jul 8, 2024

Update Crashlytics and AQS dep versions. Fixed some of the tests because of changes to Mockito. The Mockito version in the Android tests is still pinned a bit old, because after that version a lot more things break.

Copy link
Contributor

github-actions bot commented Jul 8, 2024

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

  • firebase-crashlytics

    Overall coverage changed from 10.35% (d48cb12) to 10.16% (9f7a777) by -0.19%.

    FilenameBase (d48cb12)Merge (9f7a777)Diff
    AutoValue_CrashlyticsReport_ApplicationExitInfo.java66.34%58.42%-7.92%
    AutoValue_CrashlyticsReport_Session_Event_Application_ProcessDetails.java49.12%40.35%-8.77%

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/veTiWdeajc.html

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • firebase-crashlytics

    TypeBase (d48cb12)Merge (9f7a777)Diff
    aar409 kB408 kB-1.16 kB (-0.3%)
    apk (aggressive)688 kB695 kB+6.63 kB (+1.0%)
    apk (release)5.84 MB5.84 MB-2.19 kB (-0.0%)
  • firebase-sessions

    TypeBase (d48cb12)Merge (9f7a777)Diff
    apk (aggressive)556 kB560 kB+3.63 kB (+0.7%)
    apk (release)5.53 MB5.52 MB-5.91 kB (-0.1%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/f556ZidRpO.html

Copy link
Contributor

github-actions bot commented Jul 8, 2024

Unit Test Results

  36 files   -      914    36 suites   - 914   1m 10s ⏱️ - 33m 50s
  92 tests  -   5 119    92 ✔️  -   5 098  0 💤  - 21  0 ±0 
184 runs   - 10 323  184 ✔️  - 10 281  0 💤  - 42  0 ±0 

Results for commit 47d4389. ± Comparison against base commit d48cb12.

@google-oss-bot
Copy link
Contributor

Startup Time Report 1

Note: Layout is sometimes suboptimal due to limited formatting support on GitHub. Please check this report on GCS.

Notes

Startup Times

  • fire-cls

    DeviceStatisticsDistributions
    oriole-32
    Percentiled48cb129f7a777DiffSignificant (?)
    p1011.0 ±3 ms10.6 ±3 ms-405 μs (-3.7%)NO
    p2511.7 ±3 ms11.1 ±3 ms-535 μs (-4.6%)NO
    p5012.6 ±3 ms12.1 ±3 ms-566 μs (-4.5%)NO
    p7513.9 ±3 ms13.4 ±3 ms-471 μs (-3.4%)NO
    p9015.3 ±3 ms15.0 ±3 ms-355 μs (-2.3%)NO

    20 test runs in comparison
    CommitTest Runs
    d48cb12
    • 2024-07-05_17:58:55.930065_ZmRX
    • 2024-07-05_17:58:55.930110_okrn
    • 2024-07-05_17:58:55.930120_JqLO
    • 2024-07-05_17:58:55.930130_rHcp
    • 2024-07-05_17:58:55.930138_rmjP
    • 2024-07-05_17:58:55.930145_onLA
    • 2024-07-05_17:58:55.930152_tgCo
    • 2024-07-05_17:58:55.930160_kVxY
    • 2024-07-05_17:58:55.930168_BXmn
    • 2024-07-05_17:58:55.930175_MJYJ
    9f7a777
    • 2024-07-08_19:24:08.611776_wuIM
    • 2024-07-08_19:24:08.611869_XhZL
    • 2024-07-08_19:24:08.611884_aNJX
    • 2024-07-08_19:24:08.611893_Kdmx
    • 2024-07-08_19:24:08.611900_jAOc
    • 2024-07-08_19:24:08.611907_fELp
    • 2024-07-08_19:24:08.611925_Xzth
    • 2024-07-08_19:24:08.611932_pPrN
    • 2024-07-08_19:24:08.611938_JdYx
    • 2024-07-08_19:24:08.611945_RAMy
    redfin-30
    Percentiled48cb129f7a777DiffSignificant (?)
    p1014.4 ±5 ms15.9 ±6 ms+1.46 ms (+10.1%)NO
    p2515.1 ±6 ms16.8 ±7 ms+1.64 ms (+10.8%)NO
    p5016.0 ±7 ms18.0 ±7 ms+2.00 ms (+12.6%)NO
    p7517.0 ±7 ms19.6 ±9 ms+2.63 ms (+15.5%)NO
    p9018.7 ±8 ms21.9 ±9 ms+3.21 ms (+17.2%)NO

    20 test runs in comparison
    CommitTest Runs
    d48cb12
    • 2024-07-05_17:58:55.930065_ZmRX
    • 2024-07-05_17:58:55.930110_okrn
    • 2024-07-05_17:58:55.930120_JqLO
    • 2024-07-05_17:58:55.930130_rHcp
    • 2024-07-05_17:58:55.930138_rmjP
    • 2024-07-05_17:58:55.930145_onLA
    • 2024-07-05_17:58:55.930152_tgCo
    • 2024-07-05_17:58:55.930160_kVxY
    • 2024-07-05_17:58:55.930168_BXmn
    • 2024-07-05_17:58:55.930175_MJYJ
    9f7a777
    • 2024-07-08_19:24:08.611776_wuIM
    • 2024-07-08_19:24:08.611869_XhZL
    • 2024-07-08_19:24:08.611884_aNJX
    • 2024-07-08_19:24:08.611893_Kdmx
    • 2024-07-08_19:24:08.611900_jAOc
    • 2024-07-08_19:24:08.611907_fELp
    • 2024-07-08_19:24:08.611925_Xzth
    • 2024-07-08_19:24:08.611932_pPrN
    • 2024-07-08_19:24:08.611938_JdYx
    • 2024-07-08_19:24:08.611945_RAMy
  • fire-rc

    DeviceStatisticsDistributions
    oriole-32
    Percentiled48cb129f7a777DiffSignificant (?)
    p10470 ±548 μs209 ±299 μs-261 μs (-55.6%)NO
    p25490 ±567 μs220 ±316 μs-270 μs (-55.0%)NO
    p50523 ±595 μs244 ±355 μs-279 μs (-53.3%)NO
    p75590 ±667 μs290 ±407 μs-300 μs (-50.8%)NO
    p90732 ±779 μs349 ±494 μs-383 μs (-52.3%)NO

    20 test runs in comparison
    CommitTest Runs
    d48cb12
    • 2024-07-05_17:58:55.930065_ZmRX
    • 2024-07-05_17:58:55.930110_okrn
    • 2024-07-05_17:58:55.930120_JqLO
    • 2024-07-05_17:58:55.930130_rHcp
    • 2024-07-05_17:58:55.930138_rmjP
    • 2024-07-05_17:58:55.930145_onLA
    • 2024-07-05_17:58:55.930152_tgCo
    • 2024-07-05_17:58:55.930160_kVxY
    • 2024-07-05_17:58:55.930168_BXmn
    • 2024-07-05_17:58:55.930175_MJYJ
    9f7a777
    • 2024-07-08_19:24:08.611776_wuIM
    • 2024-07-08_19:24:08.611869_XhZL
    • 2024-07-08_19:24:08.611884_aNJX
    • 2024-07-08_19:24:08.611893_Kdmx
    • 2024-07-08_19:24:08.611900_jAOc
    • 2024-07-08_19:24:08.611907_fELp
    • 2024-07-08_19:24:08.611925_Xzth
    • 2024-07-08_19:24:08.611932_pPrN
    • 2024-07-08_19:24:08.611938_JdYx
    • 2024-07-08_19:24:08.611945_RAMy
    redfin-30
    Percentiled48cb129f7a777DiffSignificant (?)
    p10213 ±132 μs530 ±623 μs+317 μs (+149.3%)NO
    p25237 ±178 μs629 ±759 μs+392 μs (+165.9%)NO
    p50289 ±290 μs737 ±882 μs+448 μs (+155.2%)NO
    p75358 ±408 μs897 ±1082 μs+539 μs (+150.8%)NO
    p90474 ±498 μs1.10 ±1 ms+629 μs (+132.7%)NO

    20 test runs in comparison
    CommitTest Runs
    d48cb12
    • 2024-07-05_17:58:55.930065_ZmRX
    • 2024-07-05_17:58:55.930110_okrn
    • 2024-07-05_17:58:55.930120_JqLO
    • 2024-07-05_17:58:55.930130_rHcp
    • 2024-07-05_17:58:55.930138_rmjP
    • 2024-07-05_17:58:55.930145_onLA
    • 2024-07-05_17:58:55.930152_tgCo
    • 2024-07-05_17:58:55.930160_kVxY
    • 2024-07-05_17:58:55.930168_BXmn
    • 2024-07-05_17:58:55.930175_MJYJ
    9f7a777
    • 2024-07-08_19:24:08.611776_wuIM
    • 2024-07-08_19:24:08.611869_XhZL
    • 2024-07-08_19:24:08.611884_aNJX
    • 2024-07-08_19:24:08.611893_Kdmx
    • 2024-07-08_19:24:08.611900_jAOc
    • 2024-07-08_19:24:08.611907_fELp
    • 2024-07-08_19:24:08.611925_Xzth
    • 2024-07-08_19:24:08.611932_pPrN
    • 2024-07-08_19:24:08.611938_JdYx
    • 2024-07-08_19:24:08.611945_RAMy
  • fire-sessions

    DeviceStatisticsDistributions
    oriole-32
    Percentiled48cb129f7a777DiffSignificant (?)
    p102.65 ±0.3 ms2.82 ±0.5 ms+173 μs (+6.5%)NO
    p252.83 ±0.4 ms3.02 ±0.5 ms+181 μs (+6.4%)NO
    p503.17 ±0.5 ms3.40 ±0.7 ms+229 μs (+7.2%)NO
    p753.63 ±0.5 ms3.90 ±0.8 ms+268 μs (+7.4%)NO
    p904.46 ±0.9 ms4.77 ±1 ms+311 μs (+7.0%)NO

    20 test runs in comparison
    CommitTest Runs
    d48cb12
    • 2024-07-05_17:58:55.930065_ZmRX
    • 2024-07-05_17:58:55.930110_okrn
    • 2024-07-05_17:58:55.930120_JqLO
    • 2024-07-05_17:58:55.930130_rHcp
    • 2024-07-05_17:58:55.930138_rmjP
    • 2024-07-05_17:58:55.930145_onLA
    • 2024-07-05_17:58:55.930152_tgCo
    • 2024-07-05_17:58:55.930160_kVxY
    • 2024-07-05_17:58:55.930168_BXmn
    • 2024-07-05_17:58:55.930175_MJYJ
    9f7a777
    • 2024-07-08_19:24:08.611776_wuIM
    • 2024-07-08_19:24:08.611869_XhZL
    • 2024-07-08_19:24:08.611884_aNJX
    • 2024-07-08_19:24:08.611893_Kdmx
    • 2024-07-08_19:24:08.611900_jAOc
    • 2024-07-08_19:24:08.611907_fELp
    • 2024-07-08_19:24:08.611925_Xzth
    • 2024-07-08_19:24:08.611932_pPrN
    • 2024-07-08_19:24:08.611938_JdYx
    • 2024-07-08_19:24:08.611945_RAMy
    redfin-30
    Percentiled48cb129f7a777DiffSignificant (?)
    p106.31 ±2 ms6.47 ±2 ms+164 μs (+2.6%)NO
    p256.81 ±2 ms7.05 ±2 ms+241 μs (+3.5%)NO
    p507.48 ±3 ms7.99 ±3 ms+509 μs (+6.8%)NO
    p758.44 ±4 ms9.53 ±5 ms+1.08 ms (+12.8%)NO
    p909.43 ±5 ms10.9 ±6 ms+1.50 ms (+15.9%)NO

    20 test runs in comparison
    CommitTest Runs
    d48cb12
    • 2024-07-05_17:58:55.930065_ZmRX
    • 2024-07-05_17:58:55.930110_okrn
    • 2024-07-05_17:58:55.930120_JqLO
    • 2024-07-05_17:58:55.930130_rHcp
    • 2024-07-05_17:58:55.930138_rmjP
    • 2024-07-05_17:58:55.930145_onLA
    • 2024-07-05_17:58:55.930152_tgCo
    • 2024-07-05_17:58:55.930160_kVxY
    • 2024-07-05_17:58:55.930168_BXmn
    • 2024-07-05_17:58:55.930175_MJYJ
    9f7a777
    • 2024-07-08_19:24:08.611776_wuIM
    • 2024-07-08_19:24:08.611869_XhZL
    • 2024-07-08_19:24:08.611884_aNJX
    • 2024-07-08_19:24:08.611893_Kdmx
    • 2024-07-08_19:24:08.611900_jAOc
    • 2024-07-08_19:24:08.611907_fELp
    • 2024-07-08_19:24:08.611925_Xzth
    • 2024-07-08_19:24:08.611932_pPrN
    • 2024-07-08_19:24:08.611938_JdYx
    • 2024-07-08_19:24:08.611945_RAMy
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentiled48cb129f7a777DiffSignificant (?)
    p10202 ±5 ms208 ±3 ms+5.89 ms (+2.9%)NO
    p25208 ±5 ms213 ±3 ms+4.93 ms (+2.4%)NO
    p50216 ±6 ms220 ±4 ms+4.65 ms (+2.2%)NO
    p75223 ±7 ms229 ±4 ms+5.61 ms (+2.5%)NO
    p90231 ±7 ms239 ±5 ms+7.45 ms (+3.2%)NO

    20 test runs in comparison
    CommitTest Runs
    d48cb12
    • 2024-07-05_17:58:55.930065_ZmRX
    • 2024-07-05_17:58:55.930110_okrn
    • 2024-07-05_17:58:55.930120_JqLO
    • 2024-07-05_17:58:55.930130_rHcp
    • 2024-07-05_17:58:55.930138_rmjP
    • 2024-07-05_17:58:55.930145_onLA
    • 2024-07-05_17:58:55.930152_tgCo
    • 2024-07-05_17:58:55.930160_kVxY
    • 2024-07-05_17:58:55.930168_BXmn
    • 2024-07-05_17:58:55.930175_MJYJ
    9f7a777
    • 2024-07-08_19:24:08.611776_wuIM
    • 2024-07-08_19:24:08.611869_XhZL
    • 2024-07-08_19:24:08.611884_aNJX
    • 2024-07-08_19:24:08.611893_Kdmx
    • 2024-07-08_19:24:08.611900_jAOc
    • 2024-07-08_19:24:08.611907_fELp
    • 2024-07-08_19:24:08.611925_Xzth
    • 2024-07-08_19:24:08.611932_pPrN
    • 2024-07-08_19:24:08.611938_JdYx
    • 2024-07-08_19:24:08.611945_RAMy
    redfin-30
    Percentiled48cb129f7a777DiffSignificant (?)
    p10241 ±2 ms267 ±6 ms+25.8 ms (+10.7%)MAYBE
    p25246 ±2 ms273 ±5 ms+26.7 ms (+10.8%)YES
    p50253 ±3 ms280 ±5 ms+26.7 ms (+10.6%)YES
    p75260 ±3 ms287 ±5 ms+27.5 ms (+10.6%)YES
    p90269 ±4 ms301 ±7 ms+32.5 ms (+12.1%)YES

    20 test runs in comparison
    CommitTest Runs
    d48cb12
    • 2024-07-05_17:58:55.930065_ZmRX
    • 2024-07-05_17:58:55.930110_okrn
    • 2024-07-05_17:58:55.930120_JqLO
    • 2024-07-05_17:58:55.930130_rHcp
    • 2024-07-05_17:58:55.930138_rmjP
    • 2024-07-05_17:58:55.930145_onLA
    • 2024-07-05_17:58:55.930152_tgCo
    • 2024-07-05_17:58:55.930160_kVxY
    • 2024-07-05_17:58:55.930168_BXmn
    • 2024-07-05_17:58:55.930175_MJYJ
    9f7a777
    • 2024-07-08_19:24:08.611776_wuIM
    • 2024-07-08_19:24:08.611869_XhZL
    • 2024-07-08_19:24:08.611884_aNJX
    • 2024-07-08_19:24:08.611893_Kdmx
    • 2024-07-08_19:24:08.611900_jAOc
    • 2024-07-08_19:24:08.611907_fELp
    • 2024-07-08_19:24:08.611925_Xzth
    • 2024-07-08_19:24:08.611932_pPrN
    • 2024-07-08_19:24:08.611938_JdYx
    • 2024-07-08_19:24:08.611945_RAMy

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/wlkwfVC2Ts/index.html

@mrober mrober requested a review from themiswang July 8, 2024 19:56
@mrober mrober enabled auto-merge (squash) July 8, 2024 19:56
Copy link
Contributor

@themiswang themiswang left a comment

Choose a reason for hiding this comment

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

LGTM

@mrober mrober merged commit ec68af6 into main Jul 8, 2024
37 checks passed
@mrober mrober deleted the crashlytics-deps-bump branch July 8, 2024 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants