starred/ProxmoxVE

Fork 0

mirror of https://github.com/community-scripts/ProxmoxVE.git synced 2026-04-26 01:36:05 +03:00

[PR #2526] [MERGED] Add ZFS to Podman. Now it works on ZFS! #3811

New issue

Closed

opened 2026-02-26 13:34:44 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 13:34:44 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/2526
Author: @jaminmc
Created: 2/20/2025
Status: ✅ Merged
Merged: 2/21/2025
Merged by: @michelroegl-brunner

Base: main ← Head: podman

📝 Commits (5)

92d7abd Allow Podman to work on ZFS containers
cc58166 Fix ZFS for Podman-Create missing folder.
4edbdaa Added option to install Portainer or Portainer agent in Podman
bd0f27d Fix source for Podman Homeassistant so Portainer and other containers can be installed.
da20f88 fix Podman not creating storage/volumes, until one exists.

📊 Changes

4 files changed (+127 additions, -5 deletions)

View changed files

📝 install/podman-homeassistant-install.sh (+61 -0)
📝 install/podman-install.sh (+61 -0)
📝 json/podman-homeassistant.json (+4 -4)
📝 json/podman.json (+1 -1)

📄 Description

✍️ Description

Added a check in the Install Script for ZFS on the root Partition of the Container, and then added the Overlay FS to it so that it works on a Container with ZFS.

Added Portainer option, Just like the Docker install script.

Link: #

✅ Prerequisites

Before this PR can be reviewed, the following must be completed:

Self-review performed – Code follows established patterns and conventions.
Testing performed – Changes have been thoroughly tested and verified.

🛠️ Type of Change

Select all that apply:

🐞 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.

📋 Additional Information (optional)

I used the information from https://forum.proxmox.com/threads/podman-on-lxc-with-zfs-backed-volume-and-overlay.138722/ to make this fix.

I renamed my main branch, and it cloed my pull request #2504, So here is my new one.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/community-scripts/ProxmoxVE/pull/2526 **Author:** [@jaminmc](https://github.com/jaminmc) **Created:** 2/20/2025 **Status:** ✅ Merged **Merged:** 2/21/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `main` ← **Head:** `podman` --- ### 📝 Commits (5) - [`92d7abd`](https://github.com/community-scripts/ProxmoxVE/commit/92d7abdd39fad19b3e2c774dc4ef581168598d60) Allow Podman to work on ZFS containers - [`cc58166`](https://github.com/community-scripts/ProxmoxVE/commit/cc581667e6aa97b9e8ae52dc6f4e699f973fa8e9) Fix ZFS for Podman-Create missing folder. - [`4edbdaa`](https://github.com/community-scripts/ProxmoxVE/commit/4edbdaa92ae849b5ebcfe501870730605f75705c) Added option to install Portainer or Portainer agent in Podman - [`bd0f27d`](https://github.com/community-scripts/ProxmoxVE/commit/bd0f27d2047273d171ca2bd1883ea14440834d7f) Fix source for Podman Homeassistant so Portainer and other containers can be installed. - [`da20f88`](https://github.com/community-scripts/ProxmoxVE/commit/da20f880b968adbe4793dc1fd1e094037ebc22c9) fix Podman not creating storage/volumes, until one exists. ### 📊 Changes **4 files changed** (+127 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `install/podman-homeassistant-install.sh` (+61 -0) 📝 `install/podman-install.sh` (+61 -0) 📝 `json/podman-homeassistant.json` (+4 -4) 📝 `json/podman.json` (+1 -1) </details> ### 📄 Description ## ✍️ Description  Added a check in the Install Script for ZFS on the root Partition of the Container, and then added the Overlay FS to it so that it works on a Container with ZFS. Added Portainer option, Just like the Docker install script. ## 🔗 Related PR / Discussion / Issue Link: # ## ✅ Prerequisites Before this PR can be reviewed, the following must be completed: - [X] **Self-review performed** – Code follows established patterns and conventions. - [X] **Testing performed** – Changes have been thoroughly tested and verified. ## 🛠️ Type of Change Select all that apply: - [X] 🐞 **Bug fix** – Resolves an issue without breaking functionality. - [X] ✨ **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. ## 📋 Additional Information (optional)  I used the information from https://forum.proxmox.com/threads/podman-on-lxc-with-zfs-backed-volume-and-overlay.138722/ to make this fix. I renamed my main branch, and it cloed my pull request #2504, So here is my new one. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 13:34:44 +03:00

closed this issue
added the
pull-request
label

kerem referenced this issue

2026-02-26 14:31:18 +03:00

[PR #3811] [MERGED] [core] update .app files #4542

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ProxmoxVE#3811

No description provided.

Rows
Columns