Initial migration to Terraform Plugin Framework #279
Conversation
|
Do you want to rebase? Seeing you make such nice commits I'm hesitant to click the |
tete17
force-pushed
the
Initial-migration-to-plugin-framework
branch
from
173b919
to
bb90409
Compare
|
Thanks for the contribution, very appreciated. After internal discussions and how we want to prioritize our work. Migrating to framework will be put on hold until August/September to finish our other tasks before the summer. We will definitely keep an eye of any changes here. |
|
Thanks for the feedback @tbroden84 I understand the concern of merging this right away. I was just trying to fix #214 as soon as I could as it is something that affects the infra I manage for my employee. I will continue rebasing this MR as it is required. Ping me if you would like some help in other places |
tete17
force-pushed
the
Initial-migration-to-plugin-framework
branch
from
bb90409
to
059151f
Compare
tete17
force-pushed
the
Initial-migration-to-plugin-framework
branch
from
059151f
to
f94a12b
Compare
tete17
force-pushed
the
Initial-migration-to-plugin-framework
branch
2 times, most recently
from
c8c65d2
to
2e3e08d
Compare
tete17
force-pushed
the
Initial-migration-to-plugin-framework
branch
from
2e3e08d
to
ac3b427
Compare
|
Hi @tbroden84 & @dentarg I just rebased the branch again, let me know if you need something from my end :) |
|
@tete17 thanks. Expect small fixes to the provider we unfortunately still have this task on hold until later. Currently internal tasks that are higher prioritized. |
This is in preparation to migrate to the plugin framework.
This combined with the muxing will allow us to migrate resources and datasources one at a time
…rm plugin framework This is the first resource to move towards the new system and should serve as a template for all others. The reason to choose this one as the first is because it is the smallest file that still has tests to prove everything still works as before.
tete17
force-pushed
the
Initial-migration-to-plugin-framework
branch
from
ac3b427
to
9a70d09
Compare
|
Friendly trimestral reminder that this is still open. No rush I know it is not priority ;) I may start working on the migration of the resource I really want in parallel |
This is the first of many resources to be migrated to the Hashicorp Plugin Framework.
I chose this one because it was the smallest file that still contained test to make sure I could prove nothing had changed.
This 3 MR does 3 things:
cloudamqp_intregration_aws_eventbridgeto the new provider making sure test are not broken.Hopefully this migration will allow us to tackle issues like #214
The error messages are fairly barebones as I haven't given them too much though feel free to suggest a full rewrite on them, I focused on making the main path work, error messages are mostly an afterthought.