Skip to content

app/upgrades: add v1.11.1 audit min_disk_free_percent floor#106

Open
mateeullahmalik wants to merge 1 commit intomasterfrom
feat/audit-min-disk-free-floor-mainnet
Open

app/upgrades: add v1.11.1 audit min_disk_free_percent floor#106
mateeullahmalik wants to merge 1 commit intomasterfrom
feat/audit-min-disk-free-floor-mainnet

Conversation

@mateeullahmalik
Copy link
Contributor

Summary

  • add a new custom upgrade handler v1.11.1
  • enforce audit.params.min_disk_free_percent >= 15 at upgrade execution
  • keep behavior monotonic: only raise to floor, never downgrade if governance already set a higher value
  • wire v1.11.1 into the central upgrade registry and upgrade tests

Context

Mainnet needs a deterministic, upgrade-time safety floor for audit host disk-free enforcement, without mutating historical v1.11.0 semantics.

Safety / Correctness

  • no store key additions/deletions
  • runs RunMigrations first, then applies bounded param update
  • fails fast if AuditKeeper wiring is missing
  • preserves deterministic behavior and minimal blast radius

Files Changed

  • app/upgrades/upgrades.go
  • app/upgrades/upgrades_test.go
  • app/upgrades/v1_11_1/upgrade.go (new)
  • app/upgrades/v1_11_1/upgrade_test.go (new)

Tests

Executed:

  • go test ./app/upgrades/... ./x/audit/v1/keeper/...

All passed.

@roomote-v0
Copy link

roomote-v0 bot commented Mar 13, 2026

Rooviewer Clock   See task

Reviewed all changed files. The upgrade handler correctly enforces a monotonic floor on audit.params.min_disk_free_percent, follows existing upgrade patterns, and has appropriate test coverage. No issues found.

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@mateeullahmalik mateeullahmalik self-assigned this Mar 13, 2026
@mateeullahmalik mateeullahmalik requested a review from a-ok123 March 13, 2026 17: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.

1 participant