-
Notifications
You must be signed in to change notification settings - Fork 504
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
Provider wants to modify host after apply #47
Comments
Confirmed it with another proxmox host:
|
Any ideas? |
Bump. Trying to utilize this project in a proof of concept. Having a hard time with the bugs. Would love to contribute anything I can, but need some sort of support. |
I'm facing the same problem and looking into it. In my case this does not fails but results in a not responding VM in the end. Still struggeling with the terraform plugin workflow. |
Let me know if you find anything. I thought it may have been the clone source VM having a network card but I ran into a lot of issues last night building the plugin. Now that (I think) we're past the bugs and crashes I am going to test this theory. |
After quite some debug messages, I'm still not at the point, were the current state is being confused. A
As I understand the structure correctly, the Any So - still I do have no clue, were the new state data is coming from. The new id (3884306486) of the network mapping is also not showing up within the logs in any of the state dictionaries. Maybe I find some time for further investigation. Maybe I have to find a way to start a debugger in Go (currenlt, I'm just using log outputs for analysis) |
I've been digging into this. The ids seem to come from a hashing function (https://github.com/hashicorp/terraform/blob/master/helper/schema/set.go#L16) ... I'm starting to trace the Diff routines now too. |
Found the cause: leaving macaddr blank and letting proxmox autogenerate throws of the terraforms hashing of sub-schemas. The work around for now is to supply a macaddr. Need to see if there's a way to prevent auto-generated macaddr in a sub-schema from triggering a change plan. |
That's great news! |
Having some new problems, can't confirm everything is working properly. Seems the provider is having some serious issues with the new agent api endpoints.
Any ideas? |
Can't seem to track this one down. Is anyone else seeing this issue on VM creation? Giving up for the night. Thanks for all the help so far! |
Pushed work around for agent types on the proxmox-api-go repo |
You are all steely eyed missile men. Thank you thank you thank you!
|
Here is my sample terraform code
After an apply the following values fail to populate in the state file:
It seems that terraform wants to create a new network device because the old one wasn't stored?
When a apply is done, it fails:
The text was updated successfully, but these errors were encountered: