fix: Proxmox input: Changed VM ID from string to int, fixes #9611

[atetevoortwis]

Small fix to properly parse JSON data from Proxmox >=7.0 installs.

Required for all PRs:

• Updated associated README.md.
• Wrote appropriate unit tests.
• Pull request title or commits are in conventional commit format

resolves #9611

• Updated the VMStat schema to use a Number for ID.
• Changed getCurrentVMStatus and getVmConfig accordingly.
• Added case to testdata to check that both string representations and integers work in JSON.

[atetevoortwis]

@rare-magma, Finally found some time for this, continuing here...

[atetevoortwis]

@srebhan With regards to your comment (originially on #9820):

is this a bug that was there from the beginning or does it depend on the proxmox version? If the latter, you need to
unmarshal to the respective type, i.e. you try with an int first and if that fails, you use it as a string...

To be honest I'm not sure: I experienced this bug when upgrading to Proxmox 7.0.0, so I expect it to be related to a change on their API.
I've updated the unit test to both use a string (representation of an int) and int's in the test JSON and that seems to work...
Although I'm completely new to Go, so let me know if this is not the way :-)

[telegraf-tiger]

😄 This pull request doesn't change the Telegraf binary size

📦 Looks like new artifacts were built from this PR.

Expand this list to get them here! 🐯

Artifact URLs

DEB	RPM	TAR GZ	ZIP
amd64.deb	aarch64.rpm	darwin_amd64.tar.gz	windows_amd64.zip
arm64.deb	armel.rpm	freebsd_amd64.tar.gz	windows_i386.zip
armel.deb	armv6hl.rpm	freebsd_armv7.tar.gz
armhf.deb	i386.rpm	freebsd_i386.tar.gz
i386.deb	ppc64le.rpm	linux_amd64.tar.gz
mips.deb	s390x.rpm	linux_arm64.tar.gz
mipsel.deb	x86_64.rpm	linux_armel.tar.gz
ppc64el.deb		linux_armhf.tar.gz
s390x.deb		linux_i386.tar.gz
		linux_mips.tar.gz
		linux_mipsel.tar.gz
		linux_ppc64le.tar.gz
		linux_s390x.tar.gz
		static_linux_amd64.tar.gz

[srebhan]

@atetevoortwis the code looks good so far. Can you please check the documentation to make sure previous IDs were struct numerical and not "abc2347jddh" type of values?

[atetevoortwis]

@atetevoortwis the code looks good so far. Can you please check the documentation to make sure previous IDs were struct numerical and not "abc2347jddh" type of values?

As far as I know Proxmox VM ID's always have been integers: https://pve.proxmox.com/pve-docs/api-viewer/#/nodes/{node}/lxc/{vmid} .

[srebhan]

Looks good to me, assuming the backward compatibility is given as discussed. Thanks @atetevoortwis for your work!

[reimda]

Thanks @atetevoortwis