1799785 - Enable new Glean App `Treeherder`

Status: RESOLVED FIXED

(Data Platform and Tools Graveyard :: Glean Platform, task, P1)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

To be filled by the requester

Application ID*: Treeherder
Application Canonical Name: Treeherder
Description: Internal Developer dashboard for CI results
Data-review response link: TBD
Repository URL: https://github.com/mozilla/treeherder
Locations of metrics.yaml files (can be many):

• glean/metrics.yaml

Locations of pings.yaml files (can be many):

• glean/pings.yaml

Dependencies**:

• glean-core

Retention Days***: N

(Optional) To be filled by the requester

Does the product require end-to-end encryption in the pipeline? No
If answered yes to the above, who should be granted access to the data?

• LDAP account 1
• LDAP account 2

Notes and guidelines

* This is the identifier used to initialize Glean (or the id used on the store on Android and Apple devices).

** Dependencies can be found in the Glean repositories. Each dependency must be listed explicitly. For example, the default Glean probes will only be included if glean itself is a dependency.

*** Number of days that raw data will be retained. A good default is 180. We can change this later to accommodate longer retention periods, though we cannot recover data that is past the retention period (for example, we cannot recover data that is 200 days old if your retention period is 180 days).
180

Need additional help?

If you need new dependencies, please file new bugs for them, separately from this one. For any questions, ask in the #glean channel.

To be filled by the Glean team

Application friendly name: my_app_name

The following are only required for products requiring encryption:

Document namespace: my-app-name

Please NI Operations on this bug to request the creation of encryption keys and an analysis project.

Comment 1

[Alessio Placitelli [:Dexter]]

Hey Joel,

is the implementation happening in a PR? Did you already add metrics/make sure everything is working via the Glean Debug View/local debugging?

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

I haven't been able to get anything to work, but I saw one of the first steps was to add an app, so I filed this bug. In general I am pretty lost and I see more new comments in the thread I started in #glean on slack.

Comment 3

[Alessio Placitelli [:Dexter]]

Hey Joel, I know you met with Bruno. Please let me know once you're satisfied with the collection and want to start collecting the data in the pipeline.

Happy to support more if you have questions!

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

I have a plan now. Ideally next week I can look to add treeherder to the app list and work on a data review.

Comment 5

[Alessio Placitelli [:Dexter]]

(In reply to Joel Maher ( :jmaher ) (UTC -8) from comment #4)

I have a plan now. Ideally next week I can look to add treeherder to the app list and work on a data review.

Great, glad to hear about this. If I may, I'd recommend prototyping and testing in the Glean Debug View before requesting the addition to the pipeline.

You don't need to worry about adding to the pipeline yourself. Once you're ready, I'll do the addition :)

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

I have the code working as expected, probably a few loose ends to clean up. I would like to start working on data review and sort out making this a production deployment. Please advise on what to do next.

Comment 7

[Alessio Placitelli [:Dexter]]

(In reply to Joel Maher ( :jmaher ) (UTC -8) from comment #6)

I have the code working as expected, probably a few loose ends to clean up. I would like to start working on data review and sort out making this a production deployment. Please advise on what to do next.

As reported by our docs, once you have the integration working in some branch/PR, you should file a data-review request on that PR/bug.

I'm also happy to take an initial look at the integration before moving on to enabling this on the pipeline.

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

:dexter, I have https://github.com/mozilla/treeherder/pull/7583, if you could take a quick look and see if there are red flags. Once done with that I will file a new bug for a data-review.

Comment 9

[Alessio Placitelli [:Dexter]]

(In reply to Joel Maher ( :jmaher ) (UTC -8) from comment #8)

:dexter, I have https://github.com/mozilla/treeherder/pull/7583, if you could take a quick look and see if there are red flags. Once done with that I will file a new bug for a data-review.

Took a first pass!

Comment 10

[Jan-Erik Rediger [:janerik]]

Attachment: pr 538: Bug 1799785 - Add Treeherder

Comment 11

[Jan-Erik Rediger [:janerik]]

badboy merged PR #538: "Bug 1799785 - Add Treeherder" in 6380c4c.

Comment 12

[Jan-Erik Rediger [:janerik]]

Attachment: Treeherder PR: Add GhA task to run the probe-scraper workflow

Attaching the other PR for completeness as well.