Skip to content

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Apr 17, 2023

This is a backport of #11976 to 1.70. I'm concerned that this test could result in trouble in CI.

…page

Fix flaky not_found_permutations test.

This fixes the `registry::not_found_permutations` test which would randomly fail since the order of http requests was not deterministic. The resolver can issue queries in parallel which can process requests out-of-order.

Fixes rust-lang#11975
@rustbot
Copy link
Collaborator

rustbot commented Apr 17, 2023

r? @epage

(rustbot has picked a reviewer for you, use r? to override)

@rustbot
Copy link
Collaborator

rustbot commented Apr 17, 2023

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.70.0. Please double check that you specified the right target!

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 17, 2023
@weihanglo
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Apr 17, 2023

📌 Commit 9009251 has been approved by weihanglo

It is now in the queue for this repository.

@bors bors 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 Apr 17, 2023
@bors
Copy link
Contributor

bors commented Apr 17, 2023

⌛ Testing commit 9009251 with merge 4d28f23...

@bors
Copy link
Contributor

bors commented Apr 17, 2023

☀️ Test successful - checks-actions
Approved by: weihanglo
Pushing 4d28f23 to rust-1.70.0...

1 similar comment
@bors
Copy link
Contributor

bors commented Apr 17, 2023

☀️ Test successful - checks-actions
Approved by: weihanglo
Pushing 4d28f23 to rust-1.70.0...

@bors
Copy link
Contributor

bors commented Apr 17, 2023

👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.

@bors bors merged commit 4d28f23 into rust-lang:rust-1.70.0 Apr 17, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request May 5, 2023
…nglo

[beta] Update cargo

2 commits in 84b7041fd2745ee6b3b4a150314f81aabb78e6b2..ec8a8a0cabb0e0cadef58902470f6c7ee7868bdc
2023-04-13 20:08:40 +0000 to 2023-04-25 19:46:25 +0000
- [beta-1.70] backport add the Win32_System_Console feature (rust-lang/cargo#12038)
- [beta-1.70] Backport not_found_permutations test fix. (rust-lang/cargo#11983)

r? `@ghost`
@ehuss ehuss added this to the 1.70.0 milestone May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: 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.

5 participants