Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ZFS to Podman. Now it works on ZFS! #2526

Merged
merged 5 commits into from
Feb 21, 2025

Conversation

jaminmc
Copy link
Contributor

@jaminmc jaminmc commented Feb 20, 2025

✍️ 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:

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

@jaminmc jaminmc requested a review from a team as a code owner February 20, 2025 18:45
@github-actions github-actions bot added update script A change that updates a script website A change to the website labels Feb 20, 2025
@jaminmc
Copy link
Contributor Author

jaminmc commented Feb 20, 2025

Ok, It is now tested. :)

@MickLesk MickLesk added feature and removed website A change to the website labels Feb 21, 2025
@michelroegl-brunner michelroegl-brunner merged commit 10c4672 into community-scripts:main Feb 21, 2025
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature update script A change that updates a script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants