Pings being throttled though throttling set to 1

RESOLVED INVALID

Status

()

RESOLVED INVALID
2 years ago
2 years ago

People

(Reporter: Silne30, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 2 obsolete attachments)

(Reporter)

Description

2 years ago
I am running a marionette test with a custom profile with throttling set to 1. An addon is installed but the ping is still not sent due to throttling.
(Reporter)

Updated

2 years ago
Flags: needinfo?(chutten)
Flags: needinfo?(alessio.placitelli)

Updated

2 years ago
Blocks: 1301776
Flags: needinfo?(chutten)

Updated

2 years ago
Flags: needinfo?(chutten)
(In reply to John Dorlus [:Silne30] from comment #0)
> I am running a marionette test with a custom profile with throttling set to
> 1. An addon is installed but the ping is still not sent due to throttling.

Please post the Telemetry logs, otherwise there's no way for us to figure out what's going on.
Would you mind also attaching the code you're using to set the pref/your current patch?
Flags: needinfo?(alessio.placitelli) → needinfo?(jdorlus)
(Reporter)

Comment 2

2 years ago
Created attachment 8852067 [details]
Test logs
Flags: needinfo?(jdorlus)
Flags: needinfo?(chutten)
(Reporter)

Comment 3

2 years ago
Created attachment 8852069 [details]
code for prefs
Flags: needinfo?(alessio.placitelli)
(In reply to John Dorlus [:Silne30] from comment #3)
> Created attachment 8852069 [details]
> code for prefs

This attachment is not setting the "toolkit.telemetry.minSubsessionLength" as suggested in bug 1301776 comment 21. Is this the wrong attachment or you didn't set it at all?
Flags: needinfo?(alessio.placitelli) → needinfo?(jdorlus)
(Reporter)

Comment 5

2 years ago
Created attachment 8852215 [details]
Correct log with pref set
Flags: needinfo?(jdorlus) → needinfo?(alessio.placitelli)
(Reporter)

Comment 6

2 years ago
Created attachment 8852216 [details]
Code to set prefs.

I also checked the value of the pref in the test and it does show 0. But still getting throttled.
Attachment #8852067 - Attachment is obsolete: true
Attachment #8852069 - Attachment is obsolete: true
(Reporter)

Comment 7

2 years ago
Something interesting to note: This throttling does not affect the sending of the testpilot ping when you install a testpilot addon. Not sure if that helps you diagnose.
(Reporter)

Comment 8

2 years ago
I also loaded a profile with the pref set in the actual profile. Still no go.
(In reply to John Dorlus [:Silne30] from comment #7)
> Something interesting to note: This throttling does not affect the sending
> of the testpilot ping when you install a testpilot addon. Not sure if that
> helps you diagnose.

That's expected. The "testpilot" ping is not a main ping and has nothing to do with the environment changes.
(In reply to John Dorlus [:Silne30] from comment #6)
> Created attachment 8852216 [details]
> Code to set prefs.
> 
> I also checked the value of the pref in the test and it does show 0. But
> still getting throttled.

I manually checked and setting the pref to 0 + restarting Firefox allows to overcome throttling.

The "toolkit.telemetry.minSubsessionLength" preference must be set in the profile before Firefox starts (or Firefox must be restarted after to enforce it): this is exactly the same as the "initDelay" pref.

So, if the logs still report throttling, it's probably because the pref didn't contain 0 when Firefox started up. As far as I can tell, that's the only reason why the throttling message can be printed [1].

If you're building your own version of Firefox, you could just add

> dump("\n**** DEBUG Minimum subsession length is " + MIN_SUBSESSION_LENGTH_MS + "\n");

Inside the "if" branch at [1] to see if the value is what you are setting.

[1] - http://searchfox.org/mozilla-central/rev/7419b368156a6efa24777b21b0e5706be89a9c2f/toolkit/components/telemetry/TelemetrySession.jsm#2092
Flags: needinfo?(alessio.placitelli) → needinfo?(jdorlus)
(Reporter)

Comment 11

2 years ago
Updating mozilla-central fixed this.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Flags: needinfo?(jdorlus)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.