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

Bumps telemetry expiration dates to 2021-05-01 #3896

Merged
merged 2 commits into from
Nov 10, 2020
Merged

Conversation

keianhzo
Copy link
Contributor

Bumps telemetry expiration dates to 2021-05-01

@keianhzo keianhzo self-assigned this Oct 22, 2020
@keianhzo keianhzo force-pushed the bump-glean-exp-dates branch from f8177b9 to c3b396a Compare October 22, 2020 10:13
@keianhzo
Copy link
Contributor Author

Request for Data Collection Renewal

** This form is for the renewal of an existing, reviewed data collection.**

** All questions are mandatory.
You must receive Data Review from a
Data Steward
on a filled-out Request before shipping your renewed data collection.**

  1. Provide a link to the initial Data Collection Review Request for this collection.
    Migrate telemetry pings to Glean (Stage 2) #2241 (comment)
    Fxa & Tabs Telemetry #2327 (comment)
    Add telemetry for creating new windows #2348 (comment)

  2. When will this collection now expire?
    2021-05-01

  3. Why was the initial period of collection insufficient?
    It is because we would like to check if they are still useful every 6 months.

@keianhzo
Copy link
Contributor Author

Request for Data Collection Renewal

Per #3896 (comment)

@harraton Please help data review.

The metrics document is here, https://github.com/MozillaReality/FirefoxReality/blob/bumpGleanDates/docs/metrics.md, and it will be shown in docs/metrics.md once we merge it.

The wiki page is here, https://github.com/MozillaReality/FirefoxReality/wiki/Telemetry#new-telemetry-system---glean

@keianhzo
Copy link
Contributor Author

@Dexterp37 Please help review as well.

@Dexterp37
Copy link

@Dexterp37 Please help review as well.

Note that this does not require a Glean / telemetry peer. You only need data-reviews and a review from your own team :)

@daoshengmu
Copy link
Contributor

The PR looks good to me. We generally put the data reviewer reply link into the block of data_reviews: instead of using the PR link. You can consider to update it once you have a data-review+

@keianhzo
Copy link
Contributor Author

@Dexterp37 do you know if we are pinging the right steward for the data review here? Any other person we should ping?

@Dexterp37
Copy link

@Dexterp37 do you know if we are pinging the right steward for the data review here? Any other person we should ping?

Hey @keianhzo , the data-review process is documented here. It also includes a list of people and a channel where to ask for help ;-)

@keianhzo
Copy link
Contributor Author

Thanks @Dexterp37!

@chutten Could you please help review the request?

@chutten
Copy link

chutten commented Oct 28, 2020

Is the provided Data Collection Review complete, correct, and data-review+ by a Data Steward?

Yes.

Is the data collection covered by the existing Firefox Privacy Notice?

Yes.


Result: datareview+

@keianhzo keianhzo merged commit f4ff016 into main Nov 10, 2020
@keianhzo keianhzo deleted the bump-glean-exp-dates branch November 10, 2020 15:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants