Update schema to allow for "special" glean_* pings (#301)

* Update schema to allow for "special" glean_* pings

See mozilla/glean#1572

* Move special case to `send_in_pings` only

Author: mdboom

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## Unreleased
 
 - Raise limit on number of statically-defined lables to 100. ([bug 1702263](https://bugzilla.mozilla.org/show_bug.cgi?id=1702263))
+- BUGFIX: Version 2.0.0 of the schema now allows the "special" `glean_.*` ping names for Glean-internal use again.
 
 ## 2.5.0 (2021-02-23)
 

glean_parser/schemas/metrics.2-0-0.schema.yaml

@@ -177,7 +177,12 @@ definitions:
           in any ping.)
         type: array
         items:
-          $ref: "#/definitions/kebab_case"
+          anyOf:
+            - $ref: "#/definitions/kebab_case"
+            # Allow "special" ping names that start with "glean_" used
+            # internally by the Glean SDK
+            - type: string
+              pattern: "^glean_.*$"
         default:
           - default
 

tests/data/core.yaml

@@ -21,6 +21,7 @@ telemetry:
       - CHANGE-ME@example.com
     send_in_pings:
       - core
+      - glean_client_info
     expires: 2100-01-01
 
 core_ping:

tests/test_translate.py

@@ -189,7 +189,7 @@ def test_getting_line_number():
     metrics = load_yaml_or_json(ROOT / "data" / "core.yaml")
 
     assert pings["custom-ping"].defined_in["line"] == 7
-    assert metrics["core_ping"]["seq"].defined_in["line"] == 27
+    assert metrics["core_ping"]["seq"].defined_in["line"] == 28
 
 
 def test_rates(tmpdir):