Closed Bug 1829850 Opened 2 years ago Closed 2 years ago

Permanent failure line for successful Talos tasks: FATAL ERROR: Non-local network connections are disabled and a connection attempt to incoming.telemetry.mozilla.org (34.120.208.123) was made.

Categories

(Core :: DOM: Security, defect, P5)

defect

Tracking

()

RESOLVED FIXED
114 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox112 --- unaffected
firefox113 --- wontfix
firefox114 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: saschanaz)

References

(Regression)

Details

(Keywords: intermittent-failure, regression, Whiteboard: [domsecurity-intermittent])

Attachments

(1 file)

Filed by: archaeopteryx [at] coole-files.de
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=413701779&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/CQL2rc4BRp-hq_47Yibm-Q/runs/0/artifacts/public/logs/live_backing.log


[task 2023-04-25T12:04:33.168Z] 12:04:33     INFO -  PID 8520 | *** You are running in headless mode.
[task 2023-04-25T12:04:33.246Z] 12:04:33     INFO -  PID 8520 | console.error: "C:\\Users\\task_1682422729\\AppData\\Local\\Temp\\tmp3s2qqb_e\\profile\\storage" "to-be-removed" 0 "" "Quota"
[task 2023-04-25T12:04:33.249Z] 12:04:33     INFO -  PID 8520 | console.error: "started removing C:\\Users\\task_1682422729\\AppData\\Local\\Temp\\tmp3s2qqb_e\\profile\\storage\\to-be-removed"
[task 2023-04-25T12:04:33.250Z] 12:04:33     INFO -  PID 8520 | console.error: "done removing C:\\Users\\task_1682422729\\AppData\\Local\\Temp\\tmp3s2qqb_e\\profile\\storage\\to-be-removed. removed 1 entries."
[task 2023-04-25T12:04:33.250Z] 12:04:33     INFO -  PID 8520 | console.error: "Pinged glean, waiting for submission."
[task 2023-04-25T12:04:33.305Z] 12:04:33     INFO -  PID 8520 | FATAL ERROR: Non-local network connections are disabled and a connection attempt to incoming.telemetry.mozilla.org (34.120.208.123) was made.

I can't reproduce due to talos tests not running locally :(

can't open file 'gecko-https-first/testing/mozharness/external_tools/tooltool.py': [Errno 2] No such file or directory

Jan-Erik helped me look at this. Might be this is an issue from bug 1788986, where the talos specific pref override to not use the production server is not properly mirrored into the QuotaManager background task.

Flags: needinfo?(fbraun)
Regressed by: 1788986
No longer regressed by: 1829639
Regressed by: 1827390
No longer regressed by: 1788986
Flags: needinfo?(krosylight)

It does follow the user preference to disable telemetry, but I guess Talos still needs the telemetry? In that case do we have some documentation about what to do here?

Flags: needinfo?(krosylight) → needinfo?(fbraun)

Actually Jan-Erik may have better idea for the docs?

Flags: needinfo?(fbraun) → needinfo?(jrediger)

The Talos config sets the legacy telemetry endpoint to localhost: https://searchfox.org/mozilla-central/source/testing/profiles/perf/user.js#82
And also sets Glean to act as if enabled, but not sending out things: https://searchfox.org/mozilla-central/source/testing/profiles/perf/user.js#83

Is the latter pref also set in the background task? (I don't know exactly how background tasks get their preference, do they just load the full profile or does it need explicit mirroring of a subset of prefs?)

Flags: needinfo?(jrediger)

It does not receive any specific prefs for now, it only receives the parameter whether to send telemetry or not. I'm not sure it's any worth to pass the Talos config in this case, perhaps it can just disable the telemetry? Does it make sense?

Flags: needinfo?(jrediger)

I don't know Talos enough, but I can imagine it wants to keep Telemetry on so that it tests the performance of the app as close as possible to real usage.
Whether that means background tasks should also keep it on, but not sending, is then up to you and Talos folks.

Flags: needinfo?(jrediger)

Makes sense. Passing the question to the current Talos triage owner.

Flags: needinfo?(aglavic)

investigating the issue will respond when I have some more information

Flags: needinfo?(aglavic)
Flags: needinfo?(aglavic)

(It's about having telemetry from asynchronous cleanup that happens after Firefox closes. I'm not sure this is in the scope of Talos)

Flags: needinfo?(aglavic)

I am 99% sure that we do not want to collect metrics from talos (or other CI test/perf jobs). There might be an exception or two.

Assignee: nobody → krosylight
Status: NEW → ASSIGNED
Pushed by krosylight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a53000efdd8c Disable telemetry from removeDirectory in performance tests r=nalexander,jmaher
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: