[PR #11020] [MERGED] core: allow empty tags & improve template search #9180

Closed
opened 2026-02-26 16:35:25 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/11020
Author: @MickLesk
Created: 1/21/2026
Status: Merged
Merged: 1/21/2026
Merged by: @tremor021

Base: mainHead: fix_core_lock_download


📝 Commits (2)

  • 79f8ba9 Allow empty Tags
  • 0371812 Optimize template search in LXC container creation

📊 Changes

1 file changed (+29 additions, -20 deletions)

View changed files

📝 misc/build.func (+29 -20)

📄 Description

✍️ Description

1.) Allow empty Tags (if all tags removed, we dont add "-tags" into pct create command
Fixes #11010

2.) Local templates are now checked and used before attempting to update the online catalog, reducing unnecessary network calls and improving speed. Online catalog updates are performed only if no local template is found, with a timeout and error message (if needed) to prevent long hangs.
Fixes #11017

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.

🔄 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/11020 **Author:** [@MickLesk](https://github.com/MickLesk) **Created:** 1/21/2026 **Status:** ✅ Merged **Merged:** 1/21/2026 **Merged by:** [@tremor021](https://github.com/tremor021) **Base:** `main` ← **Head:** `fix_core_lock_download` --- ### 📝 Commits (2) - [`79f8ba9`](https://github.com/community-scripts/ProxmoxVE/commit/79f8ba9fa19c5a178df0cbb74a25e63aa5671309) Allow empty Tags - [`0371812`](https://github.com/community-scripts/ProxmoxVE/commit/037181227119cb3d78cb076c6c17bcd4aff996ac) Optimize template search in LXC container creation ### 📊 Changes **1 file changed** (+29 additions, -20 deletions) <details> <summary>View changed files</summary> 📝 `misc/build.func` (+29 -20) </details> ### 📄 Description <!--🛑 New scripts must be submitted to [ProxmoxVED](https://github.com/community-scripts/ProxmoxVED) for testing. PRs without prior testing will be closed. --> ## ✍️ Description 1.) Allow empty Tags (if all tags removed, we dont add "-tags" into pct create command Fixes #11010 2.) Local templates are now checked and used before attempting to update the online catalog, reducing unnecessary network calls and improving speed. Online catalog updates are performed only if no local template is found, with a timeout and error message (if needed) to prevent long hangs. Fixes #11017 ## ✅ Prerequisites (**X** in brackets) - [x] **Self-review completed** – Code follows project standards. - [x] **Tested thoroughly** – Changes work as expected. - [x] **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. - [x] 🔧 **Refactoring / Code Cleanup** – Improves readability or maintainability without changing functionality. - [ ] 📝 **Documentation update** – Changes to `README`, `AppName.md`, `CONTRIBUTING.md`, or other docs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 16:35:25 +03:00
Sign in to join this conversation.
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#9180
No description provided.