Skip to content

Commit

Permalink
Disable sdk-not-ready-for-feature event (#5185) r=travis (#5186)
Browse files Browse the repository at this point in the history
* Disable sdk-not-ready event

* Add changelog

* Ignore test for disabled metrics
  • Loading branch information
jhugman authored Oct 14, 2022
1 parent 2b776cb commit 1e969ab
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 0 deletions.
5 changes: 5 additions & 0 deletions CHANGES_UNRELEASED.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,3 +18,8 @@ Use the template below to make assigning a version number during the release cut
- Description of the change with a link to the pull request ([#0000](https://github.com/mozilla/application-services/pull/0000))
-->

## Nimbus ⛅️🔬🔭

### What's Changed
- Disabled Glean events recorded when the SDK is not ready for a feature ([#5185](https://github.com/mozilla/application-services/pull/5185))
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ import org.junit.Assert.assertNull
import org.junit.Assert.assertTrue
import org.junit.Assert.assertFalse
import org.junit.Before
import org.junit.Ignore
import org.junit.Rule
import org.junit.Test
import org.junit.runner.RunWith
Expand Down Expand Up @@ -461,6 +462,7 @@ class NimbusTests {
}

@Test
@Ignore
fun `in memory cache not ready logs an event`() {
// we haven't initialized nimbus at all, it should not log any error, but it should log an
// event
Expand Down
4 changes: 4 additions & 0 deletions components/nimbus/metrics.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -125,9 +125,11 @@ nimbus_health:
feature_id:
type: string
description: The feature id of the configuration that was requested
disabled: true
bugs:
- https://mozilla-hub.atlassian.net/browse/EXP-2689
- https://mozilla-hub.atlassian.net/browse/EXP-2690
- https://mozilla-hub.atlassian.net/browse/EXP-2852
data_reviews:
- https://github.com/mozilla/application-services/pull/5091#issuecomment-1218359426
data_sensitivity:
Expand All @@ -144,8 +146,10 @@ nimbus_health:
feature_id:
type: string
description: The feature id of the configuration that was requested
disabled: true
bugs:
- https://mozilla-hub.atlassian.net/browse/EXP-2743
- https://mozilla-hub.atlassian.net/browse/EXP-2852
data_reviews:
- https://github.com/mozilla/application-services/pull/5118#issuecomment-1235827006
data_sensitivity:
Expand Down

0 comments on commit 1e969ab

Please sign in to comment.