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/google: remove conflicting names from acceptance tests #4516

Merged
merged 1 commit into from
Jan 5, 2016

Conversation

lwander
Copy link
Contributor

@lwander lwander commented Jan 5, 2016

@phinze, for the sake of completeness I made sure all test resources have randomly generated names. Addresses #4514

@lwander lwander force-pushed the b-gcp-fix-conflicting-acc-tests branch from 8934400 to fb80ec8 Compare January 5, 2016 21:57
@phinze
Copy link
Contributor

phinze commented Jan 5, 2016

Nice, thank you! Merging and will kick off a fresh run of the full suite

phinze added a commit that referenced this pull request Jan 5, 2016
provider/google: remove conflicting names from acceptance tests
@phinze phinze merged commit 990f957 into hashicorp:master Jan 5, 2016
phinze added a commit that referenced this pull request Jan 5, 2016
[origin/master] Merge pull request #4516 from lwander/b-gcp-fix-conflicting-acc-tests
990f957
@phinze
Copy link
Contributor

phinze commented Jan 5, 2016

@lwander
Copy link
Contributor Author

lwander commented Jan 5, 2016

Np, fingers crossed

@lwander
Copy link
Contributor Author

lwander commented Jan 5, 2016

@phinze, oh boy, looks like I severely underestimated the amount of hardcoded test resource names. I'll try and fix these now.

@phinze
Copy link
Contributor

phinze commented Jan 5, 2016

Thanks! Hooray for public CI acctest runs 😉

@lwander lwander deleted the b-gcp-fix-conflicting-acc-tests branch January 13, 2016 22:16
@ghost
Copy link

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

Successfully merging this pull request may close these issues.

2 participants