[PR #50] [CLOSED] NEW: Add Pulse monitoring dashboard #843

New Issue

giteasync · 2025-09-09T19:48:06-05:00

giteasync commented

2025-09-09 19:48:06 -05:00

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVED/pull/50
Author: @rcourtman
Created: 3/14/2025
Status: ❌ Closed

Base: main ← Head: pulse-cleanup

📝 Commits (3)

3fee8ae NEW: Add Pulse monitoring dashboard
79f543c FIX: Correct script formatting to follow community standards
40242fa FIX: Set TERM environment variable if not set

📊 Changes

3 files changed (+495 additions, -0 deletions)

View changed files

➕ ct/pulse.sh (+281 -0)
➕ install/pulse-install.sh (+170 -0)
➕ json/pulse.json (+44 -0)

📄 Description

🛑 New scripts must first be submitted to ProxmoxVED for testing.
PRs for new scripts that skip this process will be closed.

✍️ Description

This PR adds the Pulse monitoring dashboard, a modern and lightweight monitoring solution designed specifically for Proxmox Virtual Environment. The implementation includes three essential files:

ct/pulse.sh: Container script for managing the Pulse container
install/pulse-install.sh: Installation script for setting up Pulse
json/pulse.json: Configuration file with metadata and requirements

The scripts have been optimized with minimal comments while maintaining full functionality and following project standards.

Link: #3075 (Previous PR on wrong repository)

✅ Prerequisites (X in brackets)

Self-review completed – Code follows project standards.
Tested thoroughly – Changes work as expected.
No breaking changes – Existing functionality remains intact.
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.

🔍 Code & Security Review (X in brackets)

Follows Code_Audit.md & CONTRIBUTING.md guidelines
Uses correct script structure (AppName.sh, AppName-install.sh, AppName.json)
No hardcoded credentials

📋 Additional Information (optional)

The Pulse dashboard provides real-time monitoring capabilities for Proxmox VE environments, including:

Resource usage monitoring
Container and VM status
Network performance metrics
Storage utilization
System health indicators

The implementation uses mock data by default for demonstration purposes, with clear instructions for users to configure their own Proxmox node connections.

_{🔄 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/ProxmoxVED/pull/50 **Author:** [@rcourtman](https://github.com/rcourtman) **Created:** 3/14/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `pulse-cleanup` --- ### 📝 Commits (3) - [`3fee8ae`](https://github.com/community-scripts/ProxmoxVED/commit/3fee8aecff864286fdd150f3b688dbe69a50d6da) NEW: Add Pulse monitoring dashboard - [`79f543c`](https://github.com/community-scripts/ProxmoxVED/commit/79f543c5014713df492007f4fa7d92e956252c28) FIX: Correct script formatting to follow community standards - [`40242fa`](https://github.com/community-scripts/ProxmoxVED/commit/40242faf6ba95a292801f9257b896dfea6258d33) FIX: Set TERM environment variable if not set ### 📊 Changes **3 files changed** (+495 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `ct/pulse.sh` (+281 -0) ➕ `install/pulse-install.sh` (+170 -0) ➕ `json/pulse.json` (+44 -0) </details> ### 📄 Description 🛑 **New scripts must first be submitted to [ProxmoxVED](https://github.com/community-scripts/ProxmoxVED) for testing.** PRs for new scripts that skip this process will be closed. --- ## ✍️ Description This PR adds the Pulse monitoring dashboard, a modern and lightweight monitoring solution designed specifically for Proxmox Virtual Environment. The implementation includes three essential files: - `ct/pulse.sh`: Container script for managing the Pulse container - `install/pulse-install.sh`: Installation script for setting up Pulse - `json/pulse.json`: Configuration file with metadata and requirements The scripts have been optimized with minimal comments while maintaining full functionality and following project standards. ## 🔗 Related PR / Issue Link: #3075 (Previous PR on wrong repository) ## ✅ Prerequisites (**X** in brackets) - [X] **Self-review completed** – Code follows project standards. - [X] **Tested thoroughly** – Changes work as expected. - [X] **No breaking changes** – Existing functionality remains intact. - [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. - [X] 🆕 **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. --- ## 🔍 Code & Security Review (**X** in brackets) - [x] **Follows `Code_Audit.md` & `CONTRIBUTING.md` guidelines** - [X] **Uses correct script structure (`AppName.sh`, `AppName-install.sh`, `AppName.json`)** - [X] **No hardcoded credentials** ## 📋 Additional Information (optional) The Pulse dashboard provides real-time monitoring capabilities for Proxmox VE environments, including: - Resource usage monitoring - Container and VM status - Network performance metrics - Storage utilization - System health indicators The implementation uses mock data by default for demonstration purposes, with clear instructions for users to configure their own Proxmox node connections. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

giteasync added the

pull-request

label 2025-09-09 19:48:06 -05:00

giteasync closed this issue

2025-09-09 19:48:06 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: community-scripts/ProxmoxVED#843