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

Fix release config for config-interop #5539

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Fix release config for config-interop #5539

merged 1 commit into from
Nov 10, 2023

Conversation

rlazo
Copy link
Collaborator

@rlazo rlazo commented Nov 9, 2023

No description provided.

@rlazo rlazo requested a review from mrober November 9, 2023 23:00
Copy link
Contributor

github-actions bot commented Nov 9, 2023

Release note changes

The following had changelogs that were modified, but did not have any unreleased entries for release notes to generate from.

Changelogs

firebase-config-interop

@google-oss-bot
Copy link
Contributor

1 Warning
⚠️ Did you forget to add a changelog entry? (Add the 'no-changelog' label to the PR to silence this warning.)

Generated by 🚫 Danger

@google-oss-bot
Copy link
Contributor

@rlazo rlazo requested a review from davidmotson November 9, 2023 23:05
Copy link
Contributor

github-actions bot commented Nov 9, 2023

Unit Test Results

  44 files  +    2    44 suites  +2   1m 22s ⏱️ -7s
329 tests +149  329 ✔️ +149  0 💤 ±0  0 ±0 
658 runs  +298  658 ✔️ +298  0 💤 ±0  0 ±0 

Results for commit 45cda7a. ± Comparison against base commit 58f456f.

This pull request removes 180 and adds 329 tests. Note that renamed tests count towards both.
com.google.android.datatransport.cct.CctBackendFactoryTest ‑ create_returnCCTBackend_WhenBackendNameIsCCT
com.google.android.datatransport.cct.CctDestinationTest ‑ cctDestination_shouldOnlySupportProtoAndJson
com.google.android.datatransport.cct.CctDestinationTest ‑ cctDestination_shouldSupportProtoAndJson
com.google.android.datatransport.cct.CctTransportBackendTest ‑ decorate_whenOffline_shouldProperlyPopulateNetworkInfo
com.google.android.datatransport.cct.CctTransportBackendTest ‑ decorate_whenOnline_shouldProperlyPopulateNetworkInfo
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_CompressedResponseIsUncompressed
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_whenBackendRedirectsMoreThan5Times_shouldOnlyRedirect4Times
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_whenBackendRedirects_shouldCorrectlyFollowTheRedirectViaPost
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_whenBackendRedirectswith307_shouldCorrectlyFollowTheRedirectViaPost
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_whenBackendResponseTimesOut_shouldReturnTransientError
…
com.google.firebase.crashlytics.internal.common.CrashlyticsControllerRobolectricTest ‑ testDoCloseSession_disabledAnrs_doesNotPersistsAppExitInfo
com.google.firebase.crashlytics.internal.common.CrashlyticsControllerRobolectricTest ‑ testDoCloseSession_enabledAnrs_doesNotPersistsAppExitInfoIfItDoesntExist
com.google.firebase.crashlytics.internal.common.CrashlyticsControllerRobolectricTest ‑ testDoCloseSession_enabledAnrs_persistsAppExitInfoIfItExists
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_notPersistIfAnrBeforeSession
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_notPersistIfAppExitInfoNotAnrButWithinSession
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_persistIfAnrWithinSession
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_persistIfAnrWithinSession_multipleAppExitInfo
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testconvertInputStreamToString_worksSuccessfully
com.google.firebase.crashlytics.internal.model.CrashlyticsReportTest ‑ testGetBinaryImageUuidUtf8Bytes_returnsNullWhenUuidIsNull
com.google.firebase.crashlytics.internal.model.CrashlyticsReportTest ‑ testGetBinaryImageUuidUtf8Bytes_returnsProperBytes
…

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • firebase-config-interop

    TypeBase (58f456f)Merge (c5087af)Diff
    aar10.4 kB10.4 kB+3 B (+0.0%)
    apk (release)598 kB598 kB+4 B (+0.0%)

Test Logs

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

@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
    Percentile58f456fc5087afDiffSignificant (?)
    p109.67 ±3 ms11.2 ±3 ms+1.48 ms (+15.3%)NO
    p2510.5 ±3 ms11.8 ±3 ms+1.36 ms (+13.0%)NO
    p5011.4 ±3 ms13.0 ±3 ms+1.57 ms (+13.7%)NO
    p7512.7 ±3 ms14.3 ±4 ms+1.54 ms (+12.1%)NO
    p9014.2 ±3 ms16.1 ±4 ms+1.90 ms (+13.4%)NO

    20 test runs in comparison
    CommitTest Runs
    58f456f
    • 2023-11-09_17:13:00.761896_IpDr
    • 2023-11-09_17:13:00.761928_SVRz
    • 2023-11-09_17:13:00.761940_BeGY
    • 2023-11-09_17:13:00.761949_NgQD
    • 2023-11-09_17:13:00.761956_fRfl
    • 2023-11-09_17:13:00.761963_gepG
    • 2023-11-09_17:13:00.761975_IWCM
    • 2023-11-09_17:13:00.761982_kvPx
    • 2023-11-09_17:13:00.761989_gDss
    • 2023-11-09_17:13:00.761996_rgMa
    c5087af
    • 2023-11-09_23:11:32.521573_bpCS
    • 2023-11-09_23:11:32.521608_AwJM
    • 2023-11-09_23:11:32.521620_ghIQ
    • 2023-11-09_23:11:32.521628_VTrg
    • 2023-11-09_23:11:32.521638_oXyw
    • 2023-11-09_23:11:32.521648_IIXV
    • 2023-11-09_23:11:32.521656_gSZE
    • 2023-11-09_23:11:32.521665_Kjzh
    • 2023-11-09_23:11:32.521671_PtoY
    • 2023-11-09_23:11:32.521675_mXWP
    redfin-30
    Percentile58f456fc5087afDiffSignificant (?)
    p1014.8 ±6 ms12.9 ±4 ms-1.88 ms (-12.7%)NO
    p2515.6 ±7 ms13.7 ±5 ms-1.94 ms (-12.4%)NO
    p5016.8 ±8 ms14.7 ±6 ms-2.09 ms (-12.4%)NO
    p7518.4 ±8 ms16.2 ±7 ms-2.18 ms (-11.9%)NO
    p9020.9 ±9 ms18.6 ±9 ms-2.29 ms (-11.0%)NO

    20 test runs in comparison
    CommitTest Runs
    58f456f
    • 2023-11-09_17:13:00.761896_IpDr
    • 2023-11-09_17:13:00.761928_SVRz
    • 2023-11-09_17:13:00.761940_BeGY
    • 2023-11-09_17:13:00.761949_NgQD
    • 2023-11-09_17:13:00.761956_fRfl
    • 2023-11-09_17:13:00.761963_gepG
    • 2023-11-09_17:13:00.761975_IWCM
    • 2023-11-09_17:13:00.761982_kvPx
    • 2023-11-09_17:13:00.761989_gDss
    • 2023-11-09_17:13:00.761996_rgMa
    c5087af
    • 2023-11-09_23:11:32.521573_bpCS
    • 2023-11-09_23:11:32.521608_AwJM
    • 2023-11-09_23:11:32.521620_ghIQ
    • 2023-11-09_23:11:32.521628_VTrg
    • 2023-11-09_23:11:32.521638_oXyw
    • 2023-11-09_23:11:32.521648_IIXV
    • 2023-11-09_23:11:32.521656_gSZE
    • 2023-11-09_23:11:32.521665_Kjzh
    • 2023-11-09_23:11:32.521671_PtoY
    • 2023-11-09_23:11:32.521675_mXWP
  • fire-rc

    DeviceStatisticsDistributions
    oriole-32
    Percentile58f456fc5087afDiffSignificant (?)
    p10245 ±382 μs254 ±348 μs+8.71 μs (+3.6%)NO
    p25258 ±401 μs270 ±365 μs+12.2 μs (+4.7%)NO
    p50280 ±422 μs297 ±399 μs+17.0 μs (+6.1%)NO
    p75337 ±482 μs356 ±452 μs+19.8 μs (+5.9%)NO
    p90428 ±581 μs433 ±526 μs+5.13 μs (+1.2%)NO

    20 test runs in comparison
    CommitTest Runs
    58f456f
    • 2023-11-09_17:13:00.761896_IpDr
    • 2023-11-09_17:13:00.761928_SVRz
    • 2023-11-09_17:13:00.761940_BeGY
    • 2023-11-09_17:13:00.761949_NgQD
    • 2023-11-09_17:13:00.761956_fRfl
    • 2023-11-09_17:13:00.761963_gepG
    • 2023-11-09_17:13:00.761975_IWCM
    • 2023-11-09_17:13:00.761982_kvPx
    • 2023-11-09_17:13:00.761989_gDss
    • 2023-11-09_17:13:00.761996_rgMa
    c5087af
    • 2023-11-09_23:11:32.521573_bpCS
    • 2023-11-09_23:11:32.521608_AwJM
    • 2023-11-09_23:11:32.521620_ghIQ
    • 2023-11-09_23:11:32.521628_VTrg
    • 2023-11-09_23:11:32.521638_oXyw
    • 2023-11-09_23:11:32.521648_IIXV
    • 2023-11-09_23:11:32.521656_gSZE
    • 2023-11-09_23:11:32.521665_Kjzh
    • 2023-11-09_23:11:32.521671_PtoY
    • 2023-11-09_23:11:32.521675_mXWP
    redfin-30
    Percentile58f456fc5087afDiffSignificant (?)
    p10358 ±452 μs458 ±510 μs+100 μs (+28.0%)NO
    p25421 ±541 μs535 ±608 μs+113 μs (+26.9%)NO
    p50497 ±647 μs650 ±783 μs+153 μs (+30.8%)NO
    p75575 ±772 μs801 ±1002 μs+227 μs (+39.4%)NO
    p90695 ±972 μs1.01 ±1 ms+311 μs (+44.8%)NO

    20 test runs in comparison
    CommitTest Runs
    58f456f
    • 2023-11-09_17:13:00.761896_IpDr
    • 2023-11-09_17:13:00.761928_SVRz
    • 2023-11-09_17:13:00.761940_BeGY
    • 2023-11-09_17:13:00.761949_NgQD
    • 2023-11-09_17:13:00.761956_fRfl
    • 2023-11-09_17:13:00.761963_gepG
    • 2023-11-09_17:13:00.761975_IWCM
    • 2023-11-09_17:13:00.761982_kvPx
    • 2023-11-09_17:13:00.761989_gDss
    • 2023-11-09_17:13:00.761996_rgMa
    c5087af
    • 2023-11-09_23:11:32.521573_bpCS
    • 2023-11-09_23:11:32.521608_AwJM
    • 2023-11-09_23:11:32.521620_ghIQ
    • 2023-11-09_23:11:32.521628_VTrg
    • 2023-11-09_23:11:32.521638_oXyw
    • 2023-11-09_23:11:32.521648_IIXV
    • 2023-11-09_23:11:32.521656_gSZE
    • 2023-11-09_23:11:32.521665_Kjzh
    • 2023-11-09_23:11:32.521671_PtoY
    • 2023-11-09_23:11:32.521675_mXWP
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentile58f456fc5087afDiffSignificant (?)
    p10204 ±4 ms209 ±3 ms+4.76 ms (+2.3%)NO
    p25211 ±5 ms215 ±2 ms+4.07 ms (+1.9%)NO
    p50219 ±5 ms222 ±2 ms+3.05 ms (+1.4%)NO
    p75228 ±6 ms230 ±2 ms+2.79 ms (+1.2%)NO
    p90235 ±6 ms241 ±3 ms+6.47 ms (+2.8%)NO

    20 test runs in comparison
    CommitTest Runs
    58f456f
    • 2023-11-09_17:13:00.761896_IpDr
    • 2023-11-09_17:13:00.761928_SVRz
    • 2023-11-09_17:13:00.761940_BeGY
    • 2023-11-09_17:13:00.761949_NgQD
    • 2023-11-09_17:13:00.761956_fRfl
    • 2023-11-09_17:13:00.761963_gepG
    • 2023-11-09_17:13:00.761975_IWCM
    • 2023-11-09_17:13:00.761982_kvPx
    • 2023-11-09_17:13:00.761989_gDss
    • 2023-11-09_17:13:00.761996_rgMa
    c5087af
    • 2023-11-09_23:11:32.521573_bpCS
    • 2023-11-09_23:11:32.521608_AwJM
    • 2023-11-09_23:11:32.521620_ghIQ
    • 2023-11-09_23:11:32.521628_VTrg
    • 2023-11-09_23:11:32.521638_oXyw
    • 2023-11-09_23:11:32.521648_IIXV
    • 2023-11-09_23:11:32.521656_gSZE
    • 2023-11-09_23:11:32.521665_Kjzh
    • 2023-11-09_23:11:32.521671_PtoY
    • 2023-11-09_23:11:32.521675_mXWP
    redfin-30
    Percentile58f456fc5087afDiffSignificant (?)
    p10248 ±5 ms271 ±3 ms+22.6 ms (+9.1%)MAYBE
    p25254 ±5 ms277 ±4 ms+22.8 ms (+9.0%)MAYBE
    p50262 ±5 ms286 ±4 ms+23.2 ms (+8.9%)MAYBE
    p75271 ±6 ms295 ±5 ms+24.1 ms (+8.9%)MAYBE
    p90283 ±7 ms312 ±11 ms+29.0 ms (+10.2%)MAYBE

    20 test runs in comparison
    CommitTest Runs
    58f456f
    • 2023-11-09_17:13:00.761896_IpDr
    • 2023-11-09_17:13:00.761928_SVRz
    • 2023-11-09_17:13:00.761940_BeGY
    • 2023-11-09_17:13:00.761949_NgQD
    • 2023-11-09_17:13:00.761956_fRfl
    • 2023-11-09_17:13:00.761963_gepG
    • 2023-11-09_17:13:00.761975_IWCM
    • 2023-11-09_17:13:00.761982_kvPx
    • 2023-11-09_17:13:00.761989_gDss
    • 2023-11-09_17:13:00.761996_rgMa
    c5087af
    • 2023-11-09_23:11:32.521573_bpCS
    • 2023-11-09_23:11:32.521608_AwJM
    • 2023-11-09_23:11:32.521620_ghIQ
    • 2023-11-09_23:11:32.521628_VTrg
    • 2023-11-09_23:11:32.521638_oXyw
    • 2023-11-09_23:11:32.521648_IIXV
    • 2023-11-09_23:11:32.521656_gSZE
    • 2023-11-09_23:11:32.521665_Kjzh
    • 2023-11-09_23:11:32.521671_PtoY
    • 2023-11-09_23:11:32.521675_mXWP

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

@rlazo rlazo merged commit ab05c99 into master Nov 10, 2023
27 checks passed
@rlazo rlazo deleted the rl.config.interop.init branch November 10, 2023 15:19
rlazo added a commit that referenced this pull request Nov 10, 2023
rlazo added a commit that referenced this pull request Nov 10, 2023
@firebase firebase locked and limited conversation to collaborators Dec 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants