Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

Commit

Permalink
fix(glean): update links
Browse files Browse the repository at this point in the history
  • Loading branch information
wtfluckey committed Oct 11, 2023
1 parent 8ada20f commit 9308121
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions telemetry/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,18 +16,18 @@ When this file has been updated, Kirill will notify us and we need to make chang
2. Paste that raw JSON in the [`metrics.yaml`](./metrics.yaml) file here in elk, replacing everything that was there before
3. Check git diff to make sure I understand what is changing between the previous version and this one
4. Add back the things that the shared [`metrics.yaml`](https://gist.github.com/kirill-demtchouk/41a38336b61b938833fe011bda4a2001) file removes
- [Add lifetime parameter back](#lifetime-parameter)
- [Add `no_lint` for baseline pings](#no-lint-baseline)
- [Add lifetime parameter back](#add-lifetime-parameter-back-to-metrics-that-arent-events)
- [Add `no_lint` for baseline pings](#add-no_lint-for-baseline-pings)
5. Remove the extra things in the shared [`metrics.yaml`](https://gist.github.com/kirill-demtchouk/41a38336b61b938833fe011bda4a2001) file that aren't specific to web
- [Remove uneeded metrics](#remove-uneeded)
- [Remove uneeded metrics](#remove-metrics-not-needed-for-web)
6. Now that we've added back all the things that we need for web and removed the extraneous details that we don't need, we need to make sure that all the new additions and removals that we are trying to incorporate from the shared [`metrics.yaml`](<https://gist.github.com/kirill-demtchouk/41a38336b61b938833fe011bda4a2001>) file are included here in elk's [`metric.yaml`](./metrics.yaml) file
7. Run the `build:glean` script
8. Delete any old generated files (if needed)
8a. If a category gets removed from the `metrics.yaml` file, glean does not automatically remove the old corresponding generated file, so we need to do that manually

## Web-specific `metrics.yaml` file information

### Remove metrics not needed for web {#remove-unneeded}
### Remove metrics not needed for web

1. In the `identifiers` category:
- remove `fxa_account_id` - the elk client doesn't get the FxA ID back during auth, only a token so we will never have this
Expand All @@ -36,7 +36,7 @@ When this file has been updated, Kirill will notify us and we need to make chang
2. Remove the `backend` category
- the whole thing, including the `object_updated` metric

### Add lifetime parameter back to metrics that aren't events {#lifetime-parameter}
### Add lifetime parameter back to metrics that aren't events

For the events with a type other than `event`, we need to include the `lifetime: application` parameter to make it so that these metrics show up with every ping.

Expand All @@ -51,7 +51,7 @@ mastodon_account_id:

Note: Aly & Daniel are to sync up on this and try to figure out if there's a different way to accomplish this without using `lifetime: application`. But for now, make sure these are added back when updating the `metrics.yaml` file.

### Add `no_lint` for baseline pings {#no-lint-baseline}
### Add `no_lint` for baseline pings

Some of the metrics in the shared [`metrics.yaml`](https://gist.github.com/kirill-demtchouk/41a38336b61b938833fe011bda4a2001) file include `baseline` in the `send_in_pings` parameter. If we keep this, the `build:glean` script will fail to build. Rather than removing it, we have opted for adding a `no_lint` declaration instead.

Expand Down

0 comments on commit 9308121

Please sign in to comment.