-
Notifications
You must be signed in to change notification settings - Fork 523
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
Disk size TB not allowed #521
Comments
This issue is stale because it has been open for 60 days with no activity. Please update the provider to the latest version and, in the issue persist, provide full configuration and debug logs |
The issue has been fixed for LXC containers but not for QEMU VMs. See https://github.com/Telmate/terraform-provider-proxmox/blob/master/proxmox/resource_vm_qemu.go#L373 |
This issue is stale because it has been open for 60 days with no activity. Please update the provider to the latest version and, in the issue persist, provide full configuration and debug logs |
Issue still exists. |
This issue is stale because it has been open for 60 days with no activity. Please update the provider to the latest version and, in the issue persist, provide full configuration and debug logs |
https://github.com/Telmate/terraform-provider-proxmox/blob/master/proxmox/resource_vm_qemu.go#L430 still an issue (could you please disable this absolutely annoying bot? issues don't just disappear by themself...) |
This issue is stale because it has been open for 60 days with no activity. Please update the provider to the latest version and, in the issue persist, provide full configuration and debug logs |
https://github.com/Telmate/terraform-provider-proxmox/blob/master/proxmox/resource_vm_qemu.go#L430 still an issue (could you please disable this absolutely annoying bot? issues don't just disappear by themself...) |
This issue is stale because it has been open for 60 days with no activity. Please update the provider to the latest version and, in the issue persist, provide full configuration and debug logs |
This issue is stale because it has been open for 60 days with no activity. Please update the provider to the latest version and, in the issue persist, provide full configuration and debug logs |
@XDjackieXD #929 will fix this once a new release is created. |
Fixed in master branch |
The disk size argument does not take "T" which is valid in the Proxmox API.
When setting the disk size to for example 4096GB, Proxmox returns this as "4T" when queried for the size which is why terraform now always tries to change the size back from 4T to 4096GB (which is probably a no-op in Proxmox) on every apply.
Allowing for "T" in the size config would fix this imo.
The text was updated successfully, but these errors were encountered: