Closed Bug 1665284 Opened 5 years ago Closed 5 years ago

GV Nightly busted: GleanGeckoMetricsMapping.kt: Duplicate label in when

Categories

(Core :: Performance: General, defect, P1)

defect

Tracking

()

RESOLVED FIXED
82 Branch
Tracking Status
firefox-esr68 --- unaffected
firefox-esr78 --- unaffected
firefox80 --- unaffected
firefox81 --- unaffected
firefox82 blocking fixed

People

(Reporter: Dexter, Assigned: beth)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [telemetry:glean-rs:m?])

Attachments

(1 file)

See the original issue here.

The GV nightly build task seems to have failed from a generated class in Glean:

> Task :browser-engine-gecko-nightly:compileDebugKotlin FAILED
e: warnings found and -Werror specified
w: /build/android-components/components/browser/engine-gecko-nightly/build/generated/source/glean/debug/kotlin/GleanGeckoMetricsMapping.kt: (95, 13): Duplicate label in when

> Task :service-glean:generateReleaseRFile

Task: https://firefox-ci-tc.services.mozilla.com/tasks/groups/DDM8oIucQRyivl5-c5GUgQ
Logs: https://firefoxci.taskcluster-artifacts.net/DDM8oIucQRyivl5-c5GUgQ/0/public/logs/live_backing.log

This is blocking our automated GV Nightly version update in AC. Which means the latest GV Nightly won't get into Fenix Nightly. And then a bunch of people will start to complain. :)

Assignee: nobody → mdroettboom

The issue is that a duplicate telemetry id was introduced in the GV metrics.yaml file here:

https://searchfox.org/mozilla-central/source/toolkit/components/telemetry/geckoview/streaming/metrics.yaml#1082

I think this needs TIME_TO_LOAD_EVENT_END_NO_PRELOAD_MS -> TIME_TO_LOAD_EVENT_START_NO_PRELOAD_MS.

Assignee: mdroettboom → nobody
Component: Glean: SDK → Telemetry
Product: Data Platform and Tools → Toolkit
Regressed by: 1657188
Has Regression Range: --- → yes

The problematic definitions were added in bug 1657188 in the Core::Performance product, so I'll move the bug over there.

Component: Telemetry → Performance
Product: Toolkit → Core

Bug 1657188 introduced some new telemetry, but one of the metrics had an
incorrect gecko_datapoint, causing GV builds to fail.

Assignee: nobody → brennie
Status: NEW → ASSIGNED
Pushed by brennie@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/787605ddd0b1 Correct gecko_datapoint for load_event_start_no_preload r=Dexter
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: