1917939 - Add telemetry metric for vivo partnership

Status: RESOLVED FIXED

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

Description

[John Oberhauser]

Ensure these metrics are in place

first_session.distribution_id
metrics.distribution_id

Remove app_opened event extra key: deal_id

Comment 1

[John Oberhauser]

Attachment: Bug 1917939: Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions. Removing the deal_id extra key from the app_opened event because it's no longer needed. Updating some naming of classes.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3410ee200ff1
Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions.  Removing the deal_id extra key from the app_opened event because it's no longer needed.  Updating some naming of classes. r=android-reviewers,gmalekpour,twhite

Comment 3

[Sandor Molnar[:smolnar]]

Backed out for causing Fenix failures

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

Push with failures

Failure log -> TEST-UNEXPECTED-FAIL | org.mozilla.fenix.components.metrics.FirstSessionPingTest.checkAndSend() triggers the ping if it wasn't marked as triggered | java.lang.AssertionError: Verification failed: call 1 of 1: FirstSessionPing

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e901496a2979
Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions.  Removing the deal_id extra key from the app_opened event because it's no longer needed.  Updating some naming of classes. r=android-reviewers,gmalekpour,twhite

Comment 5

[Atila Butkovits]

Backed out for causing Fenix failures.

Backout link: https://hg.mozilla.org/integration/autoland/rev/3e1291aad7d9d1aa7537d08ddcd27a3a95d7dc6d

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=e901496a29795207ae28e679332f6527f45dd6f4

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

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0c1a6577204f
Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions.  Removing the deal_id extra key from the app_opened event because it's no longer needed.  Updating some naming of classes. r=android-reviewers,gmalekpour,twhite

Comment 7

[tszentpeteri]

Backed out for causing fenix failures related to FirstSessionPingTest.checkAndSend().

• Backout link
• Push with failures
• Failure Log:

[task 2024-10-03T22:16:34.690Z] SUITE: org.mozilla.fenix.components.metrics.FirstSessionPingTest
[task 2024-10-03T22:16:34.690Z]   TEST: WHEN build version is more than R THEN installSourcePackage returns the set package name
[task 2024-10-03T22:16:35.189Z]   SUCCESS
[task 2024-10-03T22:16:35.190Z] 
[task 2024-10-03T22:16:35.190Z]   TEST: checkAndSend() triggers the ping if it wasn't marked as triggered
[task 2024-10-03T22:16:35.489Z]     Exception in thread "DefaultDispatcher-worker-3 @coroutine#550" io.mockk.MockKException: no answer found for BrowserStore(#1383).getState() among the configured answers: ()
[task 2024-10-03T22:16:35.490Z]     at io.mockk.impl.stub.MockKStub.defaultAnswer(MockKStub.kt:93)
[task 2024-10-03T22:16:35.490Z]     at io.mockk.impl.stub.MockKStub.answer(MockKStub.kt:44)
[task 2024-10-03T22:16:35.490Z]     at io.mockk.impl.recording.states.AnsweringState.call(AnsweringState.kt:16)
[task 2024-10-03T22:16:35.490Z]     at io.mockk.impl.recording.CommonCallRecorder.call(CommonCallRecorder.kt:53)
[task 2024-10-03T22:16:35.490Z]     at io.mockk.impl.stub.MockKStub.handleInvocation(MockKStub.kt:271)
[task 2024-10-03T22:16:35.490Z]     at io.mockk.impl.instantiation.JvmMockFactoryHelper$mockHandler$1.invocation(JvmMockFactoryHelper.kt:24)
[task 2024-10-03T22:16:35.490Z]     at io.mockk.proxy.jvm.advice.Interceptor.call(Interceptor.kt:21)
[task 2024-10-03T22:16:35.490Z]     at mozilla.components.lib.state.Store.getState(Store.kt:67)
[task 2024-10-03T22:16:35.490Z]     at org.mozilla.fenix.distributions.DistributionIdUtilKt.getDistributionId(DistributionIdUtil.kt:36)
[task 2024-10-03T22:16:35.490Z]     at org.mozilla.fenix.distributions.DistributionIdUtilKt.getDistributionId$default(DistributionIdUtil.kt:32)
[task 2024-10-03T22:16:35.490Z]     at org.mozilla.fenix.components.metrics.FirstSessionPing$triggerPing$1.invokeSuspend(FirstSessionPing.kt:71)
[task 2024-10-03T22:16:35.490Z]     at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
[task 2024-10-03T22:16:35.490Z]     at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:104)
[task 2024-10-03T22:16:35.490Z]     at kotlinx.coroutines.internal.LimitedDispatcher$Worker.run(LimitedDispatcher.kt:111)
[task 2024-10-03T22:16:35.490Z]     at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:99)
[task 2024-10-03T22:16:35.490Z]     at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:584)
[task 2024-10-03T22:16:35.490Z]     at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:811)
[task 2024-10-03T22:16:35.490Z]     at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:715)
[task 2024-10-03T22:16:35.490Z]     at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:702)
[task 2024-10-03T22:16:35.490Z]     Suppressed: kotlinx.coroutines.internal.DiagnosticCoroutineContextException: [CoroutineId(550), "coroutine#550":StandaloneCoroutine{Cancelling}@7416dfbf, Dispatchers.IO]
[task 2024-10-03T22:16:40.590Z]   TEST-UNEXPECTED-FAIL | org.mozilla.fenix.components.metrics.FirstSessionPingTest.checkAndSend() triggers the ping if it wasn't marked as triggered | java.lang.AssertionError: Verification failed: call 1 of 1: FirstSessionPing(#1384).markAsTriggered$app_fenixDebug()) was not called.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/614abe81fac6
Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions.  Removing the deal_id extra key from the app_opened event because it's no longer needed.  Updating some naming of classes. r=android-reviewers,gmalekpour,twhite

Comment 9

[Noemi Erli[:noemi_erli]]

Backed out 2 changesets (Bug 1917939, Bug 1920176) for causing fenix failures in DistributionIdUtilTest CLOSED TREE

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&collapsedPushes=842382&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=74017e7cc602a32fff7613203630312728898be8&selectedTaskRun=HpJpHhmLSL6h4HN6LB7snw.0

Log: https://treeherder.mozilla.org/logviewer?job_id=477153412&repo=autoland&lineNumber=18047

Backout: https://hg.mozilla.org/integration/autoland/rev/6b61eed2f732def30e1213feeb8ecdcb15687254

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e3b744666cdc
Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions.  Removing the deal_id extra key from the app_opened event because it's no longer needed.  Updating some naming of classes. r=android-reviewers,gmalekpour,twhite

Comment 11

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/e3b744666cdc

Comment 12

[John Oberhauser]

Attachment: Bug 1917939: Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions. Removing the deal_id extra key from the app_opened event because it's no longer needed. Updating some naming of classes.

Original Revision: https://phabricator.services.mozilla.com/D222046

Comment 13

[Phabricator Automation]

beta Uplift Approval Request

• User impact if declined: No user impact. Only business related impact.
• Code covered by automated testing: yes
• Fix verified in Nightly: yes
• Needs manual QE test: no
• Steps to reproduce for manual QE testing: QE testing happened in nightly
• Risk associated with taking this patch: minimal
• Explanation of risk level: Patches touch telemetry and add vendor specific google search codes. Unit tests and QA have shown no regressions.
• String changes made/needed: none
• Is Android affected?: yes

Comment 14

[John Oberhauser]

Attachment: Bug 1917939: Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions. Removing the deal_id extra key from the app_opened event because it's no longer needed. Updating some naming of classes.

Original Revision: https://phabricator.services.mozilla.com/D222046

Comment 15

[John Oberhauser]

Attachment: Bug 1917939: Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions. Removing the deal_id extra key from the app_opened event because it's no longer needed. Updating some naming of classes.

Original Revision: https://phabricator.services.mozilla.com/D222046

Comment 16

[John Oberhauser]

Attachment: Bug 1917939: Using metrics.distribution_id and first_session.distribution_id metrics to help track vivo distributions. Removing the deal_id extra key from the app_opened event because it's no longer needed. Updating some naming of classes.

Original Revision: https://phabricator.services.mozilla.com/D222046

Comment 17

[Phabricator Automation]

beta Uplift Approval Request

• User impact if declined: No user impact. Only business related impact.
• Code covered by automated testing: yes
• Fix verified in Nightly: yes
• Needs manual QE test: no
• Steps to reproduce for manual QE testing: QE testing happened in nightly
• Risk associated with taking this patch: minimal
• Explanation of risk level: Patches touch telemetry and add vendor specific google search codes. Unit tests and QA have shown no regressions.
• String changes made/needed: none
• Is Android affected?: yes

Comment 18

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/a3518a4bf4b0