Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix bittensor related workflows (e2e testing) #1442

Merged
merged 13 commits into from
Mar 21, 2025
Merged

Fix bittensor related workflows (e2e testing) #1442

merged 13 commits into from
Mar 21, 2025

Conversation

roman-opentensor
Copy link
Contributor

@roman-opentensor roman-opentensor commented Mar 20, 2025 •
edited
Loading

  • Fixed a bug where bittensor tests were run on an already created docker image, but not based on the current code in the PR branch.
  • Bumping versions of Actions related to docker's workflow.
  • 2 workflows are combined into one (jobs still runs in parallel).
  • Docker Image created once then used for all sdk and cli tests in parallel.
  • Workflow uses/checks one run-bittensor-e2e-tests label instead of two.
  • The build check of the Docker image subtensor-localnet based on the PR branch happens in the same as e2e tests workflow.

@roman-opentensor roman-opentensor self-assigned this Mar 20, 2025
@roman-opentensor roman-opentensor added no-spec-version-bump PR does not contain changes that requires bumping the spec version skip-cargo-audit This PR fails cargo audit but needs to be merged anyway run-bittensor-cli-tests labels Mar 20, 2025
@roman-opentensor roman-opentensor added run-bittensor-e2e-tests Runs Bittensor SDK and BTCLI tests and removed run-bittensor-sdk-tests run-bittensor-e2e-tests Runs Bittensor SDK and BTCLI tests labels Mar 20, 2025
@roman-opentensor roman-opentensor requested review from sam0x17, a team and camfairchild March 20, 2025 23:34
@sam0x17 sam0x17 merged commit dfe3161 into devnet-ready Mar 21, 2025
48 of 49 checks passed
@roman-opentensor roman-opentensor deleted the fix/roman/use-docker-image-from-pr-branch branch March 21, 2025 03:13
This was referenced Mar 25, 2025
Merged
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sam0x17 sam0x17 sam0x17 approved these changes

@JohnReedV JohnReedV JohnReedV approved these changes

@camfairchild camfairchild Awaiting requested review from camfairchild

Assignees

@roman-opentensor roman-opentensor

Labels
no-spec-version-bump PR does not contain changes that requires bumping the spec version run-bittensor-e2e-tests Runs Bittensor SDK and BTCLI tests skip-cargo-audit This PR fails cargo audit but needs to be merged anyway
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@roman-opentensor @sam0x17 @JohnReedV