[PR #142] [MERGED] Implement HA Group setting #260

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/Telmate/proxmox-api-go/pull/142
Author: @RubenHoms
Created: 10/22/2021
Status: ✅ Merged
Merged: 10/22/2021
Merged by: @ggongaware

Base: master ← Head: master

📝 Commits (1)

8173eca Implement HA Group setting

📊 Changes

3 files changed (+19 additions, -8 deletions)

View changed files

📝 proxmox/client.go (+12 -3)
📝 proxmox/config_lxc.go (+4 -3)
📝 proxmox/config_qemu.go (+3 -2)

📄 Description

This change will implement the HA Group setting. We could already define the HA State (running, stopped, etc.) but with this change it'll be possible to also set what HA Group a resource should belong to.

I successfully tested this on my own proxmox instance by forking the proxmox provider and changing the scheme to accept the HA Group parameter. I wanted to merge it back here first before I open a pull request on the terraform-provider-proxmox repo to ensure nothing breaks.

_{🔄 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/142 **Author:** [@RubenHoms](https://github.com/RubenHoms) **Created:** 10/22/2021 **Status:** ✅ Merged **Merged:** 10/22/2021 **Merged by:** [@ggongaware](https://github.com/ggongaware) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`8173eca`](https://github.com/Telmate/proxmox-api-go/commit/8173eca917cedd25165ea01e7bc49d1ed3fd95f2) Implement HA Group setting ### 📊 Changes **3 files changed** (+19 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `proxmox/client.go` (+12 -3) 📝 `proxmox/config_lxc.go` (+4 -3) 📝 `proxmox/config_qemu.go` (+3 -2) </details> ### 📄 Description This change will implement the HA Group setting. We could already define the HA State (running, stopped, etc.) but with this change it'll be possible to also set what HA Group a resource should belong to. I successfully tested this on my own proxmox instance by forking the [proxmox provider](https://github.com/Telmate/terraform-provider-proxmox) and changing the scheme to accept the HA Group parameter. I wanted to merge it back here first before I open a pull request on the `terraform-provider-proxmox` repo to ensure nothing breaks. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>