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

Container is not created "unmarshal string into Go struct" #30

Closed
me-nilix opened this issue Oct 28, 2021 · 2 comments
Closed

Container is not created "unmarshal string into Go struct" #30

me-nilix opened this issue Oct 28, 2021 · 2 comments
Labels
🐛 bug Something isn't working wontfix This will not be worked on

Comments

@me-nilix
Copy link

me-nilix commented Oct 28, 2021

Describe the bug
When running terraform apply, the container is created successfully, but the plugin cannot verify this.
I get this error:

2021-10-28T02:51:49.267-0400 [DEBUG] provider.terraform-provider-proxmox_v0.4.6: 2021/10/28 02:51:49 [DEBUG] Performing HTTP GET request (path: nodes/pxe-server/lxc/101/status/current)
2021-10-28T02:51:49.274-0400 [DEBUG] provider.terraform-provider-proxmox_v0.4.6: 2021/10/28 02:51:49 [DEBUG] WARNING: Failed to decode HTTP GET response (path: nodes/pxe-server/lxc/101/status/current) - Reason: json: cannot unmarshal string into Go struct field VirtualEnvironmentContainerGetStatusResponseData.data.vmid of type int

/proxmox$ pveversion
pve-manager/6.4-13/9f411e79 (running kernel: 5.4.143-1-pve)

[INFO] CLI command args: []string{"version"}
Terraform v1.0.9
on linux_amd64

  • provider registry.terraform.io/bpg/proxmox v0.4.6
@me-nilix me-nilix added the 🐛 bug Something isn't working label Oct 28, 2021
@bpg
Copy link
Owner

bpg commented Oct 31, 2021

Hi @me-nilix

I tried example/resource_virtual_environment_container.tf on proxmox 7.0 with provider version 0.4.6 and it works.
My guess is the provider's code now is backward incompatible with proxmox 6.x which is the version you seems to be using.

I'm going to re-release the current code as v0.5.0, to indicate its backward incompatibility. Unfortunately I don't I have resources to support multiple proxmox versions, but PRs are always welcome!

Cheers,
@bpg

bpg added a commit that referenced this issue Nov 6, 2021
Recent changes in the provider to make it compatible with Proxmox v7.x broke backward compatibility with previous versions, as was reported in #30.
This PR does not introduce any changes in the provider's functionality.

- Update README.md to indicate version compatibility with PVE
@bpg
Copy link
Owner

bpg commented Nov 6, 2021

Hi @me-nilix

I think v0.4.4 of the provider (this fork, or danitso/terraform-provider-proxmox) should work for your, but I can't test it as I have no PVE 6.4 available in my dev environment.

As I mentioned earlier, I won't be able to maintain backward compatibility with older PVE versions, so closing this issue.

@bpg bpg closed this as completed Nov 6, 2021
@bpg bpg added the wontfix This will not be worked on label Nov 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants