Skip to content

Tracking issue for better testing for dist artifacts #136822

Open
1 of 4 issues completed
Open

Description

@jieyouxu

This is a tracking issue for improving the testing for dist artifacts (including but not limited to, dist compilers and dist components like rustc-src).

As of the time of writing, much of dist components are not really tested or exercised in a basic way (modulo tests through opt-dist, but not all dist artifacts go through opt-dist).

Example regressions

Activity

  1. added
    A-testsuiteArea: The testsuite used to check the correctness of rustc
    C-bugCategory: This is a bug.
    T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.
    T-infraRelevant to the infrastructure team, which will review and decide on the PR/issue.
    on Feb 10, 2025
  2. added
    needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging
    on Feb 10, 2025
  3. removed
    needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging
    on Feb 10, 2025
  4. jieyouxu commented on Feb 10, 2025

    @jieyouxu
    MemberAuthor

    I know that x86_64-unknown-linux-gnu and aarch64-unknown-linux-gnu dist artifacts do receive some degree of testing due to post-PGO opt-dist tests.

    It's the apple and windows dist artifacts that I don't know if are tested at all.

  5. ChrisDenton commented on Feb 10, 2025

    @ChrisDenton
    Member

    Yes I think not testing CI dist artefacts is the default rather than the exception.

  6. changed the title [-]`i686-pc-windows-gnu` dist artifacts are not tested[/-] [+]dist artifacts are generally not tested[/+] on Feb 10, 2025
  7. Noratrieb commented on Feb 10, 2025

    @Noratrieb
    Member

    If we call the issue like that, no one will bother to ever fix it of course. But this does actually affect more than just i686-pc-windows-gnu so someone might actually care.

  8. Noratrieb commented on Feb 23, 2025

    @Noratrieb
    Member

    A bootstrap PR broke tools completely after dist, #137469. Smoke tests would have prevented that as well.

  9. jieyouxu commented on Mar 11, 2025

    @jieyouxu
    MemberAuthor

    See also #138306. That is a case where the rustc-src dist component broke because we tried to use workspace lint table of the rust-lang/rust root manifest, which is not shipped as part of this dist component, so compiler sources no longer build on their own. I didn't catch this nor did the PR author because we would've needed to inspect the rustc-src dist component...

  10. added
    T-bootstrapRelevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
    on Mar 18, 2025
  11. changed the title [-]dist artifacts are generally not tested[/-] [+]Tracking issue for better testing for dist artifacts[/+] on Mar 18, 2025
  12. added
    C-tracking-issueCategory: An issue tracking the progress of sth. like the implementation of an RFC
    and removed
    C-bugCategory: This is a bug.
    on Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-testsuiteArea: The testsuite used to check the correctness of rustcC-tracking-issueCategory: An issue tracking the progress of sth. like the implementation of an RFCT-bootstrapRelevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-infraRelevant to the infrastructure team, which will review and decide on the PR/issue.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions