Skip to content

implement supernode healthcheck#93

Closed
j-rafique wants to merge 2 commits intomasterfrom
supernodeHealthCheck
Closed

implement supernode healthcheck#93
j-rafique wants to merge 2 commits intomasterfrom
supernodeHealthCheck

Conversation

@j-rafique
Copy link
Contributor

No description provided.

* Update: download-cascade, p2p (#92)

* feat: add GetSupernodeStatus method to retrieve supernode status and update dependencies

* feat: enhance supernode functionality and improve installation script

- Added optional parameters for Lumera binary path and installation mode in Makefile.
- Refactored GetSupernodeStatus to return a pointer type for better handling.
- Improved error handling and logging in health check and download processes.
- Implemented concurrent download attempts from supernodes for efficiency.
- Updated installation script to verify binary installation and handle different modes.
- Enhanced tests to validate supernode status and downloaded files.

* feat: Implement SupernodeService for system status reporting

- Added gRPC service SupernodeService with GetStatus method to provide system status.
- Created StatusRequest and StatusResponse messages to encapsulate request and response data.
- Integrated CPU and memory metrics retrieval using gopsutil library in SupernodeStatusService.
- Removed HealthCheck RPC from CascadeService and replaced it with a centralized status service.
- Updated CascadeAdapter to utilize the new SupernodeService for status checks.
- Implemented unit tests for SupernodeServer and SupernodeStatusService to ensure correct functionality.
- Refactored existing health check logic into common service for better reusability.

* refactor: Update SupernodeStatusresponse

---------

Co-authored-by: Matee ullah Malik <46045452+mateeullahmalik@users.noreply.github.com>
@mateeullahmalik mateeullahmalik deleted the supernodeHealthCheck branch July 15, 2025 13:20
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.

2 participants