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

net/http: TestTransportEventTraceRealDNS fails on OS X 10.11.6 #16732

Closed
aybabtme opened this issue Aug 16, 2016 · 2 comments

Comments

Projects
None yet
3 participants
@aybabtme
Copy link

commented Aug 16, 2016

  1. What version of Go are you using (go version)?

go version go1.7 darwin/amd64

  1. What operating system and processor architecture are you using (go env)?
$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/antoine/gocode"
GORACE=""
GOROOT="/Users/antoine/go"
GOTOOLDIR="/Users/antoine/go/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/gy/yc35jkns511gp7t1w9mnn6q00000gn/T/go-build579204479=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
  1. What did you do?
$ cd ~/go
$ git pull origin master
Already up-to-date.
$ git status
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean
$ git checkout go1.7
...
HEAD is now at 0d81858... go1.7
$ cd src
$ ./all.bash
...
--- FAIL: TestTransportEventTraceRealDNS (0.06s)
        transport_test.go:3386: expected error during DNS lookup
FAIL
FAIL    net/http        10.854s
  1. What did you expect to see?

TestTransportEventTraceRealDNS should pass on a clean checkout of go1.7

  1. What did you see instead?

TestTransportEventTraceRealDNS fails.

@bradfitz

This comment has been minimized.

Copy link
Member

commented Aug 16, 2016

This is basically a dup of #16716.

The Go test suite assumes you have working DNS. You appear to be using a DNS server which lies to you.

I suppose we could make that test only run in full mode and on builders.

@gopherbot

This comment has been minimized.

Copy link

commented Sep 9, 2016

CL https://golang.org/cl/28782 mentions this issue.

@gopherbot gopherbot closed this in e4691d9 Sep 12, 2016

@golang golang locked and limited conversation to collaborators Sep 12, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.