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/openstack: State Changes for Networking Resources #3712

Merged
merged 1 commit into from
Nov 3, 2015
Merged

provider/openstack: State Changes for Networking Resources #3712

merged 1 commit into from
Nov 3, 2015

Conversation

jtopjian
Copy link
Contributor

This commit adds create and delete state change checks for the "core"
networking resources.

@jtopjian
Copy link
Contributor Author

I'm going to leave this one open for a bit to see if anyone comments on it. No modifications to the existing acceptance tests were required, so it shouldn't cause anyone issues. Just being cautious.

This leads the way for #1789 which I have fixed up, added tests (which include a full network, subnet, port, secgroup, and instance stack), and verified it works. Will open that PR once this is merged in.

This commit adds create and delete state change checks for the "core"
networking resources.
jtopjian added a commit that referenced this pull request Nov 3, 2015
provider/openstack: State Changes for Networking Resources
@jtopjian jtopjian merged commit 6a791d1 into hashicorp:master Nov 3, 2015
omeid pushed a commit to omeid/terraform that referenced this pull request Mar 30, 2018
…service-exists

test/resource/aws_ecs_*: Check ECS for service existence in acceptance tests
@ghost
Copy link

ghost commented Apr 30, 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 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants