[GH-ISSUE #82] Interface conversion error in NewConfigQemuFromApi #13

Closed
opened 2026-02-28 00:40:11 +03:00 by kerem · 2 comments
Owner

Originally created by @johnpc35 on GitHub (Jul 15, 2020).
Original GitHub issue: https://github.com/Telmate/proxmox-api-go/issues/82

One of the changes in #78 is causing an error when NewConfigQemuFromApi is called:

panic: interface conversion: interface {} is float64, not bool
goroutine 52 [running]:
github.com/Telmate/proxmox-api-go/proxmox.NewConfigQemuFromApi(0xc000443528, 0xc0005f6640, 0x0, 0x0, 0x0)
	/home/johnpc/go/src/github.com/Telmate/proxmox-api-go/proxmox/config_qemu.go:457 +0x3235

This is used by the getConfig command as well as the Terrform Proxmox Provider when cloning a template or VM.

Additionally, this flag needs to be defaulted to true where possible, otherwise it keeps the default value of false which is then passed to Proxmox when creating or updating a VM.

Originally created by @johnpc35 on GitHub (Jul 15, 2020). Original GitHub issue: https://github.com/Telmate/proxmox-api-go/issues/82 One of the changes in #78 is causing an error when `NewConfigQemuFromApi` is called: ``` panic: interface conversion: interface {} is float64, not bool goroutine 52 [running]: github.com/Telmate/proxmox-api-go/proxmox.NewConfigQemuFromApi(0xc000443528, 0xc0005f6640, 0x0, 0x0, 0x0) /home/johnpc/go/src/github.com/Telmate/proxmox-api-go/proxmox/config_qemu.go:457 +0x3235 ``` This is used by the `getConfig` command as well as the Terrform Proxmox Provider when cloning a template or VM. Additionally, this flag needs to be defaulted to `true` where possible, otherwise it keeps the default value of `false` which is then passed to Proxmox when creating or updating a VM.
kerem closed this issue 2026-02-28 00:40:11 +03:00
Author
Owner

@romantomjak commented on GitHub (Jul 15, 2020):

Awesome, thanks for spotting (and fixing) the problem @johnpc35! 🚀

<!-- gh-comment-id:658939526 --> @romantomjak commented on GitHub (Jul 15, 2020): Awesome, thanks for spotting (and fixing) the problem @johnpc35! 🚀
Author
Owner

@johnpc35 commented on GitHub (Jul 15, 2020):

Tested from the Terraform Provider and it seems to be working fine now.

<!-- gh-comment-id:658954003 --> @johnpc35 commented on GitHub (Jul 15, 2020): Tested from the Terraform Provider and it seems to be working fine now.
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/proxmox-api-go#13
No description provided.