Skip to content

sn-manager updates - #143

Merged
mateeullahmalik merged 1 commit into
masterfrom
sn-manager-updates
Aug 27, 2025
Merged

mateeullahmalik merged 1 commit into
masterfrom
sn-manager-updates

Conversation

@mateeullahmalik

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements comprehensive updates to the sn-manager system, introducing self-updating capabilities, improved process monitoring, and enhanced command structure. The changes enable sn-manager to update both itself and SuperNode from a single release tarball, add robust process supervision with automatic restart capabilities, and provide granular control over SuperNode lifecycle.

Key changes:

  • Self-updating capability for sn-manager with combined tarball downloads
  • Enhanced process monitoring and supervision with automatic crash recovery
  • New command structure with dedicated SuperNode control commands
  • Improved version comparison using full semantic versioning rules

Reviewed Changes

Copilot reviewed 23 out of 24 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
sn-manager/internal/version/manager.go Implements streaming binary installation and improved error handling with cleanup
sn-manager/internal/utils/version.go Adds comprehensive semantic versioning support with pre-release handling
sn-manager/internal/updater/updater_test.go Removes entire test suite (1200+ lines)
sn-manager/internal/updater/updater.go Major refactor for combined self-update and SuperNode update functionality
sn-manager/internal/manager/manager.go Adds process monitoring, supervision, and automatic restart capabilities
sn-manager/internal/github/client.go Enhances GitHub client with tarball support and improved error handling
sn-manager/internal/config/config.go Adds configuration constants and defaults
sn-manager/go.mod Removes test dependencies
sn-manager/cmd/*.go Multiple command files implementing new SuperNode control structure
sn-manager/README.md Comprehensive documentation updates

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread sn-manager/internal/utils/version.go
Comment thread sn-manager/internal/updater/updater.go
Comment thread sn-manager/internal/updater/updater.go
Comment thread sn-manager/internal/manager/manager.go
Comment thread sn-manager/cmd/supernode_stop.go
@mateeullahmalik
mateeullahmalik merged commit cf2d52e into master Aug 27, 2025
7 checks passed
@mateeullahmalik
mateeullahmalik deleted the sn-manager-updates branch September 5, 2025 11:53
@akobrin1 akobrin1 mentioned this pull request Jun 10, 2026
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.

3 participants