-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[AI-5348] Add Proxmox recommended monitors #20860
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
Conversation
"created_at": "2025-07-25", | ||
"last_updated_at": "2025-07-25", | ||
"title": "Memory Usage High on Proxmox Node", | ||
"description": "Notify your team when the Memory usage is too high on a Proxmox Node.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @sarah-witt can you add more information to the Description, see the developer docs for more details.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated, thanks!
I forget, the host tag in this case is the actual proxmox node and not the host running the check right? Looks good to me if so. |
"name": "Memory Usage High on Proxmox VM {{host.host}} on Node {{host.proxmox_node}}", | ||
"type": "query alert", | ||
"query": "avg(last_5m):avg:proxmox.mem.used{proxmox_type:vm} by {host} / avg:proxmox.mem.total{proxmox_type:vm} by {host} * 100 > 90", | ||
"message": "{{#is_alert}}\\VM {{host.host}} on Node {{host.proxmox_node}} is reporting high memory usage (at or above 90%).\\n{{/is_alert}}\\n\\n{{#is_warning}}\\VM {{host.host}} on Node {{host.proxmox_node}} is reporting higher memory usage (at or above 80%).\\n{{/is_warning}}\\n\\n{{#is_recovery}}\\\\Memory usage for VM {{host.host}} on Node {{host.proxmox_node}} is back to normal.\\n{{/is_recovery}}", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does host.proxmox_node
work here? I could be misremembering, but I think this only works if you do a aggregation by proxmox_node tag.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh that's good to know.
Review from steveny91 is dismissed. Related teams and files:
- agent-integrations
- proxmox/assets/monitors/node_cpu_usage.json
- proxmox/assets/monitors/node_memory_usage.json
- proxmox/assets/monitors/vm_cpu_usage.json
- proxmox/assets/monitors/vm_memory_usage.json
Review from steveny91 is dismissed. Related teams and files:
- agent-integrations
- proxmox/assets/monitors/node_memory_usage.json
* add vm cpu usage * node monitor * Add memory usage * fix dates and paths * fix date * update descriptions * spacing * fix spacing and host.name * fix
What does this PR do?
Add Proxmox recommended monitors
Motivation
Review checklist (to be filled by reviewers)
qa/skip-qa
label if the PR doesn't need to be tested during QA.backport/<branch-name>
label to the PR and it will automatically open a backport PR once this one is merged