-
Notifications
You must be signed in to change notification settings - Fork 225
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 Glean to v32.4.0 #3590
Upgrade Glean to v32.4.0 #3590
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3590 +/- ##
==========================================
+ Coverage 76.77% 77.17% +0.40%
==========================================
Files 230 230
Lines 30928 30383 -545
==========================================
- Hits 23744 23449 -295
+ Misses 7184 6934 -250
Continue to review full report at Codecov.
|
🎉 thank you! |
Glean changelog: * General * Allow using quantity metric type outside of Gecko ([mozilla#1198](mozilla/glean#1198)) * Update `glean_parser` to 1.28.5 * The `SUPERFLUOUS_NO_LINT` warning has been removed from the glinter. It likely did more harm than good, and makes it hard to make metrics.yaml files that pass across different versions of `glean_parser`. * Expired metrics will now produce a linter warning, `EXPIRED_METRIC`. * Expiry dates that are more than 730 days (~2 years) in the future will produce a linter warning, `EXPIRATION_DATE_TOO_FAR`. * Allow using the Quantity metric type outside of Gecko. * New parser configs `custom_is_expired` and `custom_validate_expires` added. These are both functions that take the expires value of the metric and return a bool. (See `Metric.is_expired` and `Metric.validate_expires`). These will allow FOG to provide custom validation for its version-based `expires` values. * Add a limit of 250 pending ping files. ([mozilla#1217](mozilla/glean#1217)). Note: This also gets rid of the 2 workarounds (removed code) in AppService thanks to upstream changes.
319da93
to
5b3f4cf
Compare
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.
👍
Glean changelog:
glean_parser
to 1.28.5SUPERFLUOUS_NO_LINT
warning has been removed from the glinter. It likely did more harm than good, and makes it hard to make metrics.yaml files that pass across different versions ofglean_parser
.EXPIRED_METRIC
.EXPIRATION_DATE_TOO_FAR
.custom_is_expired
andcustom_validate_expires
added. These are both functions that take the expires value of the metric and return a bool. (SeeMetric.is_expired
andMetric.validate_expires
). These will allow FOG to provide custom validation for its version-basedexpires
values.Note: This also gets rid of the 2 workarounds (removed code) in
AppService thanks to upstream changes.
Pull Request checklist
automation/all_tests.sh
runs to completion and produces no failures[ci full]
to the PR title.