[PR #147] [MERGED] Added note regarding OVM as this as historically caused a lot of supp… #2415

Closed
opened 2025-11-20 05:29:44 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/147
Author: @newzealandpaul
Created: 11/8/2024
Status: Merged
Merged: 11/11/2024
Merged by: @BramSuurdje

Base: mainHead: ghp


📝 Commits (3)

  • 0567294 Added note regarding OVM as this as historically caused a lot of support requests
  • c505ad1 Update json/omv.json
  • fb13ab5 Change Link to community-scripts\discussion

📊 Changes

1 file changed (+7 additions, -2 deletions)

View changed files

📝 json/omv.json (+7 -2)

📄 Description

Description

Adds a note to the slightly controversial OpenMediaVault LXC that adding disks is difficult with a link to a previous discussion about how to add disks. Officially OMV does not support LXCs and at one point @tteck removed the OVM script.

Fixes #132

Type of change

Please check the relevant option(s):

  • Bug fix (non-breaking change that resolves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (a fix or feature that would cause existing functionality to change unexpectedly)
  • New script (a fully functional and thoroughly tested script or set of scripts.)
  • Documentation update required (this change requires an update to the documentation)

Prerequisites

The following efforts must be made for the PR to be considered. Please check when completed:

  • Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions)
  • Testing performed (I have tested my changes, ensuring everything works as expected)

🔄 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/147 **Author:** [@newzealandpaul](https://github.com/newzealandpaul) **Created:** 11/8/2024 **Status:** ✅ Merged **Merged:** 11/11/2024 **Merged by:** [@BramSuurdje](https://github.com/BramSuurdje) **Base:** `main` ← **Head:** `ghp` --- ### 📝 Commits (3) - [`0567294`](https://github.com/community-scripts/ProxmoxVE/commit/0567294e33233f604b8865bb4e4d30573e35407a) Added note regarding OVM as this as historically caused a lot of support requests - [`c505ad1`](https://github.com/community-scripts/ProxmoxVE/commit/c505ad18f942ddffbf5289326ac7e3eb6b0e4e23) Update json/omv.json - [`fb13ab5`](https://github.com/community-scripts/ProxmoxVE/commit/fb13ab5b32f22e7ecf7510ab2777724420c4b3ff) Change Link to community-scripts\discussion ### 📊 Changes **1 file changed** (+7 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `json/omv.json` (+7 -2) </details> ### 📄 Description ## Description Adds a note to the slightly controversial OpenMediaVault LXC that adding disks is difficult with a link to a previous discussion about how to add disks. Officially OMV does not support LXCs and at one point @tteck removed the OVM script. Fixes #132 ## Type of change Please check the relevant option(s): - [x] Bug fix (non-breaking change that resolves an issue) - [ ] New feature (non-breaking change that adds functionality) - [ ] Breaking change (a fix or feature that would cause existing functionality to change unexpectedly) - [ ] New script (a fully functional and thoroughly tested script or set of scripts.) - [ ] Documentation update required (this change requires an update to the documentation) ## Prerequisites The following efforts must be made for the PR to be considered. Please check when completed: - [x] Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions) - [ ] Testing performed (I have tested my changes, ensuring everything works as expected) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2025-11-20 05:29:44 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#2415