Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cli overhaul: guest id (no tests yet) #191

Merged
merged 5 commits into from
Sep 10, 2022

Conversation

Tinyblargon
Copy link
Collaborator

Another pull for #171

Work so far:

  • Made the commands for getting id inforamation.
  • Reimplemented the VMIdExists function to not rely on proxmox returning an error. this fuction is not used by the terraform provider so this change should't break anything there.
  • Reinplemented the legacy idstatus command as the new VMIdExists function broke this.

No tests yet for these command as they are dependent on the CRUD operations of LXC and Qemu which haven't been implemented yet.

@mleone87 mleone87 merged commit eeab51b into Telmate:master Sep 10, 2022
frostyfab pushed a commit to frostyfab/proxmox-api-go that referenced this pull request Nov 17, 2023
Cli overhaul: guest id (no tests yet)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants