Skip to content

Conversation

@michelroegl-brunner
Copy link
Member

@michelroegl-brunner michelroegl-brunner commented Oct 17, 2025

Summary

This PR adds a new LXC settings modal for managing container configurations directly from the web interface.

New Features

✨ LXC Settings Modal

  • Complete LXC configuration management with a user-friendly interface
  • Real-time configuration editing with form validation
  • Sync from server functionality to pull current container settings
  • Save to server with proper confirmation and safety checks
  • Tabbed interface with Common Settings and Advanced Settings sections

🔧 Configuration Management

  • Basic Configuration: Architecture, cores, memory, hostname, OS type, etc.
  • Network Configuration: Interface settings, bridge, MAC address, IP configuration
  • Storage Settings: Root filesystem storage and size configuration
  • Feature Toggles: Keyctl, nesting, FUSE, and additional mount features
  • Advanced Settings: Raw configuration editing for power users

🛡️ Safety Features

  • Confirmation modal with container ID verification before saving
  • Change detection to only enable save when modifications are made
  • Error handling with clear user feedback
  • SSH connection validation before attempting operations

User Experience

  • Intuitive interface with contextual help icons
  • Real-time validation and form state management
  • Loading states and progress indicators
  • Success/error messaging for all operations

Technical Implementation

  • tRPC integration for type-safe API calls
  • Prisma database integration for configuration storage
  • SSH execution for server communication
  • React hooks for state management and side effects

Files Added

  • src/app/_components/LXCSettingsModal.tsx - Main modal component
  • Database schema updates for LXC configuration storage
  • API endpoints for configuration management

This modal provides a complete solution for managing LXC container configurations without needing direct server access.

- Fix Prisma database errors in LXC config sync (advanced and rootfs field issues)
- Remove double confirmation from LXC settings modal (keep confirmation modal, remove inline input)
- Fix dependency loop in status check useEffect
- Add LXC configuration management with proper validation
- Improve error handling and user experience
@michelroegl-brunner michelroegl-brunner requested a review from a team as a code owner October 17, 2025 09:36
@michelroegl-brunner michelroegl-brunner changed the title feat: improve LXC settings modal and fix database issues feat: add LXC settings modal for container configuration management Oct 17, 2025
@michelroegl-brunner michelroegl-brunner changed the title feat: add LXC settings modal for container configuration management feat: Implement LXC settings Oct 17, 2025
@michelroegl-brunner michelroegl-brunner merged commit 537d652 into main Oct 17, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant