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

provider/alicloud: Fix allocate public ip error (#13267) #13268

Merged

Conversation

pragkent
Copy link
Contributor

@pragkent pragkent commented Apr 1, 2017

Wait for instance to be in STOPPED or RUNNING state before invoking
AllocatePublicIP API.

  • provider/alicloud: Wait for instance state before allocate public ip

  • provider/alicloud: Fix test TestAccAlicloudInstance_associatePublicIP

  • provider/alicloud: Update alicloud_instance document

Fixes: #13267

Wait for instance to be in STOPPED or RUNNING state before invoking
AllocatePublicIP API.

* provider/alicloud: Wait for instance state before allocate public ip

* provider/alicloud: Fix test `TestAccAlicloudInstance_associatePublicIP`

* provider/alicloud: Update alicloud_instance document

Fixes: hashicorp#13267
@pragkent
Copy link
Contributor Author

pragkent commented Apr 1, 2017

TEST:

make testacc TEST=./builtin/providers/alicloud TESTARGS=-run='TestAccAlicloudInstance_associatePublicIP'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/04/01 12:59:09 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/alicloud -v -run=TestAccAlicloudInstance_associatePublicIP -timeout 120m
=== RUN TestAccAlicloudInstance_associatePublicIP
--- PASS: TestAccAlicloudInstance_associatePublicIP (113.80s)
PASS
ok github.com/hashicorp/terraform/builtin/providers/alicloud 113.823s

@pragkent
Copy link
Contributor Author

Hi, is there anything else should I do before this PR could be merged?

@stack72
Copy link
Contributor

stack72 commented Apr 16, 2017

Hi @pragkent

This LGTM! Thanks for the work here :)

Paul

@stack72 stack72 merged commit 8927ad5 into hashicorp:master Apr 16, 2017
@pragkent pragkent deleted the provider-alicloud-fix-allocate-public-ip branch April 19, 2017 11:40
@ghost
Copy link

ghost commented Apr 13, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

provider/alicloud: allocate public ip error
3 participants