1649878 - Perma telemetry-integration toolkit/components/telemetry/tests/integration/tests/test_event_ping.py::test_event_ping

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect)

Description

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

Push: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&tier=3&searchStr=linux%2C18.04%2Cx64%2Copt%2Ctelemetry%2Ctests%2Csource-test-python-telemetry-integration-tests-linux1804-64%2Fopt%2Ctt%28i%29&tochange=ee812b6359befd98993f39f8cffd6735ebcd5644&fromchange=3add3a1747557d9b1e20c41afed20a94952ce5e8&selectedTaskRun=YUvmpCPUTUqnXwDLHhEU6w.0

Log: https://treeherder.mozilla.org/logviewer.html#?job_id=307600226&repo=mozilla-central

This started with the changes from bug 1599567.

Comment 1

[:prathiksha]

Looks like Raphael's on it. Thanks and let me know if there's anything I can help with.

Comment 2

[Raphael Aurich [:raphael] UTC+01:00]

Attachment: Bug 1649878 - Fix tt(i) event ping test to handle optional event fields; r?dexter!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8958392c13ac
Fix tt(i) event ping test to handle optional event fields; r=Dexter

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/8958392c13ac

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:raphael, is this bug important enough to require an uplift?
If not please set status_beta to wontfix.

For more information, please visit auto_nag documentation.

Comment 6

[Raphael Aurich [:raphael] UTC+01:00]

I'm not sure if this needs an uplift.

:chutten, what do you think?

Comment 7

[Chris H-C :chutten]

Comment on attachment 9160858 [details]
Bug 1649878 - Fix tt(i) event ping test to handle optional event fields; r?dexter!

Beta/Release Uplift Approval Request

• User impact if declined: None (intermittent test failure emails for devs, nothing for users)
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Test-only fix.
• String changes made/needed: None

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9160858 [details]
Bug 1649878 - Fix tt(i) event ping test to handle optional event fields; r?dexter!

It's test-only and can land without approval.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

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

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr78/rev/7062e50ded42