Skip to content

[BOUNTY] Fix HighMemoryUsage alert ratio expression - #9

Open
leo202000 wants to merge 6 commits into
weilixiong:mainfrom
leo202000:bounty/1-fix-highmemory-self-dividing
Open

[BOUNTY] Fix HighMemoryUsage alert ratio expression#9
leo202000 wants to merge 6 commits into
weilixiong:mainfrom
leo202000:bounty/1-fix-highmemory-self-dividing

Conversation

@leo202000

Copy link
Copy Markdown

Summary

Fix HighMemoryUsage Prometheus alert: replaced self-dividing expression with meaningful ratio and added validation for self-dividing alert expressions.

Changes

  • Fixed HighMemoryUsage from process_resident_memory_bytes / process_resident_memory_bytes > 0.9 (self-dividing, always 1) to process_resident_memory_bytes / machine_memory_bytes > 0.9 (actual memory ratio)
  • Added �alidate_alert_expressions() function that detects self-dividing expressions in alert rules

Testing

  • Python syntax validation passed
  • Alert expression now uses meaningful denominator

Checklist

  • Relevant modules build locally
  • Changes are scoped to the PR purpose
  • No external dependencies

@leo202000

Copy link
Copy Markdown
Author

Hi! This PR addresses the bounty issue and is ready for review.

Could you let me know the process for claiming the $30 (LT) bounty once merged? My Base wallet is configured and ready.

Thanks!

@leo202000
leo202000 force-pushed the bounty/1-fix-highmemory-self-dividing branch from 164375f to 9a91e75 Compare June 21, 2026 03:02
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