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

[beta-1.70] Backport not_found_permutations test fix. #11983

Merged
merged 1 commit into from
Apr 17, 2023

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