Closed Bug 1566367 Opened 3 months ago Closed 2 months ago

Implement a new GV API for Glean streaming telemetry

Categories

(GeckoView :: General, task, P1)

Unspecified
Android

Tracking

(firefox68 wontfix, firefox69 ?, firefox70 affected)

RESOLVED FIXED
Tracking Status
firefox68 --- wontfix
firefox69 --- ?
firefox70 --- affected

People

(Reporter: Dexter, Assigned: Agi)

References

(Blocks 1 open bug)

Details

(Keywords: leave-open, Whiteboard: [geckoview:fenix:m8])

Attachments

(5 files, 2 obsolete files)

This is for implementing the new GeckoView streaming API described in the proposal to integrate GeckoView and the Glean SDK, in order to exfiltrate Gecko metrics to Fenix (and other android products based off a-c).

Blocks: 1566374
Assignee: nobody → esawin
Priority: -- → P1
Summary: Implement a new API for streaming telemetry → Implement a new GV API for Glean streaming telemetry
Whiteboard: [geckoview:fenix:m7]
Duplicate of this bug: geckoview-glean

Agi is picking up the Glean/GV work while Eugen is out on leave. This bug is blocked waiting on the Glean work in Gecko.

Assignee: esawin → agi
Whiteboard: [geckoview:fenix:m7] → [geckoview:fenix:m8]

(In reply to Chris Peterson [:cpeterson] from comment #6)

Agi is picking up the Glean/GV work while Eugen is out on leave. This bug is blocked waiting on the Glean work in Gecko.

This is not necessarily blocked on bug 1566366: this is for providing the GV API and the JNI endpoints, bug 1566366 is for introducing a telemetry delegate. Implementing the delegate to hook up GV and Telemetry (patch [2.0] in this bug) can be moved in a follow-up bug or as part of bug 1566366 .

My only concern with this would be that we had to clear the the outstanding interface question to avoid changing the GV API.

I'm editing a bunch of GeckoView bugs. If you'd like to filter all this bugmail, search and destroy emails containing this UUID:

e88a5094-0fc0-4b7c-b7c5-aef00a11dbc9

Priority: P1 → P2

(In reply to Chris Peterson [:cpeterson] from comment #8)

I'm editing a bunch of GeckoView bugs. If you'd like to filter all this bugmail, search and destroy emails containing this UUID:

Hey Chris, is P2 indicating that the priority on this dropped? This is a vital piece of the GeckoView + Glean SDK integration :)

(In reply to Alessio Placitelli [:Dexter] from comment #9)

Hey Chris, is P2 indicating that the priority on this dropped? This is a vital piece of the GeckoView + Glean SDK integration :)

I'll restore priority P1. Agi is working on this bug.

OS: All → Android
Priority: P2 → P1
Attachment #9079785 - Attachment is obsolete: true
Attachment #9082840 - Attachment is obsolete: true

Landing the javadoc fix

Keywords: leave-open
Pushed by asferro@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/adb5e7efc20a
[0.0] Fix javadoc in WebExtensionController. r=snorp
Pushed by asferro@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cf556b5d97f1
[1.0] Add GV RuntimeTelemetry delegate. r=snorp
https://hg.mozilla.org/integration/autoland/rev/7c7e74beed99
[2.0] Add streaming telemetry Gecko-GV bridge. r=snorp,chutten
https://hg.mozilla.org/integration/autoland/rev/45d2398750a2
[3.0] Add GVE example telemetry delegate. r=snorp
https://hg.mozilla.org/integration/autoland/rev/3f5971e7dc98
[4.0] Add test for onTelemetryReceived. r=chutten,Dexter,snorp

🎉

Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.