1592253 - Remove the unused Fennec telemetry probes

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect, P5)

Comment 1

[(no longer active)]

Attachment: Bug 1592253 - Remove the unused Fennec telemetry probes;

Comment 2

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

We still depend on Fennec telemetry. Can we delay this or let these expire when we ultimately replace Fennec?

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/31d3e3068c32
Remove the unused Fennec telemetry probes; r=snorp

Comment 4

[(no longer active)]

(In reply to Stefan Arentz [:st3fan] from comment #2)

We still depend on Fennec telemetry. Can we delay this or let these expire when we ultimately replace Fennec?

Oh, sorry I landed this before I saw your comment. Yes, there's no rush at all, it just wasn't obvious that these are used.

I'll back out the patch for now.

Comment 5

[(no longer active)]

Attachment: Backout changeset 31d3e3068c32 (bug 1592253) because we still depend on these probes

Comment 6

[Chris H-C :chutten]

For the record, removing these from Histograms.json doesn't stop the data from being collected on Fennec. Fennec is on esr68, and so long as this patch doesn't get rebased and merged onto that branch, Fennec will go on collecting these probes.

Comment 7

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/206089a73823
Backout changeset 31d3e3068c32 because we still depend on these probes

Comment 8

[(no longer active)]

(In reply to Chris H-C :chutten from comment #6)

For the record, removing these from Histograms.json doesn't stop the data from being collected on Fennec. Fennec is on esr68, and so long as this patch doesn't get rebased and merged onto that branch, Fennec will go on collecting these probes.

So does this mean that I can land this patch?

Comment 9

[Chris H-C :chutten]

Yes. Though I'd like to make sure I understand Stefan properly first.

What are your concerns about removing these probe definitions from Histograms.json? The data will continue to be collected for Fennec builds already out there, and unless this patch is merged to esr68 then this should continue through the lifetime of the product.

Comment 10

[(no longer active)]

unless this patch is merged to esr68

(which I never intended to do, FWIW...)

Comment 11

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

(In reply to Chris H-C :chutten from comment #9)

Yes. Though I'd like to make sure I understand Stefan properly first.

What are your concerns about removing these probe definitions from Histograms.json? The data will continue to be collected for Fennec builds already out there, and unless this patch is merged to esr68 then this should continue through the lifetime of the product.

Yes, that is totally fine. Sorry for the confusion.

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8449ec58f40f
Remove the unused Fennec telemetry probes; r=snorp

Comment 13

[Natalia Csoregi [:nataliaCs]]

Backed out for bustages on TelemetryHistogramEnums.h

backout: https://hg.mozilla.org/integration/autoland/rev/787484b40d10480d0d375604595c8734e550c70e

push: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=8449ec58f40f26b421fe001898257f344b1c3e02&group_state=expanded

failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=274031533&repo=autoland&lineNumber=3878

[task 2019-11-01T07:23:11.334Z] 07:23:11 INFO - New histogram "FETCH_IS_MAINTHREAD" must have a "bug_numbers" field.
[task 2019-11-01T07:23:11.335Z] 07:23:11 INFO - Some errors occurred
[task 2019-11-01T07:23:11.335Z] 07:23:11 INFO - Exception KeyError: KeyError(139830355076864,) in <module 'threading' from '/usr/lib/python2.7/threading.pyc'> ignored
[task 2019-11-01T07:23:11.335Z] 07:23:11 INFO - backend.mk:21: recipe for target '.deps/TelemetryHistogramEnums.h.stub' failed
[task 2019-11-01T07:23:11.335Z] 07:23:11 ERROR - make[4]: *** [.deps/TelemetryHistogramEnums.h.stub] Error 1
[task 2019-11-01T07:23:11.335Z] 07:23:11 INFO - make[4]: Leaving directory '/builds/worker/workspace/build/src/obj-firefox/toolkit/components/telemetry'
[task 2019-11-01T07:23:11.335Z] 07:23:11 INFO - make[4]: *** Waiting for unfinished jobs....
[task 2019-11-01T07:23:11.451Z] 07:23:11 INFO - make[4]: Entering directory '/builds/worker/workspace/build/src/obj-firefox/xpcom/base'
[task 2019-11-01T07:23:11.451Z] 07:23:11 INFO - xpcom/base/gk_rust_utils_ffi_generated.h.stub

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/409ae38a1761
Remove the unused Fennec telemetry probes; r=snorp

Comment 15

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/409ae38a1761