Skip to content

Conversation

@pdp2121
Copy link
Collaborator

@pdp2121 pdp2121 commented Nov 26, 2025

High Level Overview of Change

With current logic, if a validator has not participated in the consensus process for more than a week, it will be removed from the list. This led to an UNL validator, even though still in the UNL list, got removed by the logic.
This PR would:

  • Retain inactive UNL validators that are still discoverable in the UNL list in the database, until it got removed from the list.
  • Increase the purging period from one week to thirty days.

Resolves #395
Replaces #397

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Tests (You added tests for code that already exists, or your new feature included in this PR)
  • Documentation Updates
  • Release

@pdp2121 pdp2121 mentioned this pull request Nov 26, 2025
7 tasks
@pdp2121 pdp2121 requested review from ckeshava and kuan121 November 26, 2025 16:50
@pdp2121 pdp2121 requested a review from kuan121 November 26, 2025 17:59
kuan121
kuan121 previously approved these changes Nov 26, 2025
kuan121
kuan121 previously approved these changes Nov 26, 2025
@pdp2121 pdp2121 merged commit 8714420 into main Dec 15, 2025
3 checks passed
@pdp2121 pdp2121 deleted the purge-retain-unl-vals branch December 15, 2025 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Retain inactive UNL validators

4 participants