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

allocate_public_ip invalid in tencentcloud_instance #62

Closed
yongzhang opened this issue Jul 5, 2019 · 3 comments
Closed

allocate_public_ip invalid in tencentcloud_instance #62

yongzhang opened this issue Jul 5, 2019 · 3 comments

Comments

@yongzhang
Copy link

provider version: 1.11

no matter true or false of allocate_public_ip defined in tencentcloud_instance, after instance created, and execute terraform apply again, it will update allocate_public_ip: "true" => "false", and it cannot be updated.

  ~ tencentcloud_instance.test
      allocate_public_ip: "true" => "false"
* tencentcloud_instance.test: 1 error(s) occurred:

* tencentcloud_instance.test: tencentcloud_instance update on allocate_public_ip is not supported yet

Additional information: my instance has eip associated after created, I'm not sure if provider detected this as public_ip?

@likexian
Copy link
Contributor

likexian commented Jul 8, 2019

Yes, eip is detected as public ip, we now don't support update on allocate_public_ip.

@zqfan
Copy link
Contributor

zqfan commented Jul 10, 2019

@hiscal2015
Actually public ip created along with cloud virtual machine is not an EIP, it cannot be detached/unassociated with this instance, hence it is not allowed to be modified, this limit comes from API side.
If you have an EIP associated with the instance after its creation, you are in fact doing a association wihch should be managemented by a resource tencentcloud_eip_association.
The probem you are describing, is introduced by https://github.com/terraform-providers/terraform-provider-tencentcloud/pull/42 , which mixes too many codes in different resource, and we cannot catch this problem during code review process. A fix should be done to revert that change (https://github.com/terraform-providers/terraform-provider-tencentcloud/blame/master/tencentcloud/resource_tc_instance.go#L686) or be carefully deal with the logic.

@likexian
Copy link
Contributor

This issue has been fixed, please remove the define of allocate_public_ip and retry.

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

No branches or pull requests

3 participants