[PR #201] [MERGED] fix: setting params with bool pointers #308

Closed
opened 2026-02-28 00:41:42 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Telmate/proxmox-api-go/pull/201
Author: @BenediktBertsch
Created: 10/3/2022
Status: Merged
Merged: 10/3/2022
Merged by: @mleone87

Base: masterHead: master


📝 Commits (3)

  • b34e59e fix: setting params with bool pointers
  • 248e6b0 fix: get the real value for the parameter
  • 454a406 simplify code and delete not needed util functions

📊 Changes

1 file changed (+16 additions, -4 deletions)

View changed files

📝 proxmox/config_qemu.go (+16 -4)

📄 Description

Fixes the current state for proxmox api for the createQemu JSON Sample.

Previous error message was:
error creating VM: 400 Parameter verification failed., error status: {"data":null,"errors":{"numa":"type check ('boolean') failed - got '<nil>'","kvm":"type check ('boolean') failed - got '<nil>'","onboot":"type check ('boolean') failed - got '<nil>'","tablet":"type check ('boolean') failed - got '<nil>'"}} (params: map[agent:0 args: boot: cores:2 cpu: description: hostpci0:host=0000:01:00.1,rombar=0 hotplug: ide2:local:iso/ubuntu-20.04.3-live-server-amd64.iso,media=cdrom kvm:<nil> machine: memory:2048 name:golang1.test.com net0:virtio=72:33:57:71:7C:6D numa:<nil> onboot:<nil> ostype:l26 sockets:1 startup: tablet:<nil> tags: virtio0:Storage10TB:80 vmid:103])


🔄 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/Telmate/proxmox-api-go/pull/201 **Author:** [@BenediktBertsch](https://github.com/BenediktBertsch) **Created:** 10/3/2022 **Status:** ✅ Merged **Merged:** 10/3/2022 **Merged by:** [@mleone87](https://github.com/mleone87) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (3) - [`b34e59e`](https://github.com/Telmate/proxmox-api-go/commit/b34e59e82f8f342865d7b4307adf1e57df6006b0) fix: setting params with bool pointers - [`248e6b0`](https://github.com/Telmate/proxmox-api-go/commit/248e6b0ab06e6e4dcf94f085505356f39a501aa8) fix: get the real value for the parameter - [`454a406`](https://github.com/Telmate/proxmox-api-go/commit/454a406349ec2912f2b2a975c35b7e4687e75088) simplify code and delete not needed util functions ### 📊 Changes **1 file changed** (+16 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `proxmox/config_qemu.go` (+16 -4) </details> ### 📄 Description Fixes the current state for proxmox api for the createQemu JSON Sample. Previous error message was: `error creating VM: 400 Parameter verification failed., error status: {"data":null,"errors":{"numa":"type check ('boolean') failed - got '<nil>'","kvm":"type check ('boolean') failed - got '<nil>'","onboot":"type check ('boolean') failed - got '<nil>'","tablet":"type check ('boolean') failed - got '<nil>'"}} (params: map[agent:0 args: boot: cores:2 cpu: description: hostpci0:host=0000:01:00.1,rombar=0 hotplug: ide2:local:iso/ubuntu-20.04.3-live-server-amd64.iso,media=cdrom kvm:<nil> machine: memory:2048 name:golang1.test.com net0:virtio=72:33:57:71:7C:6D numa:<nil> onboot:<nil> ostype:l26 sockets:1 startup: tablet:<nil> tags: virtio0:Storage10TB:80 vmid:103])` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 00:41:42 +03:00
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#308
No description provided.