Closed Bug 1618912 Opened 4 years ago Closed 4 years ago

Design a Process for updating the Glean SDK & glean_parser in mozilla-central

Categories

(Toolkit :: Telemetry, task, P1)

task

Tracking

()

RESOLVED FIXED

People

(Reporter: chutten, Assigned: Dexter)

Details

(Whiteboard: [telemetry:fog:m2])

Attachments

(1 file)

Project FOG uses both the Glean SDK and glean_parser. On Android, Gradle can be used to keep these up-to-date and in-sync.

In m-c we're currently relying on a fairly manual process for vendoring glean_parser (bug 1602773), and the Glean SDK is coming in via mach vendor rust thanks to artefacts published to crates.io (bug 1596132).

Propose a process to update the sdk and parser that will

  • Keep them in sync
  • Be straightforward if not also simple
  • Does not require special roles/powers that only a few team members will have
  • Understands that the release schedule of the Glean SDK and Firefox Desktop and Geckoview are not required to line up

This process may rely on humans to start, but it'd be ideal if it considered how we might use tooling to reduce the effort in future. This design should probably also include a survey/summary of how and how often these pieces have updated in the recent past.

This process will be documented in the Project FOG documentation (bug 1618910) after it has been proposed and accepted.

Assignee: nobody → alessio.placitelli
Priority: -- → P1
Attachment #9132834 - Flags: feedback?(mdroettboom)
Attachment #9132834 - Flags: feedback?(jrediger)
Attachment #9132834 - Flags: feedback?(chutten)
Comment on attachment 9132834 [details]
Doc for updating dependencies

Seems correct and complete, modulo some tiny details that are being hashed out.
Attachment #9132834 - Flags: feedback?(chutten) → feedback+
Comment on attachment 9132834 [details]
Doc for updating dependencies

I'm with :chutten, proposal seems solid and the few questions left are getting answered.
Attachment #9132834 - Flags: feedback?(jrediger) → feedback+
Comment on attachment 9132834 [details]
Doc for updating dependencies

Looks good to me.  Let's not block on the glean_parser/glean integration possibility, but document the more complicated thing proposed in the doc and possibly remove it later.
Attachment #9132834 - Flags: feedback?(mdroettboom) → feedback+

Closing this, all the feedback in the doc was addressed!

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

Attachment

General

Created:
Updated:
Size: