1872767 - Startup terminal-spew "ERROR firefox_on_glean::private::string] Unable to set string metric in non-main process. Ignoring."

Status: RESOLVED FIXED

(Core :: Graphics, defect, P2)

Description

[Daniel Holbert [:dholbert]]

STR:

1. Start Firefox Nightly from the terminal.
2. Watch your terminal output.

ACTUAL RESULTS:
This appears within the first few seconds of running:

[ERROR firefox_on_glean::private::string] Unable to set string metric in non-main process. Ignoring.
[ERROR firefox_on_glean::private::string] Unable to set string metric in non-main process. Ignoring.
[ERROR firefox_on_glean::private::string] Unable to set string metric in non-main process. Ignoring.
[ERROR firefox_on_glean::private::string] Unable to set string metric in non-main process. Ignoring.
[ERROR firefox_on_glean::private::string] Unable to set string metric in non-main process. Ignoring.

EXPECTED RESULTS:
No such terminal output.

mozregression yields this pushlog:
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=5e551f2bacc3d6a8294dcb0273e8299cf7a17e2d&tochange=bc706579ad2b44cdcaefe81a4b7c14b0dd535a09

which points to bug 1870021 as the culprit. (Makes sense, since that added some glean probes.)

jnicol, can you tweak things such that we don't spam this output? Maybe we need a XRE_IsParentProcess() check or something along those lines?

Comment 1

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

Set release status flags based on info from the regressing bug 1870021

Comment 2

[Bob Hood [:bhood]]

Nical, since Jamie is on parental leave this month, is this something you think you could look at?

Comment 3

[Nicolas Silva [:nical]]

Attachment: Bug 1872767 - Avoid touching gpu process glean probes outside of the main process. r=#gfx-reviewers

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3f6550f8444c
Avoid touching gpu process glean probes outside of the main process. r=gfx-reviewers,aosmond

Comment 5

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/3f6550f8444c