[PR #70] [MERGED] adding noop error and ignoring task from addtag in cloudinit #119

Closed
opened 2026-03-03 15:30:17 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/luthermonson/go-proxmox/pull/70
Author: @luthermonson
Created: 8/9/2023
Status: Merged
Merged: 8/9/2023
Merged by: @luthermonson

Base: mainHead: fix-addtag


📝 Commits (1)

  • 39430a8 adding noop error and ignoring task from addtag in cloudinit

📊 Changes

2 files changed (+14 additions, -11 deletions)

View changed files

📝 proxmox.go (+6 -0)
📝 virtual_machine.go (+8 -11)

📄 Description

regrets were have returning nil nil, adding a noop error type in case this happens again and decided to ultimately ignore the returned task in cloud init func since there wasn't much benefit to waiting for it to get done if we do actually call it

fixes #68


🔄 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/luthermonson/go-proxmox/pull/70 **Author:** [@luthermonson](https://github.com/luthermonson) **Created:** 8/9/2023 **Status:** ✅ Merged **Merged:** 8/9/2023 **Merged by:** [@luthermonson](https://github.com/luthermonson) **Base:** `main` ← **Head:** `fix-addtag` --- ### 📝 Commits (1) - [`39430a8`](https://github.com/luthermonson/go-proxmox/commit/39430a81c45098d7120dd895dbea3243a220bbb9) adding noop error and ignoring task from addtag in cloudinit ### 📊 Changes **2 files changed** (+14 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `proxmox.go` (+6 -0) 📝 `virtual_machine.go` (+8 -11) </details> ### 📄 Description regrets were have returning nil nil, adding a noop error type in case this happens again and decided to ultimately ignore the returned task in cloud init func since there wasn't much benefit to waiting for it to get done if we do actually call it fixes #68 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-03 15:30:17 +03:00
Sign in to join this conversation.
No labels
pull-request
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/go-proxmox#119
No description provided.