[PR #34] [MERGED] Feature/lxc #169

New issue

Closed

opened 2026-02-28 00:40:57 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 00:40:57 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Telmate/proxmox-api-go/pull/34
Author: @ghost
Created: 7/6/2019
Status: ✅ Merged
Merged: 7/26/2019
Merged by: @ggongaware

Base: master ← Head: feature/lxc

📝 Commits (10+)

44badbc create lxc container
0b8f4ba remove lxc.json example
f06c5cd reuse existing string concatenation functions
cb9b492 reuse failure handling
5946614 add parameters for /nodes/{n}/lxc
5693415 add all proxmox params for lxc
4d92da8 read lxc config from api
0b99ece add SetLxcConfig to update via PUT request
5749306 udpate lxc hostname with SetLxcConfig
d81973a fix cores and complete update function

📊 Changes

4 files changed (+493 additions, -1 deletions)

View changed files

📝 main.go (+17 -1)
📝 proxmox/client.go (+42 -0)
➕ proxmox/config_lxc.go (+433 -0)
📝 proxmox/config_qemu.go (+1 -0)

📄 Description

This adds CRUD features for LXC containers.

NewConfigLxc() is used to provide the default set of parameters according to the Proxmox API doc
To delete a container, DeleteVm can be reused. The same applies to starting/stopping the containers (StartVm/StopVm)
The read functionality now differentiates between vmTypes (lxc or qemu), because the Proxmox API comes with a different set of parameters for containers than for VMs.

_{🔄 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/34 **Author:** [@ghost](https://github.com/ghost) **Created:** 7/6/2019 **Status:** ✅ Merged **Merged:** 7/26/2019 **Merged by:** [@ggongaware](https://github.com/ggongaware) **Base:** `master` ← **Head:** `feature/lxc` --- ### 📝 Commits (10+) - [`44badbc`](https://github.com/Telmate/proxmox-api-go/commit/44badbc1a483fe58323d3a033a4a636aa9f7477b) create lxc container - [`0b8f4ba`](https://github.com/Telmate/proxmox-api-go/commit/0b8f4ba111f9b0fb66c684869055454bab4dda0a) remove lxc.json example - [`f06c5cd`](https://github.com/Telmate/proxmox-api-go/commit/f06c5cdd8885e0a4378b735a315e23d165e2a039) reuse existing string concatenation functions - [`cb9b492`](https://github.com/Telmate/proxmox-api-go/commit/cb9b492320316ceb200ae3fd6689fae5c30e9acd) reuse failure handling - [`5946614`](https://github.com/Telmate/proxmox-api-go/commit/5946614fd137952be0425dfe6207c5d45e1df02d) add parameters for /nodes/{n}/lxc - [`5693415`](https://github.com/Telmate/proxmox-api-go/commit/5693415a1faa5e3d056e0a7b32a7c66bc71d2f6b) add all proxmox params for lxc - [`4d92da8`](https://github.com/Telmate/proxmox-api-go/commit/4d92da86a106fa2742c10c350c00e2df71b41fcc) read lxc config from api - [`0b99ece`](https://github.com/Telmate/proxmox-api-go/commit/0b99ece3be6ce3054e98a2dbad905c6b336d45ca) add SetLxcConfig to update via PUT request - [`5749306`](https://github.com/Telmate/proxmox-api-go/commit/57493060755fa47f772dd5e96e9820ee6f9b7ebc) udpate lxc hostname with SetLxcConfig - [`d81973a`](https://github.com/Telmate/proxmox-api-go/commit/d81973a66aa1d2d8c2e8310ec21f9fe14153fd6d) fix cores and complete update function ### 📊 Changes **4 files changed** (+493 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `main.go` (+17 -1) 📝 `proxmox/client.go` (+42 -0) ➕ `proxmox/config_lxc.go` (+433 -0) 📝 `proxmox/config_qemu.go` (+1 -0) </details> ### 📄 Description This adds CRUD features for LXC containers. * `NewConfigLxc()` is used to provide the default set of parameters according to the [Proxmox API doc](https://pve.proxmox.com/pve-docs/api-viewer/index.html) * To delete a container, `DeleteVm` can be reused. The same applies to starting/stopping the containers (`StartVm`/`StopVm`) * The read functionality now differentiates between `vmTypes` (`lxc` or `qemu`), because the [Proxmox API](https://pve.proxmox.com/pve-docs/api-viewer/index.html) comes with a different set of parameters for containers than for VMs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>