[PR #527] [MERGED] Zabbix: Use Agent2 as Default | Update Script added | some other Improvements #2762

Closed
opened 2026-02-26 13:31:17 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/527
Author: @MickLesk
Created: 11/26/2024
Status: Merged
Merged: 11/27/2024
Merged by: @MickLesk

Base: mainHead: zabbix_patch


📝 Commits (5)

  • 507bbd7 Improve Zabbix Update / Agent2
  • 4b61287 Add Updateble / Documentation / Info for JSON
  • c502df2 Zabbix-Install: Switch to latest version & use zabbix-agent2
  • 9926eb5 Update json/zabbix.json
  • 6bb5c12 Update zabbix.sh

📊 Changes

3 files changed (+45 additions, -17 deletions)

View changed files

📝 ct/zabbix.sh (+24 -3)
📝 install/zabbix-install.sh (+12 -9)
📝 json/zabbix.json (+9 -5)

📄 Description

Description

  • Updating the code base
  • Script flow adapted to standard
  • Switch to Zabbix-Agent2 (for LXC's)
  • Update script with backup data
  • some other little improvements

Breaking Change, since the script is now running with a different agent, I cannot and do not want to anchor this in the CT.sh script, it causes too many problems. So if you want to use Agent2, you have to install it yourself.

Fixes #174

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

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)
  • Documentation updated (I have updated any relevant documentation)

🔄 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/527 **Author:** [@MickLesk](https://github.com/MickLesk) **Created:** 11/26/2024 **Status:** ✅ Merged **Merged:** 11/27/2024 **Merged by:** [@MickLesk](https://github.com/MickLesk) **Base:** `main` ← **Head:** `zabbix_patch` --- ### 📝 Commits (5) - [`507bbd7`](https://github.com/community-scripts/ProxmoxVE/commit/507bbd72ea7204b3f509244c94d53715a573bdb9) Improve Zabbix Update / Agent2 - [`4b61287`](https://github.com/community-scripts/ProxmoxVE/commit/4b61287449f0a3fb91fc7b01ade1a03b77acb328) Add Updateble / Documentation / Info for JSON - [`c502df2`](https://github.com/community-scripts/ProxmoxVE/commit/c502df21601a7a1d33784fb71c119506ca691fcd) Zabbix-Install: Switch to latest version & use zabbix-agent2 - [`9926eb5`](https://github.com/community-scripts/ProxmoxVE/commit/9926eb51a25a2b484b2a3dd2a511e4ca65adf8d2) Update json/zabbix.json - [`6bb5c12`](https://github.com/community-scripts/ProxmoxVE/commit/6bb5c12a814594e151873a8372fd75cd2e1d618b) Update zabbix.sh ### 📊 Changes **3 files changed** (+45 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `ct/zabbix.sh` (+24 -3) 📝 `install/zabbix-install.sh` (+12 -9) 📝 `json/zabbix.json` (+9 -5) </details> ### 📄 Description ## Description - Updating the code base - Script flow adapted to standard - Switch to Zabbix-Agent2 (for LXC's) - Update script with backup data - some other little improvements Breaking Change, since the script is now running with a different agent, I cannot and do not want to anchor this in the CT.sh script, it causes too many problems. So if you want to use Agent2, you have to install it yourself. Fixes #174 ## Type of change Please check the relevant option(s): - [x] Bug fix (non-breaking change that resolves an issue) - [x] New feature (non-breaking change that adds functionality) - [x] 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.) ## 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) - [x] Testing performed (I have tested my changes, ensuring everything works as expected) - [x] Documentation updated (I have updated any relevant documentation) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 13:31:17 +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#2762
No description provided.