Closed Bug 1954708 Opened 1 month ago Closed 1 month ago

dns.lookup_method telemetry records multiple lookup-methods for the same lookup

Categories

(Core :: Networking: DNS, defect, P2)

defect

Tracking

()

RESOLVED FIXED
138 Branch
Tracking Status
firefox138 --- fixed

People

(Reporter: omansfeld, Assigned: omansfeld)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-priority-next][necko-triaged])

Attachments

(1 file)

We are possibly counting both METHOD_HIT and METHOD_NEGATIVE_HIT here for the same lookup. We're also possibly counting both METHOD_HIT and METHOD_RENEWAL inside of ConditionallyRefreshRecord(...).

I'll be adding a nsHostRecord::IsRenewal boolean that's set to true where we currently record METHOD_RENEWAL for Bug 1952221 - that should be useful for seperating those recordings inside of FromCache(...).

Severity: N/A → S3
Pushed by omansfeld@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6b7d59d34cf5 lookup_method telemetry now only records one lookup-method per lookup. r=necko-reviewers,valentin
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
Blocks: 1937022
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: