Skip to content

Avoid uploading package-version on forced-channel jobs#158187

Open
Mark-Simulacrum wants to merge 1 commit into
rust-lang:mainfrom
Mark-Simulacrum:fix-stable-pkg-version
Open

Avoid uploading package-version on forced-channel jobs#158187
Mark-Simulacrum wants to merge 1 commit into
rust-lang:mainfrom
Mark-Simulacrum:fix-stable-pkg-version

Conversation

@Mark-Simulacrum

Copy link
Copy Markdown
Member

This caused Crater to fail when installing the toolchain for commit 057ed8b, because it assumed the toolchain files would have package-version 1.98.0 (instead of nightly).

This adjusts the code to always upload based on the canonical channel instead of the override.

See also notes on Zulip - #t-infra > Crater maybe stuck? @ 💬

This caused Crater to fail when installing the toolchain for commit
057ed8b, because it assumed the
toolchain files would have package-version 1.98.0 (instead of nightly).

This adjusts the code to always upload based on the canonical channel
instead of the override.
@rustbot rustbot added A-CI Area: Our Github Actions CI 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. labels Jun 20, 2026
@Mark-Simulacrum

Copy link
Copy Markdown
Member Author

@bors try jobs=dist-x86_64-linux,x86_64-gnu-stable

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jun 20, 2026
Avoid uploading package-version on forced-channel jobs


try-job: dist-x86_64-linux
try-job: x86_64-gnu-stable
@rust-bors

rust-bors Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: 4108ad6 (4108ad65115e0a2ae429a947962fc32b5fb20f5a)
Base parent: fffed27 (fffed276b51cf514622e8d3109d620501cc01670)

@Mark-Simulacrum Mark-Simulacrum marked this pull request as ready for review June 21, 2026 00:21
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 21, 2026
@rustbot rustbot removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Jun 21, 2026
@rustbot

rustbot commented Jun 21, 2026

Copy link
Copy Markdown
Collaborator

r? @marcoieni

rustbot has assigned @marcoieni.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: infra-ci
  • infra-ci expanded to Kobzol, Mark-Simulacrum, jdno, jieyouxu, marcoieni
  • Random selection from jdno, marcoieni

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CI Area: Our Github Actions CI 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants