[PR #397] [CLOSED] Task upid return #437

New issue

Closed

opened 2026-02-28 00:42:12 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 00:42:12 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Telmate/proxmox-api-go/pull/397
Author: @fireflg
Created: 1/20/2025
Status: ❌ Closed

Base: master ← Head: task_upid_return

📝 Commits (2)

53f63d1 Change return value from task WaitForCompletion
f326edf Add return value task upid

📊 Changes

1 file changed (+507 additions, -130 deletions)

View changed files

📝 proxmox/client.go (+507 -130)

📄 Description

Hi, in my opinion you need to add taskupid to the return values
For example, I use the vzdump method and want to check the logs(after task completed, (GetItemList)), but I lost the task UPID, so I can’t check the logs because I only have the task status.

_{🔄 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/397 **Author:** [@fireflg](https://github.com/fireflg) **Created:** 1/20/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `task_upid_return` --- ### 📝 Commits (2) - [`53f63d1`](https://github.com/Telmate/proxmox-api-go/commit/53f63d1c3369957edef86c4d48ff614e454770f4) Change return value from task WaitForCompletion - [`f326edf`](https://github.com/Telmate/proxmox-api-go/commit/f326edf3dc55263aaae4cf29cb21c1d64739364a) Add return value task upid ### 📊 Changes **1 file changed** (+507 additions, -130 deletions) <details> <summary>View changed files</summary> 📝 `proxmox/client.go` (+507 -130) </details> ### 📄 Description Hi, in my opinion you need to add taskupid to the return values For example, I use the vzdump method and want to check the logs(after task completed, (GetItemList)), but I lost the task UPID, so I can’t check the logs because I only have the task status. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>