Skip to content

feat: simplify local flow#1548

Draft
karlem wants to merge 15 commits intomainfrom
storage-node-fix
Draft

feat: simplify local flow#1548
karlem wants to merge 15 commits intomainfrom
storage-node-fix

Conversation

@karlem
Copy link
Contributor

@karlem karlem commented Mar 17, 2026

No description provided.

@karlem karlem force-pushed the storage-node-fix branch from 34636d6 to 0c4f208 Compare March 19, 2026 14:15
karlem and others added 11 commits March 19, 2026 20:37
- Added `--compile` option to `update-binaries` command for selecting between local and remote builds.
- Introduced local build functionality in `health.sh` for cross-compilation from macOS to Linux.
- Updated configuration file to include a path for local IPC repository.
- Enhanced documentation in the script for clarity on new options and usage examples.
- Enhanced the cross-compilation logic in `health.sh` to prefer `cargo-zigbuild` over `cross`, providing clearer installation instructions.
- Updated documentation in `ipc-subnet-manager.sh` to reflect the new requirements for macOS to Linux builds.
- Improved error handling and user guidance for cross-compilation failures.
- Replaced direct SSH commands with `exec_on_host` for creating remote directories, improving code clarity and maintainability.
- Updated binary transfer functions to use `copy_to_host`, enhancing consistency in deployment logic.
- Removed redundant variable assignments for SSH user and IP, simplifying the function's parameters.
- Updated `Cargo.toml` to disable default features for `ethers` and `reqwest`, adding `rustls` support to avoid OpenSSL issues during cross-compilation.
- Introduced a new `Cross.toml` file for cross-compilation configuration, including OpenSSL header installation.
- Enhanced `ipc-subnet-config-local.yml` and `ipc-subnet-config.yml` with new storage service configurations and updated subnet parameters.
- Updated `ipc-subnet-manager.sh` to include new commands for managing storage services and improved options for binary updates.
- Changed subnet ID in `ipc-subnet-config.yml` to a new value.
- Disabled deployment and activation of the subnet in initialization settings for a more controlled setup.
- Updated the `start_storage_services` function to use `exec_on_host_simple` for executing commands on remote hosts, improving code consistency and readability.
- Adjusted multiple command executions related to storage configuration and service startup to utilize the simplified execution method.
…tions

- Added new commands for storage operations including `cp`, `ls`, `cat`, `mv`, and `stat`, allowing users to manage files and objects in storage.
- Introduced a `client` module for HTTP interactions with the storage gateway API, facilitating blob uploads and downloads.
- Enhanced `StorageConfig` to support dynamic gateway URL retrieval from various sources.
- Implemented path parsing for storage URIs, improving the handling of storage paths.
- Updated `Cargo.toml` to include new dependencies for enhanced functionality.
- Added comprehensive error handling and user feedback for storage operations.
- Updated `ipc-subnet-config.yml` to set `deploy_subnet` and `activate_subnet` to true, allowing for automatic deployment and activation of subnet and gateway contracts during initialization.
- Added preflight checks in `health.sh` to ensure necessary dependencies and binaries are available before starting storage services, improving robustness and user guidance for setup.
- Changed subnet ID and updated parent registry and gateway addresses in `ipc-subnet-config.yml` for improved network configuration.
- Enhanced `exec.sh` to check process status more reliably by capturing output and trimming whitespace.
- Improved `health.sh` to ensure proper cleanup of stale RocksDB locks and added fallback mechanisms for stopping processes, enhancing robustness during node shutdown.
- Added validation for extracted subnet ID against on-chain subnet list to ensure consistency and correctness in subnet management.
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