Skip to content

Switch to environment files to change the environment on GHA - #77418

Merged
bors merged 1 commit into
rust-lang:masterfrom
emilyalbini:ci-environment-files
Oct 2, 2020
Merged

Switch to environment files to change the environment on GHA#77418
bors merged 1 commit into
rust-lang:masterfrom
emilyalbini:ci-environment-files

Conversation

@emilyalbini

Copy link
Copy Markdown
Member

This PR switches our CI to use GitHub's new environment files to set environment variables and add items to $PATH, as GitHub plans to disable the old ::set-env and ::add-path log commands due to a security issue.

Our self-hosted runners are on version 2.273.4, so they already have support for this.

r? @Mark-Simulacrum

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 1, 2020
@emilyalbini

Copy link
Copy Markdown
MemberAuthor

@bors p=1000
@rustbot modify labels: T-infra beta-nominated beta-accepted

@rustbotrustbot added beta-accepted Accepted for backporting to the compiler in the beta channel. beta-nominated Nominated for backporting to the compiler in the beta channel. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Oct 1, 2020
@camelidcamelid added the A-github-actions Area: GitHub Actions (GHA) label Oct 1, 2020
@Mark-Simulacrum

Copy link
Copy Markdown
Member

@bors r+ p=100

@bors

bors commented Oct 2, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit b710f9c has been approved by Mark-Simulacrum

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 2, 2020
@bors

bors commented Oct 2, 2020

Copy link
Copy Markdown
Collaborator

⌛ Testing commit b710f9c with merge a585aae...

@bors

bors commented Oct 2, 2020

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-actions, checks-azure
Approved by: Mark-Simulacrum
Pushing a585aae to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Oct 2, 2020
@bors
bors merged commit a585aae into rust-lang:masterOct 2, 2020
@rustbotrustbot added this to the 1.48.0 milestone Oct 2, 2020
@Mark-SimulacrumMark-Simulacrum removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Oct 3, 2020
@Mark-SimulacrumMark-Simulacrum modified the milestones: 1.48.0, 1.47.0Oct 3, 2020
@Mark-SimulacrumMark-Simulacrum mentioned this pull request Oct 3, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 3, 2020
…k-Simulacrum
[beta] backports
This backports a number of PRs to beta, not all of which have been approved (yet).
* Switch to environment files to change the environment on GHA rust-lang#77418
* cache types during normalization rust-lang#76928
* Fixing memory exhaustion when formatting short code suggestion rust-lang#76598
* Issue 72408 nested closures exponential rust-lang#72412
r? `@Mark-Simulacrum`
@emilyalbini
emilyalbini deleted the ci-environment-files branch December 23, 2020 22:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-github-actionsArea: GitHub Actions (GHA)beta-acceptedAccepted for backporting to the compiler in the beta channel.merged-by-borsThis PR was explicitly merged by bors.S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-infraRelevant 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.

6 participants

@emilyalbini@Mark-Simulacrum@bors@rust-highfive@camelid@rustbot