Skip to content

Commit

Permalink
Merge pull request #8386 from mozilla-mobile/5718-update-event-names
Browse files Browse the repository at this point in the history
VPN-5718 - Update metric names based on new standards
  • Loading branch information
brizental authored Nov 2, 2023
2 parents d1e3798 + 0d3b073 commit 4f377fa
Show file tree
Hide file tree
Showing 14 changed files with 1,018 additions and 1,045 deletions.
3 changes: 3 additions & 0 deletions android/daemon/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,9 @@ apply plugin: "org.mozilla.telemetry.glean-gradle-plugin"
ext.gleanNamespace = "mozilla.telemetry.glean"
ext.gleanYamlFiles = [
"$rootDir/../../../src/telemetry/metrics.yaml",
"$rootDir/../../../src/telemetry/interaction_metrics.yaml",
"$rootDir/../../../src/telemetry/impression_metrics.yaml",
"$rootDir/../../../src/telemetry/outcome_metrics.yaml",
"$rootDir/../../../src/telemetry/pings.yaml"
]

Expand Down
3 changes: 3 additions & 0 deletions android/vpnClient/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,9 @@ apply plugin: "org.mozilla.telemetry.glean-gradle-plugin"
ext.gleanNamespace = "mozilla.telemetry.glean"
ext.gleanYamlFiles = [
"$rootDir/../../../src/telemetry/metrics.yaml",
"$rootDir/../../../src/telemetry/interaction_metrics.yaml",
"$rootDir/../../../src/telemetry/impression_metrics.yaml",
"$rootDir/../../../src/telemetry/outcome_metrics.yaml",
"$rootDir/../../../src/telemetry/pings.yaml"
]

Expand Down
7 changes: 6 additions & 1 deletion qtglean/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,12 @@ else()
endif()

list(APPEND PINGS_LIST ${CMAKE_SOURCE_DIR}/src/telemetry/pings.yaml)
list(APPEND METRICS_LIST ${CMAKE_SOURCE_DIR}/src/telemetry/metrics.yaml)
list(APPEND METRICS_LIST
${CMAKE_SOURCE_DIR}/src/telemetry/metrics.yaml
${CMAKE_SOURCE_DIR}/src/telemetry/interaction_metrics.yaml
${CMAKE_SOURCE_DIR}/src/telemetry/impression_metrics.yaml
${CMAKE_SOURCE_DIR}/src/telemetry/outcome_metrics.yaml
)

## Generate metrics and pings files
add_custom_command(
Expand Down
3 changes: 3 additions & 0 deletions qtglean/glean_parser_ext/run_glean_parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,9 @@ def create_dir(path):
]
metrics_files = [
os.path.join(telemetry_path, "metrics.yaml"),
os.path.join(telemetry_path, "interaction_metrics.yaml"),
os.path.join(telemetry_path, "impression_metrics.yaml"),
os.path.join(telemetry_path, "outcome_metrics.yaml"),
]

# Generate C++ header files
Expand Down
312 changes: 312 additions & 0 deletions src/telemetry/impression_metrics.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,312 @@
# This Source Code Form is subject to the terms of the Mozilla Public
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.

---
$schema: moz://mozilla.org/schemas/glean/metrics/2-0-0

# "impression" category events, are events where the user has just moved on to page X
# Events under this category are recorded when a new screen is loaded
impression:
main_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has just moved on to the main screen
i.e. the screen with the VPN toggle button.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5718
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8386#issuecomment-1780543747
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys: &impression_extra_keys
screen:
description: The id of the screen the user is on
type: string
speed_test_loading_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has just moved on to the screen that shows
the loading animation for the speed test results
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5366
- https://mozilla-hub.atlassian.net/browse/VPN-5718
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8060#issuecomment-1724272061
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8386#issuecomment-1780543747
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys: *impression_extra_keys
speed_test_completed_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has just moved on to the screen that shows
the speed test results
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5366
- https://mozilla-hub.atlassian.net/browse/VPN-5718
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8060#issuecomment-1724272061
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8386#issuecomment-1780543747
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys: *impression_extra_keys
speed_test_error_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has just moved on to the screen that shows
the speed test error
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5366
- https://mozilla-hub.atlassian.net/browse/VPN-5718
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8060#issuecomment-1724272061
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8386#issuecomment-1780543747
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys: *impression_extra_keys
connection_info_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has just moved on to the screen that shows
the connection information
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5366
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8060#issuecomment-1724272061
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys: *impression_extra_keys
signup_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has entered the "Sign up" screen.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5251
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8411#issuecomment-1783192931
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys:
screen:
description: |
The screen where the interaction happened
type: string
enter_email_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has entered the "Enter email" screen.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5251
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8411#issuecomment-1783192931
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys:
screen:
description: |
The screen where the interaction happened
type: string
create_password_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has entered the "Create password" screen.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5251
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8411#issuecomment-1783192931
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys:
screen:
description: |
The screen where the interaction happened
type: string
enter_verification_code_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has entered the screen to enter email verification code.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5251
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8411#issuecomment-1783192931
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys:
screen:
description: |
The screen where the interaction happened
type: string
enter_password_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has entered the "Enter password" screen.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5251
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8411#issuecomment-1783192931
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys:
screen:
description: |
The screen where the interaction happened
type: string
enter_security_code_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has entered the screen to enter a TOTP security code.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5251
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8411#issuecomment-1783192931
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys:
screen:
description: |
The screen where the interaction happened
type: string
subscription_needed_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has entered the "Subscription needed" screen.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5251
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8411#issuecomment-1783192931
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys:
screen:
description: |
The screen where the interaction happened
type: string
confirming_subscription_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has entered the "Confirming subscription" screen.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5251
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8411#issuecomment-1783192931
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys:
screen:
description: |
The screen where the interaction happened
type: string
continue_in_browser_screen:
type: event
lifetime: ping
send_in_pings:
- main
description: |
The user has entered the "Continue in browser" screen.
bugs:
- https://mozilla-hub.atlassian.net/browse/VPN-5251
data_reviews:
- https://github.com/mozilla-mobile/mozilla-vpn-client/pull/8411#issuecomment-1783192931
data_sensitivity:
- interaction
notification_emails:
- [email protected]
- [email protected]
expires: never
extra_keys:
screen:
description: |
The screen where the interaction happened
type: string
Loading

0 comments on commit 4f377fa

Please sign in to comment.