1891509 - Implement the telemetry required to capture the user response

Status: RESOLVED FIXED

(Firefox for Android :: Experimentation and Telemetry, task, P1)

Description

[twhite]

Add telemetry that captures the user's response to a survey question.

Comment 1

[twhite]

Attachment: Bug 1891509 - Add telemetry to capture micro survey user response

Comment 2

[twhite]

Comment 3

[twhite]

Attachment: Data review 1891509

Comment 4

[twhite]

Attachment: Data review 1891509

Comment 5

[Roger Yang [:royang]]

Comment on attachment 9397382 [details]
Data review 1891509

DATA COLLECTION REVIEW RESPONSE:

1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?

Yes, through the metrics.yaml file and the Glean Dictionary.

2. Is there a control mechanism that allows the user to turn the data collection on and off?

Yes. This collection is Telemetry so can be controlled through Firefox's Preferences.

3. If the request is for permanent data collection, is there someone who will monitor the data over time?

[e-mail] will be responsible for monitoring the data over time

4. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 2, Interaction data

5. Is the data collection request for default-on or default-off?

Default on for all channels.

6. Does the instrumentation include the addition of any new identifiers?

No

7. Is the data collection covered by the existing Firefox privacy notice?

Yes

8. Does the data collection use a third-party collection tool?

No

Result: data-review+

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/074b8a59783f
Add telemetry to capture micro survey user response r=Roger,android-reviewers

Comment 7

[Cosmin Sabou [:CosminS]]

Backed out changeset 074b8a59783f (Bug 1891509) for causing fenix lint failures.

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=074b8a59783fe1d8f28029851f978ffe9241e3ad&selectedTaskRun=PK6R47GgRkqal2tjsA7DnA.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=455404732&repo=autoland

Backout link: https://hg.mozilla.org/integration/autoland/rev/8a81f8b96a87f026dc90386b15691571d0e2997a

[task 2024-04-22T16:54:47.662Z]   Requirement not upgraded as not directly required: attrs>=17.4.0 in /builds/worker/checkouts/gecko/mobile/android/gradle/dotgradle-online/glean/bootstrap-4.5.11/Miniconda3/lib/python3.7/site-packages (from jsonschema>=3.0.2->glean_parser~=13.0) (23.2.0)
[task 2024-04-22T16:54:47.662Z]   Requirement not upgraded as not directly required: pyrsistent!=0.17.0,!=0.17.1,!=0.17.2,>=0.14.0 in /builds/worker/checkouts/gecko/mobile/android/gradle/dotgradle-online/glean/bootstrap-4.5.11/Miniconda3/lib/python3.7/site-packages (from jsonschema>=3.0.2->glean_parser~=13.0) (0.19.3)
[task 2024-04-22T16:54:47.662Z]   Requirement not upgraded as not directly required: pkgutil-resolve-name>=1.3.10; python_version < "3.9" in /builds/worker/checkouts/gecko/mobile/android/gradle/dotgradle-online/glean/bootstrap-4.5.11/Miniconda3/lib/python3.7/site-packages (from jsonschema>=3.0.2->glean_parser~=13.0) (1.3.10)
[task 2024-04-22T16:54:47.662Z]   Requirement not upgraded as not directly required: MarkupSafe>=2.0 in /builds/worker/checkouts/gecko/mobile/android/gradle/dotgradle-online/glean/bootstrap-4.5.11/Miniconda3/lib/python3.7/site-packages (from Jinja2>=2.10.1->glean_parser~=13.0) (2.1.5)
[task 2024-04-22T16:54:47.662Z]   Requirement not upgraded as not directly required: zipp>=3.1.0; python_version < "3.10" in /builds/worker/checkouts/gecko/mobile/android/gradle/dotgradle-online/glean/bootstrap-4.5.11/Miniconda3/lib/python3.7/site-packages (from importlib-resources>=1.4.0; python_version < "3.9"->jsonschema>=3.0.2->glean_parser~=13.0) (3.15.0)
[task 2024-04-22T16:54:47.662Z]   ??? Found 1 errors.
[task 2024-04-22T16:54:47.662Z]   Sorry, Glean found some glinter nits:
[task 2024-04-22T16:54:47.662Z]   ERROR: INVALID_TAGS: micro_survey.response: Invalid tags specified in metric: MicroSurvey
[task 2024-04-22T16:54:47.662Z]   
[task 2024-04-22T16:54:47.662Z]   Please fix the above nits to continue.
[task 2024-04-22T16:54:47.662Z]   To disable a check, add a `no_lint` parameter with a list of check names to disable.
[task 2024-04-22T16:54:47.662Z]   This parameter can appear with each individual metric, or at the top-level to affect the entire file.
[task 2024-04-22T16:54:47.662Z]   ERROR running glean_parser v13.0.1

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/542cbb16e2f4
Add telemetry to capture micro survey user response r=Roger,android-reviewers,vdreghici

Comment 9

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/542cbb16e2f4