Skip to content

Migrate GitHub Actions to Blacksmith runners#13

Open
jai-deepsource wants to merge 1 commit intomasterfrom
chore/migrate-to-blacksmith
Open

Migrate GitHub Actions to Blacksmith runners#13
jai-deepsource wants to merge 1 commit intomasterfrom
chore/migrate-to-blacksmith

Conversation

@jai-deepsource
Copy link
Contributor

Summary

  • Migrate GitHub Actions workflows from GitHub-hosted runners to Blacksmith runners
  • Delete unused/deprecated workflow files where applicable
  • Heavy workflows (builds, tests, Docker) use blacksmith-4vcpu-ubuntu-2404
  • Light workflows (dispatch, lint, sync) use blacksmith-2vcpu-ubuntu-2404
  • macOS, Windows, and self-hosted runners are left unchanged

Related

  • ENG-4173 (Blacksmith migration)
  • ENG-4174 (Workflow cleanup)

- Replace GitHub-hosted ubuntu runners with Blacksmith equivalents
- Remove unused/deprecated workflow files where applicable
- Use blacksmith-4vcpu-ubuntu-2404 for builds and tests
- Use blacksmith-2vcpu-ubuntu-2404 for lightweight jobs
Copy link
Contributor

@aseem-deepsource aseem-deepsource left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high tier 4vcpu is not needed here, let's use the 2vcpu lowest spec ones.

jobs:
setup:
runs-on: ubuntu-latest
runs-on: blacksmith-4vcpu-ubuntu-2404
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
runs-on: blacksmith-4vcpu-ubuntu-2404
runs-on: blacksmith-2vcpu-ubuntu-2404

release:
needs: [ setup ]
runs-on: ubuntu-latest
runs-on: blacksmith-4vcpu-ubuntu-2404
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
runs-on: blacksmith-4vcpu-ubuntu-2404
runs-on: blacksmith-2vcpu-ubuntu-2404

packages: write # needed for ghcr access

runs-on: ubuntu-latest
runs-on: blacksmith-4vcpu-ubuntu-2404
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
runs-on: blacksmith-4vcpu-ubuntu-2404
runs-on: blacksmith-2vcpu-ubuntu-2404

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants