Skip to content

Ensure stake threshold is respected when handing out vpermits #1629

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.

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

Merged
merged 4 commits into from
May 19, 2025

Conversation

keithtensor
Copy link
Contributor

Fixes #1058.

@github-actions github-actions bot added the run-bittensor-e2e-tests Runs Bittensor SDK and BTCLI tests label May 13, 2025
@keithtensor
Copy link
Contributor Author

Looks like the fix is working too well, to the point where if the stake of a validator is 0, it gets automatically pruned out to not be a validator, which makes testing a bit annoying. We'll have to set the stake of the validator to at least be 1 first.

@keithtensor keithtensor added the skip-cargo-audit This PR fails cargo audit but needs to be merged anyway label May 13, 2025
@camfairchild
Copy link
Contributor

Kinda makes #1614 redundant

@sam0x17 sam0x17 merged commit fd70d5d into devnet-ready May 19, 2025
53 checks passed
@keithtensor keithtensor deleted the vpermit-respect-min-stake branch May 19, 2025 13:46
This was referenced May 19, 2025
@basfroman
Copy link
Contributor

SDK e2e test fixed opentensor/bittensor#2884

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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
Development

Successfully merging this pull request may close these issues.

5 participants