Skip to content

chore(gha): run integration tests on Depot runners - #238

Merged
so0k merged 2 commits into
open-constructs:mainfrom
sakul-learning:ci/depot-integration-runners
Jun 4, 2026
Merged

so0k merged 2 commits into
open-constructs:mainfrom
sakul-learning:ci/depot-integration-runners

Conversation

@sakul-learning

@sakul-learning sakul-learning commented Jun 4, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Move Linux integration test jobs to Depot's depot-ubuntu-24.04-16 runner.
  • Move provider integration preparation and Linux provider matrix jobs to Depot runners.
  • Keep Windows integration/provider jobs on their existing Windows runners.

Test Plan

  • python3 -c 'import yaml; [yaml.safe_load(open(f)) for f in [".github/workflows/integration.yml", ".github/workflows/provider-integration.yml"]]'
  • /home/ubuntu/.cache/go-tools/actionlint -ignore 'label "depot-ubuntu-24.04-16" is unknown' -ignore 'constant expression "false" in condition' .github/workflows/integration.yml .github/workflows/provider-integration.yml
  • git diff --check

@sakul-learning
sakul-learning requested a review from a team as a code owner June 4, 2026 13:41
@sakul-learning

Copy link
Copy Markdown
Contributor Author

Good point. I looked up the Depot runner sizes and depot-ubuntu-24.04-16 is 16 vCPU / 64 GB, which is larger than these individual integration jobs should need.

GitHub's standard Linux runner for public repos is 4 vCPU / 16 GB, so I switched the Linux integration and provider-integration jobs to depot-ubuntu-24.04-4 instead. That keeps the job size comparable to the public GitHub-hosted runner capacity while still moving the jobs off the GitHub-hosted runner queue. The main win here is avoiding the long free-runner queue time rather than increasing per-job compute.

Updated in 22fdb7f.

@sakul-learning sakul-learning changed the title ci: run integration tests on Depot runners chore(gha): run integration tests on Depot runners Jun 4, 2026
@so0k
so0k enabled auto-merge (squash) June 4, 2026 16:36
@so0k
so0k merged commit 4f1a674 into open-constructs:main Jun 4, 2026
257 checks passed
so0k added a commit that referenced this pull request Jun 5, 2026
## 0.23.3

### chore

- chore(gha): run integration tests on Depot runners
[\#238](#238)
- chore(deps): bump typescript to 5.9.3 across the repo
[\#231](#231)
- chore: Upgrade jsii
[\#223](#223)
- chore: Revise README links to documentation and language support
[\#221](#221)
- chore: Pin node version to 22.22.2 via .nvmrc
[\#212](#212)
- chore: Add gradle cache to the examples and integration CI workflows
[\#211](#211)
- chore: Add PR workflow concurrency group and CI Label Filter job
[\#204](#204)
- chore(tests): add verdaccio to package.json
[\#202](#202)
- chore(tests): don't require other language builds
[\#201](#201)
- chore(tests): Ensure we terminate Verdaccio
[\#199](#199)
- chore: rewrite https://cdk.tf links
[\#197](#197)
- chore: Update examples help files
[\#194](#194)

### feat

- feat: faster JSON stringify
[\#224](#224)
- feat(lib): allow disabling creation stacks
[\#215](#215)

### fix

- fix(lib): surface stderr/stdout on exec() failures via toString()
[\#207](#207)

---------

Co-authored-by: so0k <vincent.drl@gmail.com>
Co-authored-by: sakul-learning <lucas@sheesh.host>
@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants