1686421 - Only enable network.dns.upgrade_with_https_rr on nightly and collect more probes

Status: RESOLVED FIXED

(Core :: Networking, task, P2)

Description

[Kershaw Chang [:kershaw]]

We'd like to be more conservative about this feature, so only enable it on nightly.
For probes we want to collect:

• how many http requests have HTTPS RR
• Among HTTPS RRs, how many RR support http3

Comment 1

[Kershaw Chang [:kershaw]]

Attachment: Bug 1686421 - Enable network.dns.upgrade_with_https_rr only on nightly, r=dragana

Comment 2

[Kershaw Chang [:kershaw]]

Attachment: Bug 1686421 - Don't modify channel's conn info when doing https upgrade

Comment 3

[Kershaw Chang [:kershaw]]

Attachment: Bug 1686421 - Add probes to understand how HTTPS RR is used

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/00cb90f883f3
Enable network.dns.upgrade_with_https_rr only on nightly, r=dragana

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/00cb90f883f3

Comment 6

[Kershaw Chang [:kershaw]]

(In reply to Andreea Pavel [:apavel] from comment #5)

https://hg.mozilla.org/mozilla-central/rev/00cb90f883f3

Sorry, there are two more patches need to be landed. I forgot to add leave-open.

Comment 7

[Mathew Hodson]

network.dns.upgrade_with_https_rr was disabled in bug 1686828

Comment 8

[Kershaw Chang [:kershaw]]

Attachment: bug1686421_telemetry_data_review.txt

Comment 9

[Chris H-C :chutten]

Comment on attachment 9199746 [details]
bug1686421_telemetry_data_review.txt

Load-balance to Megan

Comment 10

[Kershaw Chang [:kershaw]]

Attachment: Bug 1686421 - Enable network.dns.upgrade_with_https_rr on nightly

Comment 11

[Kershaw Chang [:kershaw]]

Attachment: Bug 1686421 - Add probes to understand the usage of https rr

Comment 12

[Kershaw Chang [:kershaw]]

Attachment: bug1686421_scalar_data_review.txt

Comment 13

[Kershaw Chang [:kershaw]]

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

Comment on attachment 9199746 [details]
bug1686421_telemetry_data_review.txt

Load-balance to Megan

ni for data-review request.
Thanks.

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7a032ef812aa
Enable network.dns.upgrade_with_https_rr on nightly r=necko-reviewers,dragana

Comment 15

[Megan McCorquodale]

Comment on attachment 9199746 [details]
bug1686421_telemetry_data_review.txt

1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?
   This will be documented in the probe dictionary.

2. Is there a control mechanism that allows the user to turn the data collection on and off?
   Users can opt out of telemetry.

3. If the request is for permanent data collection, is there someone who will monitor the data over time?
   :kershaw will monitor.

4. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
   Category 1, Technical data.

5. Is the data collection request for default-on or default-off?
   Default on.

6. Does the instrumentation include the addition of any new identifiers?
   No new identifiers.

7. Is the data collection covered by the existing Firefox privacy notice?
   Yes.

8. Does there need to be a check-in in the future to determine whether to renew the data?
   No.

9. Does the data collection use a third-party collection tool?
   No.

---

data-review +

Comment 16

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/7a032ef812aa

Comment 17

[Chris H-C :chutten]

Comment on attachment 9201762 [details]
bug1686421_scalar_data_review.txt

DATA COLLECTION REVIEW RESPONSE:

Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?

Yes.

Is there a control mechanism that allows the user to turn the data collection on and off?

Yes. This collection is Telemetry so can be controlled through Firefox's Preferences.

If the request is for permanent data collection, is there someone who will monitor the data over time?

Yes, :kershaw is responsible.

Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 1, Technical.

Is the data collection request for default-on or default-off?

Default on for all channels (eventually)

Does the instrumentation include the addition of any new identifiers?

No.

Is the data collection covered by the existing Firefox privacy notice?

Yes.

Does there need to be a check-in in the future to determine whether to renew the data?

No. This collection is permanent.

---

Result: datareview+

Comment 18

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a7d86de736e5
Add probes to understand how HTTPS RR is used r=necko-reviewers,dragana

Comment 19

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/01cddc4e35f3
Add probes to understand the usage of https rr r=necko-reviewers,dragana

Comment 20

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/a7d86de736e5
https://hg.mozilla.org/mozilla-central/rev/01cddc4e35f3