Skip to content

Subnet Emission Owner Toggle#2657

Merged
sam0x17 merged 11 commits into
devnet-readyfrom
SubnetEmissionEnabled
May 14, 2026
Merged

Subnet Emission Owner Toggle#2657
sam0x17 merged 11 commits into
devnet-readyfrom
SubnetEmissionEnabled

Conversation

@JohnReedV
Copy link
Copy Markdown
Contributor

@JohnReedV JohnReedV commented May 12, 2026

Description

This PR adds an owner/root admin-utils call to disable pool-side subnet emission.

  • Adds a SubnetEmissionEnabled toggle
  • Disables alpha_in, tao_in, and chain buys when toggled off
  • Redistributes disabled subnet TAO emission to enabled subnets to preserve full block emission
  • Preserves alpha_out, root prop, and normal validator/server emissions
  • Resets the toggle for newly initialized subnets
  • Adds owner/root rate-limit tracking for the new setting
  • Adds tests for redistribution and multi-subnet toggle behavior

@JohnReedV JohnReedV added the skip-cargo-audit This PR fails cargo audit but needs to be merged anyway label May 12, 2026
Copy link
Copy Markdown
Collaborator

@l0r1s l0r1s left a comment

Choose a reason for hiding this comment

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

Missing tests

Comment thread pallets/admin-utils/src/lib.rs Outdated
Comment thread pallets/subtensor/src/coinbase/subnet_emissions.rs Outdated
Comment thread pallets/subtensor/src/coinbase/subnet_emissions.rs Outdated
Comment thread pallets/subtensor/src/coinbase/subnet_emissions.rs Outdated
l0r1s
l0r1s previously approved these changes May 13, 2026
evgeny-s
evgeny-s previously approved these changes May 13, 2026
l0r1s
l0r1s previously approved these changes May 13, 2026
@github-actions github-actions Bot dismissed stale reviews from l0r1s and evgeny-s via bfbd4fb May 13, 2026 22:45
@sam0x17 sam0x17 merged commit d1dee08 into devnet-ready May 14, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

4 participants