-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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 New Datasource: Appmesh Virtual Router #29283
Conversation
Community NoteVoting for Prioritization
For Submitters
|
Hey @alowder94 👋 Thank you very much for your contribution! At times, our maintainers need to make direct edits to pull requests in order to help get it ready to be merged. Your current settings do not allow maintainers to make such edits. To help facilitate this, update your pull request to allow such edits as described in GitHub's Allowing changes to a pull request branch created from a fork documentation. (If you're using a fork owned by an organization, your organization may not allow you to change this setting. If that is the case, let us know.) |
My organization does not allow maintainer access |
@ewbankkit When you have a chance, can you trigger the check run for me so I can see if there are any other changes needed before this PR is ready? Thanks! |
Closing as this PR was merged into another and merged into main from there |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Description
This creates a data source for AWS Appmesh Virtual Router.
This will allow you to pull a virtual router into a project other than the one it was created in - which will allow for greater modularization of appmesh resources. Current state - if you want to use a virtual router, the route and node that it points to must also be created in the same project.
Relations
Closes #26908.
Relates #17590.
References
Test Command Used: go test -v internal/service/appmesh/virtual_router_data_source_test.go internal/service/appmesh/virtual_router_data_source.go internal/service/appmesh/flex.go internal/service/appmesh/virtual_router_test.go internal/service/appmesh/tags_gen.go
Only one test is written for this data source because both mesh_name and name of the router are required - since this creates a 1 to 1 relationship no optional fields are needed, and no disappears test needed for data source, leaving only the basic test.
Output from Acceptance Testing
$ go test -v internal/service/appmesh/virtual_router_data_source_test.go internal/service/appmesh/virtual_router_data_source.go internal/service/appmesh/flex.go internal/service/appmesh/virtual_router_test.go internal/service/appmesh/tags_gen.go
=== RUN TestAccAppMeshVirtualRouterDataSource_basic
=== PAUSE TestAccAppMeshVirtualRouterDataSource_basic
=== CONT TestAccAppMeshVirtualRouterDataSource_basic
--- PASS: TestAccAppMeshVirtualRouterDataSource_basic (50.74s)
PASS
ok command-line-arguments 65.793s