Skip to content

Migrate workflows that need write access to use hosted runners#2077

Merged
pietern merged 2 commits intomainfrom
switch-runners
Jan 6, 2025
Merged

Migrate workflows that need write access to use hosted runners#2077
pietern merged 2 commits intomainfrom
switch-runners

Conversation

@pietern
Copy link
Copy Markdown
Contributor

@pietern pietern commented Jan 6, 2025 •
edited
Loading

Changes

Migrate workflows to Databricks-hosted GitHub Actions runners.

The GitHub-hosted runners can no longer be used because of security hardening.

@denik
Copy link
Copy Markdown
Contributor

denik commented Jan 6, 2025 •
edited by pietern
Loading

  1. Could you add motivation for this change?
  2. Will we be be to move the integration test workflow into this repository?

@pietern
Copy link
Copy Markdown
Contributor Author

pietern commented Jan 6, 2025

@denik

  1. Added mention of security hardening to the summary.
  2. No, that's unrelated.

@pietern pietern added this pull request to the merge queue Jan 6, 2025
Merged via the queue into main with commit c262b30 Jan 6, 2025
@pietern pietern deleted the switch-runners branch January 6, 2025 15:40
@pietern pietern mentioned this pull request Jan 6, 2025
Merged
github-merge-queue bot pushed a commit that referenced this pull request Jan 6, 2025
@pietern
Enable integration workflows (#2086)
3629c9e 
## Changes

This reverts commit 3155285.

These workflows were disabled in #2085.

They should work again now that we're using self-hosted runners (see
#2077).

## Tests

(inline)
This was referenced Jan 6, 2025
Closed
Merged
github-merge-queue bot pushed a commit that referenced this pull request Jan 8, 2025
@pietern
Update runner for the create-xyz-pr jobs in the release workflow (#2098)
42b34c7 
## Changes

I missed these in #2077 and they failed because of it on the v0.238.0
release.
pietern added a commit that referenced this pull request Jan 9, 2025
@pietern
Update runner for the publish-winget job
c195f58 
This action uses a token to access the release artifacts and as such needs to
execute on runner that's on the allowlist.

Related PRs:
* #2098
* #2077
@pietern pietern mentioned this pull request Jan 9, 2025
Merged
github-merge-queue bot pushed a commit that referenced this pull request Jan 9, 2025
@pietern
Update runner for the publish-winget job (#2105)
3b3ede6 
## Changes

This action uses a token to access the release artifacts and, as such,
needs to execute on the runner that's on the allowlist.

Related PRs:
* #2098
* #2077
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@denik denik denik approved these changes

@andrewnester andrewnester andrewnester approved these changes

@shreyas-goenka shreyas-goenka Awaiting requested review from shreyas-goenka shreyas-goenka is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pietern @denik @andrewnester