Skip to content

Remove the connect_timeout_unroutable test. - #57584

Merged
bors merged 1 commit into
rust-lang:masterfrom
nnethercote:rm-connect_timeout_unroutable
Jan 14, 2019
Merged

Remove the connect_timeout_unroutable test.#57584
bors merged 1 commit into
rust-lang:masterfrom
nnethercote:rm-connect_timeout_unroutable

Conversation

@nnethercote

Copy link
Copy Markdown
Contributor

It requires an unreachable IP address, but there is no such thing, and
this has caused it to fail for multiple people.

Fixes#44698, fixes#50065.

r? @sfackler

It requires an unreachable IP address, but there is no such thing, and
this has caused it to fail for multiple people.
Fixesrust-lang#44698, fixesrust-lang#50065.
@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 13, 2019
@sfackler

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Jan 13, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 24a9ac7 has been approved by sfackler

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 13, 2019
@CentrilCentril mentioned this pull request Jan 14, 2019
Centril added a commit to Centril/rust that referenced this pull request Jan 14, 2019
…utable, r=sfackler
Remove the `connect_timeout_unroutable` test.
It requires an unreachable IP address, but there is no such thing, and
this has caused it to fail for multiple people.
Fixesrust-lang#44698, fixesrust-lang#50065.
r? @sfackler
Centril added a commit to Centril/rust that referenced this pull request Jan 14, 2019
…utable, r=sfackler
Remove the `connect_timeout_unroutable` test.
It requires an unreachable IP address, but there is no such thing, and
this has caused it to fail for multiple people.
Fixesrust-lang#44698, fixesrust-lang#50065.
r? @sfackler
@CentrilCentril mentioned this pull request Jan 14, 2019
bors added a commit that referenced this pull request Jan 14, 2019
Rollup of 6 pull requests
Successful merges:
- #57232 (Parallelize and optimize parts of HIR map creation)
- #57418 (MetadataOnlyCodegenBackend: run the collector only once)
- #57465 (Stabilize cfg_target_vendor)
- #57477 (clarify resolve typo suggestion)
- #57556 (privacy: Fix private-in-public check for existential types)
- #57584 (Remove the `connect_timeout_unroutable` test.)
Failed merges:
r? @ghost
@bors
bors merged commit 24a9ac7 into rust-lang:masterJan 14, 2019
@nnethercote
nnethercote deleted the rm-connect_timeout_unroutable branch January 14, 2019 23:01
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

net::tcp::tests::connect_timeout_unroutable can fail with ConnectionRefused test failure: net::tcp::tests::connect_timeout_unroutable

4 participants

@nnethercote@sfackler@bors@rust-highfive