Telmate / proxmox-api-go

Consume the proxmox API in golang
MIT License
344 stars 230 forks source link

Overhaul: Qemu Network interfaces. #341

Open Tinyblargon opened 2 weeks ago

Tinyblargon commented 2 weeks ago

The network adapters of ConfigQemu{} don't neatly map to the data-structure the PVE API uses. We currently use an array. PVE uses an map[id]settings.

With our current implementation it's not possible to have network interfaces that aren't consecutive. In PVE users can have interface1 and interface3 configured. Our current implantation can't distinguish between.

and

In this instance interface5 is the second interface. so it will be the second item in the array. Before you start working on this please discuss it with me.

Tinyblargon commented 1 week ago

This will include #333