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] Rustc bisect #18445

Closed
wants to merge 8 commits into from
Closed

[Do not merge] Rustc bisect #18445

wants to merge 8 commits into from

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Sep 11, 2017

I could not reproduce locally the assertions failures in #18420, so I’ll try bisecting compiler versions on try CI.


This change is Reviewable

@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 11, 2017

bors-servo added a commit that referenced this pull request Sep 11, 2017
[Do not merge] Rustc bisect

I could not reproduce locally the assertions failures in #18420, so I’ll try bisecting compiler versions on try CI.
@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

Trying commit 9bcc7d2 with merge c895049...

@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

💔 Test failed - linux-rel-wpt

@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 11, 2017

  ▶ FAIL [expected PASS] /_mozilla/css/css-position-3/position-sticky-table-th-right.html
  └   → /_mozilla/css/css-position-3/position-sticky-table-th-right.html e9b9d42d83d50f68d193d258f7b8e977d06a08b7
/_mozilla/css/css-position-3/position-sticky-table-th-right-ref.html 8a37208e44c1426c46cd704a1051120fa696710f
Testing e9b9d42d83d50f68d193d258f7b8e977d06a08b7 == 8a37208e44c1426c46cd704a1051120fa696710f

Looks like assertion failed: distance_to_stack_bottom <= max_allowable_distance did not occur with Nightly 2017-09-04.

Trying 2017-09-06 now:

@bors-servo try

@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

Trying commit 29a028d with merge ce32027...

bors-servo added a commit that referenced this pull request Sep 11, 2017
[Do not merge] Rustc bisect

I could not reproduce locally the assertions failures in #18420, so I’ll try bisecting compiler versions on try CI.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18445)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 11, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

Trying commit 9f9648f with merge 0e9f67c...

bors-servo added a commit that referenced this pull request Sep 11, 2017
[Do not merge] Rustc bisect

I could not reproduce locally the assertions failures in #18420, so I’ll try bisecting compiler versions on try CI.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18445)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 11, 2017

2017-09-06 and 2017-09-07 are "good" (did not produce that assertion failure in one try run each) too. Trying 2017-09-08 now, same as in #18420.

@bors-servo try

@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

Trying commit c5e6869 with merge 544a022...

bors-servo added a commit that referenced this pull request Sep 11, 2017
[Do not merge] Rustc bisect

I could not reproduce locally the assertions failures in #18420, so I’ll try bisecting compiler versions on try CI.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18445)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

💔 Test failed - linux-rel-wpt

This reverts commit c5e6869.
@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 11, 2017

assertion failed: distance_to_stack_bottom <= max_allowable_distance spotted on 2017-09-08. Trying 2017-09-07 again in case the previous run was a coincidence.

@bors-servo try

@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

Trying commit e738503 with merge 7bce39b...

bors-servo added a commit that referenced this pull request Sep 11, 2017
[Do not merge] Rustc bisect

I could not reproduce locally the assertions failures in #18420, so I’ll try bisecting compiler versions on try CI.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18445)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 11, 2017

Alright, I think it’s fairly safe to say that something in rust-lang/rust@3681220...d93036a is causing this assertions to be triggered.

@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 11, 2017

Now with some mach hacking to use per-PR builds from Rust CI instead of Nightly.

@bors-servo try

@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

Trying commit 043474e with merge 536f788...

bors-servo added a commit that referenced this pull request Sep 11, 2017
[Do not merge] Rustc bisect

I could not reproduce locally the assertions failures in #18420, so I’ll try bisecting compiler versions on try CI.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18445)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

💔 Test failed - linux-dev

@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 11, 2017

Tidy is unhappy, but other jobs look green enough to call rust-lang/rust@05e3c96 "good".

Trying rust-lang/rust@d7d75ef now:

@bors-servo try

@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

Trying commit b5fd42a with merge 2dea6c0...

bors-servo added a commit that referenced this pull request Sep 11, 2017
[Do not merge] Rustc bisect

I could not reproduce locally the assertions failures in #18420, so I’ll try bisecting compiler versions on try CI.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18445)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 11, 2017

💔 Test failed - linux-rel-wpt

@jdm
Copy link
Member

jdm commented Sep 11, 2017

ERROR:servo: assertion failed: distance_to_stack_bottom <= max_allowable_distance
@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 12, 2017

Thanks Josh. Trying rust-lang/rust@a6a9d4c, last run to reduce this to a single Rust PR.

@bors-servo try

@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 12, 2017

Oops, it works better if I don’t forget git push.

@bors-servo try

@bors-servo
Copy link
Contributor

bors-servo commented Sep 12, 2017

Trying commit cd98775 with merge 038f219...

bors-servo added a commit that referenced this pull request Sep 12, 2017
[Do not merge] Rustc bisect

I could not reproduce locally the assertions failures in #18420, so I’ll try bisecting compiler versions on try CI.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18445)
<!-- Reviewable:end -->
@SimonSapin
Copy link
Member Author

SimonSapin commented Sep 12, 2017

Looks like homu didn’t get the memo, but buildbot jobs are all green. This leaves rust-lang/rust#43931 as responsible (though possibly indirectly).

@SimonSapin SimonSapin closed this Sep 12, 2017
@SimonSapin SimonSapin deleted the rustup-bisect branch Sep 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.