Skip to content
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.

Sign up for GitHub

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

feat(inputs.proxmox): Allow to add VM-id and status as tag #16066

Merged
merged 11 commits into from
Jan 16, 2025
Merged

feat(inputs.proxmox): Allow to add VM-id and status as tag #16066

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
03c98ce
Update proxmox.go
superdiego98 Oct 21, 2024
505bac0
Update proxmox.go
superdiego98 Oct 21, 2024
ccdca07
restore default
superdiego98 Oct 21, 2024
1deab12
Add tag vm_id
superdiego98 Oct 21, 2024
efc0c60
add vm_id tag
superdiego98 Oct 21, 2024
fc8c473
Add vm_id tag in readme and test
superdiego98 Oct 22, 2024
19905f0
Correct proxmox_test.go
superdiego98 Oct 22, 2024
9a12fb0
Update structs.go
superdiego98 Nov 10, 2024
2f97b39
Cleanup code a bit
srebhan Jan 15, 2025
effd5ad
Conditionally add VM-ID and VM-status as a tag
srebhan Jan 15, 2025
1af6fe5
Check parameters
srebhan Jan 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Correct proxmox_test.go
  • Loading branch information
@superdiego98 @srebhan
superdiego98 authored and srebhan committed Jan 15, 2025
commit 19905f0f276020d2fbdd7a1f1ae7ca271a1dbfbb
1 change: 1 addition & 0 deletions plugins/inputs/proxmox/proxmox_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@ func TestGatherLxcData(t *testing.T) {
"vm_name": "container1",
"vm_fqdn": "container1.test.example.com",
"vm_type": "lxc",
"vm_id": "111",
}
acc.AssertContainsTaggedFields(t, "proxmox", testFields, testTags)
}
Expand Down