[PR #65] [MERGED] Fix when the vcpus parameter is committed #198

New issue

Closed

opened 2026-02-28 00:41:04 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 00:41:04 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Telmate/proxmox-api-go/pull/65
Author: @V-Paranoiaque
Created: 12/6/2019
Status: ✅ Merged
Merged: 12/6/2019
Merged by: @ggongaware

Base: master ← Head: master

📝 Commits (2)

bb074b6 Fix var
6789d87 Fix when the parameter vcpus is omitted

📊 Changes

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

View changed files

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

📄 Description

The documentation says the parameter can be set to 0 but Proxmox throws an error. Instead, the delete parameter has to be used to remove the property.

_{🔄 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/65 **Author:** [@V-Paranoiaque](https://github.com/V-Paranoiaque) **Created:** 12/6/2019 **Status:** ✅ Merged **Merged:** 12/6/2019 **Merged by:** [@ggongaware](https://github.com/ggongaware) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`bb074b6`](https://github.com/Telmate/proxmox-api-go/commit/bb074b6422e99efaae722516c065ff16fbbfe26a) Fix var - [`6789d87`](https://github.com/Telmate/proxmox-api-go/commit/6789d876e08893b6e7e15e7a5abd69be17f65157) Fix when the parameter vcpus is omitted ### 📊 Changes **1 file changed** (+24 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `proxmox/config_qemu.go` (+24 -4) </details> ### 📄 Description The documentation says the parameter can be set to 0 but Proxmox throws an error. Instead, the delete parameter has to be used to remove the property. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>