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

Add initial Direct Connect resources #12567

Closed
wants to merge 3 commits into from

Conversation

bodgit
Copy link
Contributor

@bodgit bodgit commented Mar 9, 2017

This is based in part on the work started in #5212 however I've just adapted the single resource I need:

  • aws_directconnect_public_virtual_interface_confirm

I've updated it with the missing attributes and import functionality and also renamed the resource slightly so it fits better with the naming convention of other resources, (IMHO!).

Sadly this has the same sticking point in that testing it the same way as most other AWS resources doesn't really work as a) it involves more than one AWS account and b) Direct Connect infrastructure quite often involves some manual operator involvement or can take some time to be verified by AWS staff.

The directconnectiface package gives the ability to mock the API so I could test the Terraform resource without having to create anything physical/virtual however I'm struggling to work out how I can slot it in to the testing framework, it looks like I need to make my own AWSClient with a mocked dcconn.

I can't see any existing resources that I can copy or use for inspiration so I'm asking for some guidance here. I'd really like to get this resource added as it's the last outstanding piece I have left and maybe once the tests are figured out it'll help with adding the remaining Direct Connect resources.

@akumria
Copy link

akumria commented May 25, 2017

[bikeshed]: It really would be nice if the resource was named 'aws_dx_...'

Comparing the two PRs, I generally prefer PR#5212 since cover both the public and private virtual interface confirmation.

@bodgit
Copy link
Contributor Author

bodgit commented Jun 15, 2017

I'll close this in preference to hashicorp/terraform-provider-aws#876.

@bodgit bodgit closed this Jun 15, 2017
@ghost
Copy link

ghost commented Apr 8, 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 8, 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.

3 participants