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.

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

Implement signer changes for time-based tenure extensions #5467

Open
obycode opened this issue Nov 15, 2024 · 0 comments
Open

Implement signer changes for time-based tenure extensions #5467

obycode opened this issue Nov 15, 2024 · 0 comments
Assignees
Labels
Milestone

Comments

@obycode
Copy link
Contributor

obycode commented Nov 15, 2024

The signer configuration will specify a tenure extend time period. The first version of this to go live on mainnet should start off with this value defaulting to something like 5 minutes. As we validate that these tenure extends do not cause any problems, we can spread the word to signers to iteratively lower this number.

When a new burn block arrives, record the current time, idle_start, and initialize an idle_countdown counter to the configured duration. When a block proposal arrives, record the time, process_start. The block validation endpoint will validate the block and return the cost of that block. If the block has a non-zero cost, subtract (process_start - idle_start) from the idle_countdown. If it has a 0 cost, then subtract (now - idle_start) from the idle_countdown. This difference in how the idle time is computed is important to encourage miners to continue mining blocks with STX transfers after their budget is spent but before enough idle time has passed for a tenure extend.

In the BlockResponse for this proposal, include a timestamp which is current time plus idle_countdown. Once the response is sent, record the current time again to idle_start. Repeat with each block proposal.

We keep track of "idle" time instead of just flat wall time because it allows the signers to factor in how long it actually takes to process the blocks. This will flatten out the total processing time in scenarios where the cost budgeting is overly pessimistic, causing us to see some blocks that can spend the entire budget and be processed in 3 seconds, while others that spend the entire budget take 3 minutes to process.

If a block proposal arrives that contains a TenureExtend transaction and the tenure_consensus_hash is equal to the burn_view_consensus_hash, check that the current idle_countdown is less than or equal to 0 (letting this value go negative is useful feedback to the miner). If so, process the block as usual, else, reject the block. This rejection would have a new reason code.

See #5434

@github-project-automation github-project-automation bot moved this to Status: 🆕 New in Stacks Core Eng Nov 15, 2024
@aldur aldur modified the milestones: SIP-29, Tenure extend Nov 18, 2024
@aldur aldur moved this from Status: 🆕 New to Status: 📋 Backlog in Stacks Core Eng Nov 18, 2024
@aldur aldur added the signer label Nov 18, 2024
@hstove hstove moved this from Status: 💻 In Progress to Status: In Review in Stacks Core Eng Dec 3, 2024
@aldur aldur moved this from Status: In Review to Status: ✅ Done in Stacks Core Eng Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Status: ✅ Done
Development

No branches or pull requests

3 participants