Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: wcth.ooni.io: dns_nxdomain_error #1409

Closed
bassosimone opened this issue Mar 15, 2021 · 4 comments
Closed

ci: wcth.ooni.io: dns_nxdomain_error #1409

bassosimone opened this issue Mar 15, 2021 · 4 comments
Assignees
Labels

Comments

@bassosimone
Copy link
Contributor

-- FAIL: TestSmoke (1.60s)
panic: client.Do failed: Post "https://wcth.ooni.io/": dns_nxdomain_error [recovered]
	panic: client.Do failed: Post "https://wcth.ooni.io/": dns_nxdomain_error

goroutine 19 [running]:
testing.tRunner.func1.2(0xc40ac0, 0xc000064020)
	/opt/hostedtoolcache/go/1.16.2/x64/src/testing/testing.go:1144 +0x49f
testing.tRunner.func1(0xc00008d080)
	/opt/hostedtoolcache/go/1.16.2/x64/src/testing/testing.go:1147 +0x695
panic(0xc40ac0, 0xc000064020)
	/opt/hostedtoolcache/go/1.16.2/x64/src/runtime/panic.go:971 +0x499
github.com/ooni/probe-cli/v3/internal/engine/runtimex.PanicOnError(...)
	/home/runner/work/probe-cli/probe-cli/internal/engine/runtimex/runtimex.go:10
github.com/ooni/probe-cli/v3/internal/cmd/oohelper.main()
	/home/runner/work/probe-cli/probe-cli/internal/cmd/oohelper/oohelper.go:44 +0x868
github.com/ooni/probe-cli/v3/internal/cmd/oohelper.TestSmoke(0xc00008d080)
	/home/runner/work/probe-cli/probe-cli/internal/cmd/oohelper/oohelper_test.go:7 +0x7b
testing.tRunner(0xc00008d080, 0xd3c790)
	/opt/hostedtoolcache/go/1.16.2/x64/src/testing/testing.go:1194 +0x203
created by testing.(*T).Run
	/opt/hostedtoolcache/go/1.16.2/x64/src/testing/testing.go:1239 +0x5d8

See https://github.com/ooni/probe-cli/pull/256/checks?check_run_id=2112332477#step:5:94.

Cannot reproduce locally. I wonder if this issue is related to some rules set up by jafar tests?

@bassosimone
Copy link
Contributor Author

So, this is not related by anything done by Jafar. In fact, the Jafar codebases does not reference wcth.ooni.io.

@bassosimone
Copy link
Contributor Author

This is the only recurrent error blocking #1369 🤔

bassosimone added a commit to ooni/probe-cli that referenced this issue Apr 1, 2021
@bassosimone
Copy link
Contributor Author

Here's a verbose trace:

2021/04/01 16:59:58 debug resolve www.example.com...
2021/04/01 17:00:00 debug resolve www.example.com... ([93.184.216.34], <nil>) in 1.595358882s
2021/04/01 17:00:00 debug out: {"http_request":"http://www.example.com","http_request_headers":{"Accept":["text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8"],"Accept-Language":["en-US;q=0.8,en;q=0.5"],"User-Agent":["Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.90 Safari/537.36"]},"tcp_connect":["93.184.216.34:80"]}
2021/04/01 17:00:00 debug > POST https://wcth.ooni.io/
2021/04/01 17:00:00 debug > User-Agent: oohelper/3.9.0 ooniprobe-engine/3.9.0
2021/04/01 17:00:00 debug > Content-Type: application/json
2021/04/01 17:00:00 debug > Host: wcth.ooni.io     
2021/04/01 17:00:00 debug >                        
2021/04/01 17:00:00 debug resolve wcth.ooni.io...  
2021/04/01 17:00:00 debug resolve wcth.ooni.io... ([], dns_nxdomain_error) in 1.026807ms
2021/04/01 17:00:00 debug < dns_nxdomain_error     
--- FAIL: TestSmoke (1.60s)
panic: client.Do failed: Post "https://wcth.ooni.io/": dns_nxdomain_error [recovered]
	panic: client.Do failed: Post "https://wcth.ooni.io/": dns_nxdomain_error

goroutine 19 [running]:
testing.tRunner.func1.2(0xc4f320, 0xc0002f7360)
	/opt/hostedtoolcache/go/1.16.2/x64/src/testing/testing.go:1144 +0x49f
testing.tRunner.func1(0xc000087080)
	/opt/hostedtoolcache/go/1.16.2/x64/src/testing/testing.go:1147 +0x695
panic(0xc4f320, 0xc0002f7360)
	/opt/hostedtoolcache/go/1.16.2/x64/src/runtime/panic.go:971 +0x499
github.com/ooni/probe-cli/v3/internal/engine/runtimex.PanicOnError(...)
	/home/runner/work/probe-cli/probe-cli/internal/engine/runtimex/runtimex.go:10
github.com/ooni/probe-cli/v3/internal/cmd/oohelper.main()
	/home/runner/work/probe-cli/probe-cli/internal/cmd/oohelper/oohelper.go:44 +0x868
github.com/ooni/probe-cli/v3/internal/cmd/oohelper.TestSmoke(0xc000087080)
	/home/runner/work/probe-cli/probe-cli/internal/cmd/oohelper/oohelper_test.go:8 +0xaa
testing.tRunner(0xc000087080, 0xd4ca90)
	/opt/hostedtoolcache/go/1.16.2/x64/src/testing/testing.go:1194 +0x203
created by testing.(*T).Run
	/opt/hostedtoolcache/go/1.16.2/x64/src/testing/testing.go:1239 +0x5d8
FAIL	github.com/ooni/probe-cli/v3/internal/cmd/oohelper	1.706s

From https://github.com/ooni/probe-cli/pull/282/checks?check_run_id=2247983946

bassosimone added a commit to ooni/probe-cli that referenced this issue Apr 1, 2021
* chore(oohelper): increase tests verbosity

Hopefully this helps with ooni/probe#1409.

* fix(oohelper): use a nonstandard resolver

* fix previous

* make the diff pleasant/committable/correct
@bassosimone
Copy link
Contributor Author

I've committed a workaround for this weird issue!

ainghazal pushed a commit to ainghazal/probe-cli that referenced this issue Mar 8, 2022
* chore(oohelper): increase tests verbosity

Hopefully this helps with ooni/probe#1409.

* fix(oohelper): use a nonstandard resolver

* fix previous

* make the diff pleasant/committable/correct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants