-
Notifications
You must be signed in to change notification settings - Fork 1.3k
For #22870: Renew/remove metrics set to expire in February #23453
For #22870: Renew/remove metrics set to expire in February #23453
Conversation
fe0da9e
to
cb6240e
Compare
Request for Data Collection RenewalNever expireTotal: 9 ———
———
———
———
———
———
================== ———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
———
|
Data Collection Renewal Review
Yes
Yes Resultdata-review+ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! Update the data review URL and 🚢. Thanks for doing this.
dbc4ee1
to
51fb215
Compare
@Mergifyio backport releases_v97.0.0 |
✅ Backports have been created
|
…ary (mozilla-mobile#23453) * For mozilla-mobile#22870: Renew/remove metrics set to expire in February * Add data review and add notification owner for never expire metrics
Uplift PR: #23496 |
* For #22870: Renew/remove metrics set to expire in February (#23453) * For #22870: Renew/remove metrics set to expire in February * Add data review and add notification owner for never expire metrics * Close #22956: Define "tags" for metrics in the app repository Co-authored-by: William Lachance <wlach@protonmail.com> Define "tags" for metrics in the app repository Using the new "tags" feature in Glean, we can remove most of the manual process/pain of defining tags in a seperate Glean annotations repository (see: https://github.com/mozilla-mobile/fenix/wiki/Add-a-Glean-Annotation-for-an-event) and just put them beside the metrics themselves. This should make it much easier to keep this metadata up to date, since it can be added in the same pull request as the instrumentation itself. To keep the list of tags up to date with the Fenix issue labels, a new script `update-glean-tags.py` has been added to the repository. It should not need to be run often. This pull request uses a scraped version of the defined tags in the Glean annotations repository. After it lands, we can remove those and just use the tagging information here as the main source of information. Co-authored-by: Roger Yang <royang@mozilla.com>
@Mergifyio backport releases_v96.0.0 |
✅ Backports have been created
|
* For #22870: Renew/remove metrics set to expire in February (#23453) * For #22870: Renew/remove metrics set to expire in February * Add data review and add notification owner for never expire metrics * Close #22956: Define "tags" for metrics in the app repository Co-authored-by: William Lachance <wlach@protonmail.com> Define "tags" for metrics in the app repository Using the new "tags" feature in Glean, we can remove most of the manual process/pain of defining tags in a seperate Glean annotations repository (see: https://github.com/mozilla-mobile/fenix/wiki/Add-a-Glean-Annotation-for-an-event) and just put them beside the metrics themselves. This should make it much easier to keep this metadata up to date, since it can be added in the same pull request as the instrumentation itself. To keep the list of tags up to date with the Fenix issue labels, a new script `update-glean-tags.py` has been added to the repository. It should not need to be run often. This pull request uses a scraped version of the defined tags in the Glean annotations repository. After it lands, we can remove those and just use the tagging information here as the main source of information. Co-authored-by: Roger Yang <royang@mozilla.com> (cherry picked from commit 08790a0)
…23453) (#23510) * For #22870: Renew/remove metrics set to expire in February (backport #23453) * Issue ##22870: Temporarily ignoring tests for StorageStatsMetrics * No issue: Update PWA package name in mainMenuInstallPWATest (#23335) Co-authored-by: Jonathan Almeida <jalmeida@mozilla.com> Co-authored-by: Aaron Train <aaron.train@gmail.com>
…ary (mozilla-mobile#23453) * For mozilla-mobile#22870: Renew/remove metrics set to expire in February * Add data review and add notification owner for never expire metrics
For #22870
Pull Request checklist
To download an APK when reviewing a PR: