[GH-ISSUE #513] migrate: getVmInfo to `guest infos` #139

New issue

Closed

opened 2026-02-28 00:40:43 +03:00 by kerem · 2 comments

kerem commented

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

Owner

Originally created by @NemoDacremont on GitHub (Nov 9, 2025).
Original GitHub issue: https://github.com/Telmate/proxmox-api-go/issues/513

Originally created by @NemoDacremont on GitHub (Nov 9, 2025). Original GitHub issue: https://github.com/Telmate/proxmox-api-go/issues/513

kerem closed this issue

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

kerem commented

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

Author

Owner

@NemoDacremont commented on GitHub (Nov 9, 2025):

maybe i'll call it just vminfo, anyway I find a bug that leads to that :

> proxmox-api-go -i get vminfo 101
uint8: 0
null
Error: 501 Method 'GET /nodes//unknown/101/config' not implemented

This is probably coming from NewConfigQemuFromApi

EDIT: this is more precisely because vmRef is poorly initialized and won't have defined node attribute. This leads to wrong url generated in getGuestConfig from file client__api.go later

@NemoDacremont commented on GitHub (Nov 9, 2025): maybe i'll call it just `vminfo`, anyway I find a bug that leads to that : ``` > proxmox-api-go -i get vminfo 101 uint8: 0 null Error: 501 Method 'GET /nodes//unknown/101/config' not implemented ``` This is probably coming from NewConfigQemuFromApi EDIT: this is more precisely because vmRef is poorly initialized and won't have defined node attribute. This leads to wrong url generated in `getGuestConfig` from file `client__api.go` later

kerem commented

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

Author

Owner

@NemoDacremont commented on GitHub (Nov 9, 2025):

I worked on a working fix, I kind of had to introduced a new function NewVmRefFromApi(ctx context.Context, vmId GuestId, client *Client) (vmr *VmRef, err error)

I feel like it may be nice to have this function to reference a guest only by its ID, it is however a little less efficient since it has to loop over proxmox guest resources. I'll try to create the PR tomorrow

@NemoDacremont commented on GitHub (Nov 9, 2025): I worked on a working fix, I kind of had to introduced a new function `NewVmRefFromApi(ctx context.Context, vmId GuestId, client *Client) (vmr *VmRef, err error)` I feel like it may be nice to have this function to reference a guest only by its ID, it is however a little less efficient since it has to loop over proxmox guest resources. I'll try to create the PR tomorrow

kerem referenced this issue

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

[PR #139] [MERGED] Clone lxc implementation #258