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

chore: adds aiven as provider #152

Closed
wants to merge 1 commit into from
Closed

Conversation

wanisfahmyDE
Copy link

No description provided.

@wanisfahmyDE wanisfahmyDE requested a review from a team as a code owner January 10, 2023 16:26
@wanisfahmyDE wanisfahmyDE requested review from mutahhir and ansgarm and removed request for a team January 10, 2023 16:26
@hashicorp-cla
Copy link

hashicorp-cla commented Jan 10, 2023

CLA assistant check
All committers have signed the CLA.

@xiehan
Copy link
Contributor

xiehan commented Jan 10, 2023

Hi @wanisfahmyDE, thanks for the PR! We've been meaning to formalize the process around these types of requests so I apologize that that's not set up yet.

First, I want to make sure you are aware that you can already use any provider available on the Terraform Registry, including the aiven provider, even though we don't have pre-built versions available for all of them; in order to use this provider, you would follow the instructions documented here: https://developer.hashicorp.com/terraform/cdktf/concepts/providers#add-provider-to-cdktf-json or just run the command cdktf provider add aiven/aiven and it'll generate the provider bindings for you locally and allow you to use this provider in your project.

In terms of creating a pre-built provider for aiven, which is what your PR here does, the best we can do right now is pass that request along to Aiven and see if they're willing and able to publish and maintain it. Maintaining pre-built providers is quite resource-intensive for us, so we're gradually shifting to a model where the only pre-built providers that HashiCorp maintains are for the Terraform providers classified as official. For partners like Aiven, we'll be working toward shifting maintenance over to them. Unfortunately, we still need to build tooling to make it easier for partners to publish and maintain their own pre-built provider, so it could be some months before we a pre-built version of the aiven provider is available.

In the meantime, hopefully the workflow that I described in my second paragraph is sufficient for your use case.

@wanisfahmyDE
Copy link
Author

Hi @xiehan, thanks for the feedback and for the quick response! we are aware of the local approach, just wanted to enhance the process.

This PR was a quick attempt to tackle aiven/terraform-provider-aiven#971

That being said, I would leave it to the Aiven provider maintainers @Serpentiel @ivan-savciuc to take this topic forward

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for at least 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue so we can investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 11, 2023
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.

3 participants