Skip to content

Refactor: Docker-VM (Multi-OS / Cloud-Init / Stabilization) #9047

Merged
CrazyWolf13 merged 6 commits intomainfrom
docker_deb13
Feb 5, 2026
Merged

Refactor: Docker-VM (Multi-OS / Cloud-Init / Stabilization) #9047
CrazyWolf13 merged 6 commits intomainfrom
docker_deb13

Conversation

@MickLesk
Copy link
Member

@MickLesk MickLesk commented Nov 11, 2025

✍️ Description

  • Multi-OS Support – Added Debian 12/13 + Ubuntu 22.04/24.04 selection via select_os()
  • Cloud-Init Integration – Optional Cloud-Init config; Ubuntu auto-enabled, Debian optional
  • Modular Architecture – External function files (vm-core.func, cloud-init.func, api.func) from git.community-scripts.org
  • Better Defaults – Q35 instead of i440fx; Host CPU instead of KVM64; 4GB RAM instead of 2GB
  • Docker via get.docker.com – Replaced manual repo setup with official one-liner + first-boot fallback service
  • Image Caching – Downloads cached in /var/lib/vz/template/cache/ for reuse
  • Simplified Disk Resize – Uses qemu-img resize instead of virt-resize
  • Auto-Login for nocloud – Serial console auto-login configured when Cloud-Init disabled
  • IP Detection – Automatic IP retrieval via QEMU guest agent in final summary
  • Code Cleanup – Removed inline ASCII art/colors; added section separators; fixed function naming (exit_script)

🔗 Related PR / Issue

Fixes: #9040

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

Updated the script to use Debian 13 Qcow2 Disk Image and removed redundant installation of libguestfs-tools.
@github-actions github-actions bot added update script A change that updates a script vm labels Nov 11, 2025
@Yash-xoxo
Copy link

switch to arch?

@MickLesk
Copy link
Member Author

MickLesk commented Dec 8, 2025

No

@Yash-xoxo
Copy link

suffer

@AnthonyMichaelTDM
Copy link

I see that this is a draft until tested, I didn't test the script directly, but I did migrate my docker vm (created a long time ago with the script currently in main) from Debian 12 to Debian 13 and haven't had issues.

@dafi87
Copy link

dafi87 commented Jan 16, 2026

I would test this but I'm on PVE 9.1.2, which isn't supported by your updated script yet.

@MickLesk
Copy link
Member Author

Sure in Dev repo

@dafi87
Copy link

dafi87 commented Jan 16, 2026

I thought the point was to test what's in this pull request. But thanks anyway.

@MickLesk MickLesk changed the title Docker-VM: upgrade to Debian 13 Refactor: Docker-VM (Multi-OS / Cloud-Init / Stabilization) Feb 4, 2026
@MickLesk MickLesk marked this pull request as ready for review February 4, 2026 08:38
@MickLesk MickLesk requested a review from a team as a code owner February 4, 2026 08:38
@CrazyWolf13 CrazyWolf13 merged commit 129f2f6 into main Feb 5, 2026
1 check passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 9, 2026
@MickLesk MickLesk deleted the docker_deb13 branch February 17, 2026 08:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

feature update script A change that updates a script vm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docker VM does not install Docker

6 participants