Review and improve installation scripts robustness #14
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.
Complete rewrite of installation system with separation of concerns, comprehensive error handling, and platform-specific optimizations.
New Architecture
Library Modules (scripts/install/lib/)
common.sh: Core utilities, logging, error handling, rollbackplatform-detect.sh: Platform/OS/cloud/resource detectionpreflight.sh: Comprehensive pre-installation validationauth-setup.sh: Authentication configuration (OIDC/token/none)validation.sh: Post-installation testing and health checksPlatform-Specific Installers
install.sh: Main dispatcher with auto-detectioninstall-standalone.sh: Universal installer, works anywhereinstall-proxmox.sh: ProxMox LXC optimizationsinstall-ec2.sh: Cloud environment optimizationsKey Features
Robustness
Pre-flight Validation
Post-installation Validation
Flexibility
Configuration Templates (scripts/install/templates/)
install.conf.example: General template with all optionsproxmox-lxc.conf: ProxMox LXC with Tailscale OAuthec2-cloud.conf: Cloud environments with security notesdevelopment.conf: Development setup (no auth)Usage Examples
Integration
Updated root
install.shandct/install.shto use new modular system while maintaining backward compatibility with ProxMox build system.Documentation
Improvements Over Previous Scripts
Addresses: Improved robustness, efficiency, and support for ProxMox LXC and arbitrary cloud/VM scenarios.