Remove automatic deployment prompt from setup script
Changes: - scripts/setup-homelab.sh: Remove interactive deployment prompt - Users must now run deploy script manually - Simplifies both scripts (no sudo workarounds needed) - Clearer two-step process: setup then deploy - Documentation updates: - README.md: Updated step 3-4 with manual deployment - docs/getting-started.md: Removed step 6 (log out), clarified steps - docs/manual-setup.md: Added sudo to deploy command - docs/troubleshooting/COMMON-ISSUES.md: Added sudo to all deploy commands Rationale: - Automatic deployment via 'su -' cannot work with sudo requirement - Manual two-step process is clearer and more reliable - Setup focuses on configuration, deploy focuses on services
This commit is contained in:
@@ -587,43 +587,14 @@ echo ""
|
||||
log_info "Setup complete!"
|
||||
echo ""
|
||||
|
||||
# Prompt to run deployment script
|
||||
if [ "${NVIDIA_REBOOT_NEEDED:-false}" != true ]; then
|
||||
echo ""
|
||||
read -p "Would you like to run the deployment script now? [Y/n]: " -n 1 -r RUN_DEPLOY
|
||||
echo ""
|
||||
|
||||
# Default to yes if empty
|
||||
RUN_DEPLOY=${RUN_DEPLOY:-Y}
|
||||
|
||||
if [[ $RUN_DEPLOY =~ ^[Yy]$ ]]; then
|
||||
log_info "Starting deployment script..."
|
||||
echo ""
|
||||
|
||||
# Check if user needs to log out first for Docker group
|
||||
if ! groups "$ACTUAL_USER" | grep -q docker; then
|
||||
log_warning "You need to log out and back in for Docker group permissions."
|
||||
log_info "Run this command after logging back in:"
|
||||
echo ""
|
||||
echo " cd ~/AI-Homelab && ./scripts/deploy-homelab.sh"
|
||||
echo ""
|
||||
else
|
||||
# Run deployment script as the actual user with sudo
|
||||
cd "$(dirname "$0")/.." || exit 1
|
||||
su - "$ACTUAL_USER" -c "cd $PWD && sudo ./scripts/deploy-homelab.sh"
|
||||
fi
|
||||
else
|
||||
log_info "Deployment skipped. Run it manually when ready:"
|
||||
echo ""
|
||||
echo " cd ~/AI-Homelab"
|
||||
echo " ./scripts/deploy-homelab.sh"
|
||||
echo ""
|
||||
fi
|
||||
else
|
||||
# Instructions for deployment
|
||||
if [ "${NVIDIA_REBOOT_NEEDED:-false}" = true ]; then
|
||||
log_info "Please reboot your system for NVIDIA drivers, then run:"
|
||||
echo ""
|
||||
echo " cd ~/AI-Homelab"
|
||||
echo " ./scripts/deploy-homelab.sh"
|
||||
echo ""
|
||||
else
|
||||
log_info "Next step - deploy your homelab services:"
|
||||
fi
|
||||
echo ""
|
||||
echo " cd ~/AI-Homelab"
|
||||
echo " sudo ./scripts/deploy-homelab.sh"
|
||||
echo ""
|
||||
|
||||
|
||||
Reference in New Issue
Block a user