Merged
Conversation
…update dependencies
- 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.
mateeullahmalik
force-pushed
the
update/downloads-p2p
branch
from
effb131 to
36acc70
Compare
j-rafique
approved these changes
Jul 3, 2025
mateeullahmalik
added a commit
that referenced
this pull request
Jul 7, 2025
* 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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.