Skip to content

Tracking issue for migrating from Glacier to compiletest ICE/crash tracking #125459

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

Open
2 of 9 tasks
fmease opened this issue May 23, 2024 · 3 comments
Open
2 of 9 tasks
Labels
A-compiletest Area: The compiletest test runner A-testsuite Area: The testsuite used to check the correctness of rustc C-tracking-issue Category: An issue tracking the progress of sth. like the implementation of an RFC T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@fmease
Copy link
Member

fmease commented May 23, 2024

Tracks the necessary steps to "fully"1 transition from rust-lang/glacier to compiletest ICE/crash tracking inside tests/crashes/.
See also this Zulip topic.

Steps

  • Implement rustc ICE/crash tracking: compiletest ice tracking #122997
  • Implement rustdoc ICE/crash tracking
  • Implement rustfmt ICE/crash tracking (questionable)
  • Implement ICE/crash tracking via run-make/rmake (to supersede Glacier's shell scripts)
  • Archive rust-lang/glacier: 🧊 Put glacier on ice 🧊 team#1459
  • Sunset triagebot support for Glacier
  • Update the rustc-dev-guide
    • Remove mentions of Glacier
    • Add small section about tests/crashes/: Overview, rough procedure (adding //@ known-bug, S-bug-has-test Status: This bug is tracked inside the repo by a `known-bug` test. )
    • Mention tests/crashes/ in places where we reference E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
  • Update the Rust Forge
  • If technically feasible, add a CI check that a PR fixing crash tests mentions all fixed issues via GH auto-close keywords like Fixes to prevent the accumulation of "ghost issues".

Unresolved Questions

  • Should we implement rustdoc support via subdirectories/tests suites (e.g., tests/crashes/rustdoc/) or via a new compiletest directive (e.g., //@ bin: rustdoc)
    • If we do the former, should we move rustc tests from tests/crashes/ to tests/crashes/rustc/?
  • Should we define precise procedures for adding new tests to tests/crashes?

Footnotes

  1. Glacier|_{rust-lang/rust}, i.e., restriction of Glacier to rust-lang/rust (excluding git submodules/subtrees), i.e., Glacier without support for rustfmt, Cargo, Clippy, Miri ICE/crash tracking

@fmease fmease added A-testsuite Area: The testsuite used to check the correctness of rustc T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. C-tracking-issue Category: An issue tracking the progress of sth. like the implementation of an RFC labels May 23, 2024
@fmease
Copy link
Member Author

fmease commented May 23, 2024

cc @matthiaskrgr

@fmease fmease changed the title Tracking issue for migrating from Glacier to compiletest ICE tracking Tracking issue for migrating from Glacier to compiletest ICE/crash tracking May 23, 2024
@matthiaskrgr
Copy link
Member

I'm not sure if it makes a lot of sense to track crashes inside the rustc repo for tools (rustfmt, clippy, miri etc) that are actually living inside their own repos because in most cases I guess you would "accidentally" fix something during a submodule/tree sync?

@fmease
Copy link
Member Author

fmease commented May 23, 2024

Good point. I guess that means T-rustfmt/T-cargo/... for example will need to introduce their own mechanism assuming they're interested in doing that and that "we" won't try to find a Glacier replacement for them now that it's officially archived.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-compiletest Area: The compiletest test runner A-testsuite Area: The testsuite used to check the correctness of rustc C-tracking-issue Category: An issue tracking the progress of sth. like the implementation of an RFC T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

3 participants