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

Add Cobalt Telemetry for the Loader App #2667

Merged
merged 2 commits into from
Mar 25, 2024
Merged

Conversation

hlwarriner
Copy link
Contributor

A LoaderAppMetrics Starboard extension is added so that measurements taken by the Loader App can be reliably accessed by Cobalt once loaded. This PR just adds one metric for now, for b/329458881, but this pattern and extension can be used for b/329445690 and other Loader App metrics.

b/329458881

Change-Id: I940d07b058d197afa2ad8b1160eba469f1055f8a

A LoaderAppMetrics Starboard extension is added so that measurements
taken by the Loader App can be reliably accessed by Cobalt once loaded.
This PR just adds one metric for now, for b/329458881, but this pattern
and extension can be used for b/329445690 and other Loader App metrics.

b/329458881

Change-Id: I940d07b058d197afa2ad8b1160eba469f1055f8a
Copy link

codecov bot commented Mar 21, 2024

Codecov Report

Attention: Patch coverage is 77.77778% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 58.25%. Comparing base (db11969) to head (f63aade).
Report is 57 commits behind head on main.

❗ Current head f63aade differs from pull request most recent head d0add78. Consider uploading reports for the commit d0add78 to get more accurate results

Files Patch % Lines
starboard/shared/starboard/loader_app_metrics.cc 66.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2667      +/-   ##
==========================================
- Coverage   58.25%   58.25%   -0.01%     
==========================================
  Files        1794     1798       +4     
  Lines       84889    84936      +47     
==========================================
+ Hits        49450    49477      +27     
- Misses      35439    35459      +20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hlwarriner
Copy link
Contributor Author

Cc @yuying-y

Change-Id: Ie689f7b020f624c21e42d16ec59cb535440638c7
@hlwarriner hlwarriner merged commit bf93736 into youtube:main Mar 25, 2024
338 of 339 checks passed
@hlwarriner hlwarriner added the cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch label Apr 2, 2024
cobalt-github-releaser-bot pushed a commit that referenced this pull request Apr 2, 2024
A LoaderAppMetrics Starboard extension is added so that measurements
taken by the Loader App can be reliably accessed by Cobalt once loaded.
This PR just adds one metric for now, for b/329458881, but this pattern
and extension can be used for b/329445690 and other Loader App metrics.

b/329458881

Change-Id: I940d07b058d197afa2ad8b1160eba469f1055f8a
(cherry picked from commit bf93736)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants