1460537 - [meta] Unified connectivity checker

Status: RESOLVED FIXED

(Core :: Networking, enhancement, P5)

Description

[Valentin Gosu [:valentin] (he/him)]

We want a service similar to the captive portal check that will provide information about the network path to a fixed endpoint.
This will let us know if we can use features such as TLS 1.3, IPv6, etc - that might not be supported by the network or they might be blocked by middleboxes.

Comment 1

[Valentin Gosu [:valentin] (he/him)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b1499aa617f71d4711bfa5f971cbe5a72bfec656

Comment 2

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1460537 - Connectivity Service - Add DNSv4 and DNSv6 checks r=dragana!

Comment 3

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1460537 - Connectivity Service - Add check for IPv4/v6 connectivity r?dragana!

This is likely not the best approach to detecting IP connectivity.
The check has a slight delay until the failure counter reaches the threshold.
A more reliable way will be added in a follow-up.

Depends on D7844

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bd3f7151c697
Connectivity Service - Add DNSv4 and DNSv6 checks r=dragana
https://hg.mozilla.org/integration/autoland/rev/ae2b0a2cd8a7
Connectivity Service - Add check for IPv4/v6 connectivity r=dragana

Comment 5

[Cristina Coroiu [:ccoroiu]]

Backed out 2 changesets (bug 1460537) for xpcshell failures at netwerk/test/unit/test_network_connectivity_service.js

Backout: https://hg.mozilla.org/integration/autoland/rev/f5a40dea590259a0ac9bb2c79190f8245b16b6eb

Failure push: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=ae2b0a2cd8a752d2e0783baaf7be8a1d69ea385c&selectedJob=207809678

Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=207809678&repo=autoland&lineNumber=10482

17:14:34     INFO -  TEST-START | netwerk/test/unit/test_network_connectivity_service.js
17:14:34  WARNING -  TEST-UNEXPECTED-FAIL | netwerk/test/unit/test_network_connectivity_service.js | xpcshell return code: 0
17:14:34     INFO -  TEST-INFO took 331ms
17:14:34     INFO -  >>>>>>>
17:14:34     INFO -  (xpcshell/head.js) | test MAIN run_test pending (1)
17:14:34     INFO -  (xpcshell/head.js) | test run_next_test 0 pending (2)
17:14:34     INFO -  (xpcshell/head.js) | test MAIN run_test finished (2)
17:14:34     INFO -  running event loop
17:14:34     INFO -  netwerk/test/unit/test_network_connectivity_service.js | Starting testDNS
17:14:34     INFO -  (xpcshell/head.js) | test testDNS pending (2)
17:14:34     INFO -  (xpcshell/head.js) | test run_next_test 0 finished (2)
17:14:34     INFO -  TEST-PASS | netwerk/test/unit/test_network_connectivity_service.js | testDNS - [testDNS : 28] Check DNSv4 support (expect OK) - 1 == 1
17:14:34  WARNING -  TEST-UNEXPECTED-FAIL | netwerk/test/unit/test_network_connectivity_service.js | testDNS - [testDNS : 29] Check DNSv6 support (expect OK) - 2 == 1
17:14:34     INFO -  Z:/task_1540484781/build/tests/xpcshell/tests/netwerk/test/unit/test_network_connectivity_service.js:testDNS:29
17:14:34     INFO -  exiting test
17:14:34     INFO -  Unexpected exception NS_ERROR_ABORT:
17:14:34     INFO -  _abort_failed_test@Z:\task_1540484781\build\tests\xpcshell\head.js:746:9
17:14:34     INFO -  do_report_result@Z:\task_1540484781\build\tests\xpcshell\head.js:853:5
17:14:34     INFO -  Assert<@Z:\task_1540484781\build\tests\xpcshell\head.js:55:5
17:14:34     INFO -  proto.report@resource://testing-common/Assert.jsm:214:5
17:14:34     INFO -  equal@resource://testing-common/Assert.jsm:246:3
17:14:34     INFO -  testDNS@Z:/task_1540484781/build/tests/xpcshell/tests/netwerk/test/unit/test_network_connectivity_service.js:29:3
17:14:34     INFO -  async*run_next_test/_run_next_test/<@Z:\task_1540484781\build\tests\xpcshell\head.js:1441:22
17:14:34     INFO -  async*_run_next_test@Z:\task_1540484781\build\tests\xpcshell\head.js:1441:10
17:14:34     INFO -  run@Z:\task_1540484781\build\tests\xpcshell\head.js:692:9
17:14:34     INFO -  _do_main@Z:\task_1540484781\build\tests\xpcshell\head.js:219:3
17:14:34     INFO -  _execute_test@Z:\task_1540484781\build\tests\xpcshell\head.js:533:5
17:14:34     INFO -  @-e:1:1
17:14:34     INFO -  exiting test
17:14:34     INFO -  <<<<<<<

Comment 6

[Valentin Gosu [:valentin] (he/him)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=84bbf5d7e041ae83c18ea93680769f6e1d26eccf

Comment 7

[Valentin Gosu [:valentin] (he/him)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4937b16ee6daf02cca11863c49ccb0af0bea4f15

Comment 8

[Valentin Gosu [:valentin] (he/him)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=2812c1a5fa2cff19b768de9e4801a00e405aaa98

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0f86b2056d5a
Connectivity Service - Add DNSv4 and DNSv6 checks r=dragana
https://hg.mozilla.org/integration/autoland/rev/f2143be625a7
Connectivity Service - Add check for IPv4/v6 connectivity r=dragana

Comment 10

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/0f86b2056d5a
https://hg.mozilla.org/mozilla-central/rev/f2143be625a7