Open Bug 1685234 Opened 4 years ago Updated 3 years ago

Corroborate is set to run but its telemetry is expired

Categories

(Firefox :: General, defect, P2)

defect

Tracking

()

People

(Reporter: agashlin, Assigned: rhelmer)

References

Details

The corroborate.omnijar_corrupted and corroborate.omnijar_mismatch scalars expired in 80, but it seems that it will still run on late idle (bug 1627040) because the pref is still enabled (since bug 1608308). I don't think it has any function independent of the telemetry. Probably the pref should be disabled?

Hi Robert, is there an issue filed to renew those expired scalars that we may mark this as a duplicate of?
(or in alternative we may set priority on it?).

Flags: needinfo?(rhelmer)

The Bugbug bot thinks this bug should belong to the 'Toolkit::Telemetry' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Component: General → Telemetry
Product: Firefox → Toolkit
Component: Telemetry → General
Product: Toolkit → Firefox

(In reply to Luca Greco [:rpl] [:luca] [:lgreco] from comment #1)

Hi Robert, is there an issue filed to renew those expired scalars that we may mark this as a duplicate of?
(or in alternative we may set priority on it?).

I'm not sure we need to keep this around, I believe there's been some newer work that supersedes this too. I'll investigate and get back to this, it's likely we can just remove the corroborate component and the associated probes.

Severity: -- → S4

Note that https://searchfox.org/mozilla-central/search?q=omnijar_corrup&path= has no hits so omnijar-corrupted is already gone, but the omnijar_mismatch one is still around, along with some code to do these checks...

Priority: -- → P3

Hey Doug, since you worked on Bug 1627040. What do you think about this one?

Flags: needinfo?(dothayer)

I don't have any authority to speak on the continuing value of this, but it is a nontrivial amount of work to be doing for no reason, even if it's now out of the startup path, so I would hope that we're still getting some value out of it.

Flags: needinfo?(dothayer)

(In reply to Doug Thayer [:dthayer] (he/him) from comment #6)

I don't have any authority to speak on the continuing value of this, but it is a nontrivial amount of work to be doing for no reason, even if it's now out of the startup path, so I would hope that we're still getting some value out of it.

Thanks. Upgrading the severity and priority based on this assessment.

:heycam, as the author of bug 1608308, any thoughts on this?

Severity: S4 → S3
Flags: needinfo?(cam)
Priority: P3 → P2

Cameron's no longer at Mozilla. I'll renew rhelmer's needinfo request instead.

Flags: needinfo?(rhelmer)
Flags: needinfo?(cam)

(In reply to Robert Helmer [:rhelmer] from comment #3)

I'm not sure we need to keep this around, I believe there's been some newer work that supersedes this too. I'll investigate and get back to this, it's likely we can just remove the corroborate component and the associated probes.

Hey, any update on this?

Flags: needinfo?(rhelmer)

(In reply to Dão Gottwald [::dao] from comment #9)

(In reply to Robert Helmer [:rhelmer] from comment #3)

I'm not sure we need to keep this around, I believe there's been some newer work that supersedes this too. I'll investigate and get back to this, it's likely we can just remove the corroborate component and the associated probes.

Hey, any update on this?

I can go ahead and remove this, thanks for the ping.

Assignee: nobody → rhelmer
Flags: needinfo?(rhelmer)
You need to log in before you can comment on or make changes to this bug.