-
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
feat: add aws_vpc_ipam & aws_vpc_ipams data sources #40459
base: main
Are you sure you want to change the base?
Conversation
Community NoteVoting for Prioritization
For Submitters
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Welcome @Volatus 👋
It looks like this is your first Pull Request submission to the Terraform AWS Provider! If you haven’t already done so please make sure you have checked out our CONTRIBUTOR guide and FAQ to make sure your contribution is adhering to best practice and has all the necessary elements in place for a successful approval.
Also take a look at our FAQ which details how we prioritize Pull Requests for inclusion.
Thanks again, and welcome to the community! 😃
Hi @Volatus - Thanks for your submission! As of 2023-10-20 all net-new data sources are required to use Terraform Plugin Framework. More context is available in this pinned issue and the provider contributor guide.
Once prioritized, maintainers will convert Plugin-SDK based data sources as part of the review process, but if you'd like to do the conversion in the meantime we would greatly appreciate it! |
@jar-b Understood, I'll work on converting this. |
f15dfa6
to
0db6217
Compare
@jar-b I've made the necessary changes to use the plugin framework instead of SDK v2. Let me know if something needs changing. |
0db6217
to
de734be
Compare
de734be
to
774d8c8
Compare
@jar-b , @ewbankkit, any chance this gets any attention, please? Some of us do rely on this functionality and it's a ready PR just dusting on the shelf for over a month. |
Signed-off-by: Ismayil Mirzali <[email protected]>
Signed-off-by: Ismayil Mirzali <[email protected]>
Signed-off-by: Ismayil Mirzali <[email protected]>
350f278
to
e634bd6
Compare
Description
This PR introduces 2 new data sources:
aws_vpc_ipam
andaws_vpc_ipams
, following the convention of theaws_vpc_ipam_pool
andaws_vpc_ipam_pools
data sources.Relations
References
Output from Acceptance Testing
Signed-off-by: Ismayil Mirzali [email protected]