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

Add a hack to workaround MSVC CI issues #129509

Closed
wants to merge 1 commit into from

Conversation

ChrisDenton
Copy link
Member

I'm still attempting to figure out the issue but the failure rate atm seems really high so I'd like to propose an interim hack.

This adds a single delay of 5 minutes, which is absurdly long in CPU time but practically nothing compared to overall CI time.

@rustbot
Copy link
Collaborator

rustbot commented Aug 24, 2024

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Aug 24, 2024
@Mark-Simulacrum
Copy link
Member

Hm, I seem to recall us having every-30-second printing that was needed for avoidance of some kind of cancellation mechanism. But maybe not actually needed anymore.

r=me

@ChrisDenton
Copy link
Member Author

@bors r=Mark-Simulacrum rollup

@bors
Copy link
Contributor

bors commented Aug 24, 2024

📌 Commit b267457 has been approved by Mark-Simulacrum

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 Aug 24, 2024
@ChrisDenton
Copy link
Member Author

@bors p=1 so this will be picked up if the CI keeps failing

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 24, 2024
…-Simulacrum

Add a hack to workaround MSVC CI issues

I'm still attempting to figure out the issue but the failure rate atm seems really high so I'd like to propose an interim hack.

This adds a single delay of 5 minutes, which is absurdly long in CPU time but practically nothing compared to overall CI time.
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 24, 2024
…iaskrgr

Rollup of 10 pull requests

Successful merges:

 - rust-lang#128596 (stabilize const_fn_floating_point_arithmetic)
 - rust-lang#129199 (make writes_through_immutable_pointer a hard error)
 - rust-lang#129246 (Retroactively feature gate `ConstArgKind::Path`)
 - rust-lang#129290 (Pin `cc` to 1.0.105)
 - rust-lang#129323 (Implement `ptr::fn_addr_eq`)
 - rust-lang#129500 (remove invalid `TyCompat` relation for effects)
 - rust-lang#129501 (panicking: improve hint for Miri's RUST_BACKTRACE behavior)
 - rust-lang#129505 (interpret: ImmTy: tighten sanity checks in offset logic)
 - rust-lang#129509 (Add a hack to workaround MSVC CI issues)
 - rust-lang#129510 (Fix `elided_named_lifetimes` in code)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors
Copy link
Contributor

bors commented Aug 24, 2024

⌛ Testing commit b267457 with merge 29b1240...

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 24, 2024
…imulacrum

Add a hack to workaround MSVC CI issues

I'm still attempting to figure out the issue but the failure rate atm seems really high so I'd like to propose an interim hack.

This adds a single delay of 5 minutes, which is absurdly long in CPU time but practically nothing compared to overall CI time.
@ChrisDenton
Copy link
Member Author

@bors r-

That it passed in my tests seems to just have been by chance unfortunately. I'll need to find another approach,

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 24, 2024
@ChrisDenton ChrisDenton deleted the hack-hack-hack branch August 25, 2024 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants