Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 84b9b6d

Browse files
committedMay 19, 2024
Auto merge of rust-lang#125145 - Kobzol:ci-toolstate, r=Mark-Simulacrum
CI: fix toolstate publishing Toolstate publishing after something broke was not working (discovered [here](rust-lang#124050 (comment))). The toolstate env. vars should only be needed for the publishing step, so I moved them there. The toolstate script is also being checked in `mingw-check` on PR and auto CI, but it doesn't really seem to do anything, and it shouldn't require the token.
2 parents 7c73595 + 7b6a3d0 commit 84b9b6d

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed
 

‎.github/workflows/ci.yml

+2-1
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,6 @@ jobs:
190190
env:
191191
AWS_ACCESS_KEY_ID: ${{ env.CACHES_AWS_ACCESS_KEY_ID }}
192192
AWS_SECRET_ACCESS_KEY: ${{ secrets[format('AWS_SECRET_ACCESS_KEY_{0}', env.CACHES_AWS_ACCESS_KEY_ID)] }}
193-
TOOLSTATE_REPO_ACCESS_TOKEN: ${{ secrets.TOOLSTATE_REPO_ACCESS_TOKEN }}
194193

195194
- name: create github artifacts
196195
run: src/ci/scripts/create-doc-artifacts.sh
@@ -241,3 +240,5 @@ jobs:
241240
if: needs.calculate_matrix.outputs.run_type == 'auto'
242241
env:
243242
TOOLSTATE_REPO_ACCESS_TOKEN: ${{ secrets.TOOLSTATE_REPO_ACCESS_TOKEN }}
243+
TOOLSTATE_ISSUES_API_URL: https://api.github.com/repos/rust-lang/rust/issues
244+
TOOLSTATE_PUBLISH: 1

‎src/ci/github-actions/jobs.yml

-2
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,6 @@ envs:
5050
production:
5151
&production
5252
DEPLOY_BUCKET: rust-lang-ci2
53-
TOOLSTATE_ISSUES_API_URL: https://api.github.com/repos/rust-lang/rust/issues
54-
TOOLSTATE_PUBLISH: 1
5553
# AWS_SECRET_ACCESS_KEYs are stored in GitHub's secrets storage, named
5654
# AWS_SECRET_ACCESS_KEY_<keyid>. Including the key id in the name allows to
5755
# rotate them in a single branch while keeping the old key in another

‎src/tools/publish_toolstate.py

+1-2
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@
33

44
# This script computes the new "current" toolstate for the toolstate repo (not to be
55
# confused with publishing the test results, which happens in `src/bootstrap/toolstate.rs`).
6-
# It gets called from `src/ci/publish_toolstate.sh` when a new commit lands on `master`
7-
# (i.e., after it passed all checks on `auto`).
6+
# It gets called from `src/ci/publish_toolstate.sh` at the end of an `auto` build.
87

98
from __future__ import print_function
109

0 commit comments

Comments
 (0)
Failed to load comments.