Skip to content

Commit 5717fce

Browse files
committed
Hack: Skip gecko.version and gecko.build_id to get gecko & firefox-desktop back working
1 parent c179422 commit 5717fce

File tree

1 file changed

+14
-0
lines changed

1 file changed

+14
-0
lines changed

probe_scraper/glean_checks.py

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,14 @@
1616

1717
from .scrapers.git_scraper import Commit
1818

19+
# Ugly hack to skip certain metrics which we know aren't duplicated,
20+
# but show up duplicated due to them being moved from one
21+
# to the other application/library.
22+
SKIP_METRICS = {
23+
"gecko.version": ["gecko", "pine", "firefox-desktop"],
24+
"gecko.build_id": ["gecko", "pine", "firefox-desktop"],
25+
}
26+
1927

2028
def _metric_sort_key(metric: Dict[str, Any]):
2129
return (
@@ -142,6 +150,12 @@ def check_for_duplicate_metrics(repositories, metrics_by_repo, emails):
142150
# Exempt cases when one of the sources is Geckoview Streaming to
143151
# avoid false positive duplication accross app channels.
144152
v = [dep for dep in v if "engine-gecko" not in dep]
153+
154+
if k in SKIP_METRICS.keys():
155+
potential_deps = SKIP_METRICS[k]
156+
if any([dep for dep in potential_deps if dep in v]):
157+
continue
158+
145159
if len(v) > 1:
146160
duplicate_sources[k] = v
147161

0 commit comments

Comments
 (0)