Skip to content

(WIP)- P2poolv2 memory/storage usage view - #35

Draft
R27-pixel wants to merge 6 commits into
p2poolv2:mainfrom
R27-pixel:p2poolv2-memory
Draft

R27-pixel wants to merge 6 commits into
p2poolv2:mainfrom
R27-pixel:p2poolv2-memory

Conversation

@R27-pixel

@R27-pixel R27-pixel commented Sep 23, 2026 •

Copy link
Copy Markdown
Member

This PR adds live RAM (RSS) process memory monitoring under the System tab in the PDM TUI for managed P2Poolv2 user-level systemd instances (p2poolv2@<instance>.service).

With this addition, P2Pool resource monitoring in PDM is structured into two clear domains:

  • Storage Tab: Monitors persistent disk usage (database directory location and stored size on disk from config-<instance>.toml).
  • System Tab: Monitors live process execution (systemd service status, process controls Start/Stop/Restart, and active RAM/RSS memory usage queried via systemd MemoryCurrent).

Verification & Testing

  • Unit tests added and passing across service, app state, and TUI components.
  • Tested and verified live against a running p2poolv2@signet instance in the TUI.
Screenshot 2026-10-01 031019 Screenshot 2026-09-24 014613

@R27-pixel
R27-pixel marked this pull request as draft September 23, 2026 20:30
@codecov

codecov Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 81.03113% with 195 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/p2poolv2_service.rs 87.29% 70 Missing ⚠️
src/app.rs 73.70% 66 Missing ⚠️
src/components/p2pool_status_view.rs 78.72% 30 Missing ⚠️
src/main.rs 65.88% 29 Missing ⚠️

📢 Thoughts on this report? Let us know!

…r managed P2Poolv2 user-level systemd instances (p2poolv2@<instance>.service)
@R27-pixel

R27-pixel commented Sep 30, 2026 •

Copy link
Copy Markdown
Member Author

Keeping this in draft for now. Remaining items before review:

  • Rebase on the parent PR once it's merged
  • Address code cleanup and refactoring
  • Improve test coverage / address Codecov misses

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