Skip to content
This repository was archived by the owner on Jul 10, 2026. It is now read-only.

Run bittensor e2e concurrently#358

Merged
thewhaleking merged 9 commits into
stagingfrom
feat/thewhaleking/run-bittensor-e2e-concurrently
Jun 1, 2026
Merged

Run bittensor e2e concurrently#358
thewhaleking merged 9 commits into
stagingfrom
feat/thewhaleking/run-bittensor-e2e-concurrently

Conversation

@thewhaleking

@thewhaleking thewhaleking commented Jun 1, 2026

Copy link
Copy Markdown
Contributor
  1. Use a reusable image
  2. Run each test case concurrently rather than each test file
  3. Adds retry for flaky tests

@github-actions github-actions Bot added run-bittensor-sdk-tests Runs Bittensor SDK tests. run-bittensor-cli-tests Runs BTCLI tests. labels Jun 1, 2026
basfroman
basfroman previously approved these changes Jun 1, 2026

@basfroman basfroman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This logic update looks so grate. Just consider additional improvement which I mentioned in the comment. Otherwise LGTM

Comment thread .github/workflows/check-sdk-tests.yml Outdated
basfroman
basfroman previously approved these changes Jun 1, 2026

@basfroman basfroman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: consider adding && needs.build-e2e-image.result == 'success' to avoid spawning all matrix jobs whenbuild-e2e-image fails

@thewhaleking thewhaleking merged commit 89e58fa into staging Jun 1, 2026
186 checks passed
@thewhaleking thewhaleking deleted the feat/thewhaleking/run-bittensor-e2e-concurrently branch June 1, 2026 17:20
This was referenced Jun 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

run-bittensor-cli-tests Runs BTCLI tests. run-bittensor-sdk-tests Runs Bittensor SDK tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants