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

[DO NOT MERGE] testing dump-ice-to-disk #128958

Closed
wants to merge 2 commits into from

Conversation

jieyouxu
Copy link
Member

Now how flakey is this

r? @ghost

try-job: i686-mingw

@rustbot rustbot added A-run-make Area: port run-make Makefiles to rmake.rs S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 11, 2024
@jieyouxu
Copy link
Member Author

@bors try

@jieyouxu jieyouxu added S-experimental Status: Ongoing experiment that does not require reviewing and won't be merged in its current state. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 11, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 11, 2024
[DO NOT MERGE] testing `dump-ice-to-disk`

Now how flakey is this

r? `@ghost`

try-job: i686-mingw
@bors
Copy link
Contributor

bors commented Aug 11, 2024

⌛ Trying commit 960e7b5 with merge f9dcb45...

@bors
Copy link
Contributor

bors commented Aug 11, 2024

☀️ Try build successful - checks-actions
Build commit: f9dcb45 (f9dcb45c20d725d30a40852bf4eb711353e3cc2c)

@jieyouxu
Copy link
Member Author

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 11, 2024
[DO NOT MERGE] testing `dump-ice-to-disk`

Now how flakey is this

r? `@ghost`

try-job: i686-mingw
@bors
Copy link
Contributor

bors commented Aug 11, 2024

⌛ Trying commit 210577e with merge c83dfe4...

@bors
Copy link
Contributor

bors commented Aug 11, 2024

☀️ Try build successful - checks-actions
Build commit: c83dfe4 (c83dfe4ecde3ec3c157dceaa816b47b0d349cb64)

@jieyouxu
Copy link
Member Author

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 11, 2024
[DO NOT MERGE] testing `dump-ice-to-disk`

Now how flakey is this

r? `@ghost`

try-job: i686-mingw
@bors
Copy link
Contributor

bors commented Aug 11, 2024

⌛ Trying commit 210577e with merge 9e1ba1a...

@bors
Copy link
Contributor

bors commented Aug 11, 2024

☀️ Try build successful - checks-actions
Build commit: 9e1ba1a (9e1ba1ac99fc10f06d352485a1696849eb925f78)

@jieyouxu
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Aug 11, 2024

⌛ Trying commit 210577e with merge 7c123c0...

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 11, 2024
[DO NOT MERGE] testing `dump-ice-to-disk`

Now how flakey is this

r? `@ghost`

try-job: i686-mingw
@bors
Copy link
Contributor

bors commented Aug 11, 2024

☀️ Try build successful - checks-actions
Build commit: 7c123c0 (7c123c0261a9a4bfdc0975f17815c4cc1a390fd7)

@jieyouxu
Copy link
Member Author

Stop succeeding, start failing!

@jieyouxu
Copy link
Member Author

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 11, 2024
[DO NOT MERGE] testing `dump-ice-to-disk`

Now how flakey is this

r? `@ghost`

try-job: i686-mingw
@bors
Copy link
Contributor

bors commented Aug 11, 2024

⌛ Trying commit 210577e with merge cae0ba6...

@bors
Copy link
Contributor

bors commented Aug 11, 2024

☀️ Try build successful - checks-actions
Build commit: cae0ba6 (cae0ba6a1e5100ee11000c15da246f8a18723a34)

@jieyouxu
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Aug 12, 2024

⌛ Trying commit 210577e with merge 8821303...

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 12, 2024
[DO NOT MERGE] testing `dump-ice-to-disk`

Now how flakey is this

r? `@ghost`

try-job: i686-mingw
@bors
Copy link
Contributor

bors commented Aug 12, 2024

☀️ Try build successful - checks-actions
Build commit: 8821303 (8821303ac12e8b892aef82c6d8e1234470698728)

@jieyouxu
Copy link
Member Author

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 12, 2024
[DO NOT MERGE] testing `dump-ice-to-disk`

Now how flakey is this

r? `@ghost`

try-job: i686-mingw
@bors
Copy link
Contributor

bors commented Aug 12, 2024

⌛ Trying commit 210577e with merge 15cf78a...

@bors
Copy link
Contributor

bors commented Aug 12, 2024

☀️ Try build successful - checks-actions
Build commit: 15cf78a (15cf78ad4a6e2c242f60bc71d0893b658cff0fbf)

@jieyouxu
Copy link
Member Author

@bors try

@jieyouxu
Copy link
Member Author

One last try.

@bors
Copy link
Contributor

bors commented Aug 12, 2024

⌛ Trying commit 210577e with merge 265f66c...

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 12, 2024
[DO NOT MERGE] testing `dump-ice-to-disk`

Now how flakey is this

r? `@ghost`

try-job: i686-mingw
@bors
Copy link
Contributor

bors commented Aug 12, 2024

☀️ Try build successful - checks-actions
Build commit: 265f66c (265f66c8576d03d28ac1c6b2b1d9a27dd106379e)

@jieyouxu
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Aug 12, 2024

⌛ Trying commit 210577e with merge c9eadc1...

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 12, 2024
[DO NOT MERGE] testing `dump-ice-to-disk`

Now how flakey is this

r? `@ghost`

try-job: i686-mingw
@ChrisDenton
Copy link
Member

Maybe your changes fixed it somehow?

@jieyouxu
Copy link
Member Author

Maybe your changes fixed it somehow?

If this try job still passes, I'm going to try remove the ignore-windows for the test and try re-enabling this test on windows in full CI. Maybe something about clearing ICE files versus quarantine it separately in individual temp dirs, or somehow the RUSTC_ICE env fixed it.

@bors
Copy link
Contributor

bors commented Aug 12, 2024

☀️ Try build successful - checks-actions
Build commit: c9eadc1 (c9eadc1d4e918fb08729a39a36655ce3ad9133cf)

rust-cloud-vms bot pushed a commit to jieyouxu/rust that referenced this pull request Aug 15, 2024
This test was previously flakey on `i686-mingw`, but since some
modifications I could no longer make it fail on `i686-mingw`.

See <rust-lang#128958> for multiple try
job runs.
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 15, 2024
Re-enable `dump-ice-to-disk` for Windows

This test was previously flakey on `i686-mingw`, but since some modifications (quarantining each ICE test in separate tmp dirs, adding/removing `RUSTC_ICE` env vars as suitable to prevent any kind of environmental influence), I could no longer make it fail on `i686-mingw`.

I tried running this test (without the `ignore-windows` of course) a bunch of times via `i686-mingw` try jobs and it refused to fail (see rust-lang#128958).

In any case, if this turns out to be still flakey on `i686-mingw`, we can revert the removal of `ignore-windows` but this time we'll have way more context for why the test failed.

Running the `i686-mingw` alongside some Windows jobs for basic santiy check. But the try jobs succeeding is insufficient to guarantee reproducibility.

cc rust-lang#129115 for backlink.

try-job: x86_64-msvc
try-job: x86_64-mingw
try-job: i686-msvc
try-job: i686-mingw
@jieyouxu jieyouxu closed this Aug 17, 2024
tgross35 added a commit to tgross35/rust that referenced this pull request Aug 17, 2024
…bank

Re-enable `dump-ice-to-disk` for Windows

This test was previously flakey on `i686-mingw` (reason unknown), but since some modifications (quarantining each ICE test in separate tmp dirs, adding/removing `RUSTC_ICE` env vars as suitable to prevent any kind of environmental influence), I could no longer make it fail on `i686-mingw`.

I tried running this test (without the `ignore-windows` of course) a bunch of times via `i686-mingw` try jobs and it refused to fail (see rust-lang#128958). I was also never able to reproduce the failure locally.

In any case, if this turns out to be still flakey on `i686-mingw`, we can revert the removal of `ignore-windows` but this time we'll have way more context for why the test failed.

Running the `i686-mingw` alongside some Windows jobs for basic santiy check. But the try jobs succeeding is insufficient to guarantee reproducibility.

cc rust-lang#129115 for backlink.

try-job: x86_64-msvc
try-job: x86_64-mingw
try-job: i686-msvc
try-job: i686-mingw
tgross35 added a commit to tgross35/rust that referenced this pull request Aug 17, 2024
…bank

Re-enable `dump-ice-to-disk` for Windows

This test was previously flakey on `i686-mingw` (reason unknown), but since some modifications (quarantining each ICE test in separate tmp dirs, adding/removing `RUSTC_ICE` env vars as suitable to prevent any kind of environmental influence), I could no longer make it fail on `i686-mingw`.

I tried running this test (without the `ignore-windows` of course) a bunch of times via `i686-mingw` try jobs and it refused to fail (see rust-lang#128958). I was also never able to reproduce the failure locally.

In any case, if this turns out to be still flakey on `i686-mingw`, we can revert the removal of `ignore-windows` but this time we'll have way more context for why the test failed.

Running the `i686-mingw` alongside some Windows jobs for basic santiy check. But the try jobs succeeding is insufficient to guarantee reproducibility.

cc rust-lang#129115 for backlink.

try-job: x86_64-msvc
try-job: x86_64-mingw
try-job: i686-msvc
try-job: i686-mingw
tgross35 added a commit to tgross35/rust that referenced this pull request Aug 17, 2024
…bank

Re-enable `dump-ice-to-disk` for Windows

This test was previously flakey on `i686-mingw` (reason unknown), but since some modifications (quarantining each ICE test in separate tmp dirs, adding/removing `RUSTC_ICE` env vars as suitable to prevent any kind of environmental influence), I could no longer make it fail on `i686-mingw`.

I tried running this test (without the `ignore-windows` of course) a bunch of times via `i686-mingw` try jobs and it refused to fail (see rust-lang#128958). I was also never able to reproduce the failure locally.

In any case, if this turns out to be still flakey on `i686-mingw`, we can revert the removal of `ignore-windows` but this time we'll have way more context for why the test failed.

Running the `i686-mingw` alongside some Windows jobs for basic santiy check. But the try jobs succeeding is insufficient to guarantee reproducibility.

cc rust-lang#129115 for backlink.

try-job: x86_64-msvc
try-job: x86_64-mingw
try-job: i686-msvc
try-job: i686-mingw
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 18, 2024
Re-enable `dump-ice-to-disk` for Windows

This test was previously flakey on `i686-mingw` (reason unknown), but since some modifications (quarantining each ICE test in separate tmp dirs, adding/removing `RUSTC_ICE` env vars as suitable to prevent any kind of environmental influence), I could no longer make it fail on `i686-mingw`.

I tried running this test (without the `ignore-windows` of course) a bunch of times via `i686-mingw` try jobs and it refused to fail (see rust-lang#128958). I was also never able to reproduce the failure locally.

In any case, if this turns out to be still flakey on `i686-mingw`, we can revert the removal of `ignore-windows` but this time we'll have way more context for why the test failed.

Running the `i686-mingw` alongside some Windows jobs for basic santiy check. But the try jobs succeeding is insufficient to guarantee reproducibility.

cc rust-lang#129115 for backlink.

try-job: x86_64-msvc
try-job: x86_64-mingw
try-job: i686-msvc
try-job: i686-mingw
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-run-make Area: port run-make Makefiles to rmake.rs S-experimental Status: Ongoing experiment that does not require reviewing and won't be merged in its current state.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants