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

[Discussion] Official HashiCorp verification status #299

Closed
jackson-theisen opened this issue Aug 1, 2022 · 13 comments · Fixed by opensearch-project/terraform-provider-opensearch#7

Comments

@jackson-theisen
Copy link

I noticed that this provider was initially PR'd to HashiCorp's official Terraform repository back when all of their 'Official' providers were managed within that repository. They've since refactored their approach a bit and now manage 'official' providers in their own repository within the HashiCorp organization.

I wanted to inquire around whether there are any plans to either submit this repo to https://github.com/opensearch-project and get it verified by HashiCorp through their partnerships program, or potentially even submit it as an official provider to be owned and maintained by HashiCorp.

@phillbaker deserves to be recognized for his efforts on this project, but since it is by far the best solution out there today that is compatible with OpenSearch, it'd be awesome to see this get some official recognition. Selfishly, I would also like this to become officially recognized so I don't have to deal with any red tape in an enterprise setting. @phillbaker - what are your thoughts on this? I'd be happy to help with anything necessary to make this happen.

@phillbaker
Copy link
Owner

Hi @jackson-theisen, not opposed to transferring ownership on my side, if there is corresponding interest on the opensearch-project side and maintainers willing to take that on.

If you'd be able to shepherd that process through, I'm open to it!

@jackson-theisen
Copy link
Author

jackson-theisen commented Aug 3, 2022

@phillbaker I really admire your willingness to transfer the ownership. I'd be happy to attempt to shepherd the process through but will likely need to reach out to you around any opinionated changes that might be requested. Mainly, my concern is that if we want https://github.com/opensearch-project to adopt it, they'll want it branded as opensearch-specific. In my personal opinion, shifting the focus to solely supporting OpenSearch would be a good idea as the elasticsearch community has their own provider out now: https://github.com/elastic/terraform-provider-ec. Shame that the community wound up so divided on this topic, but it is what it is at this point. Will reach out to opensearch-project maintainers today to take their temperature on adopting this project.

Edit: The maintainers of the OpenSearch project moved my issue to opensearch-project/opensearch-devops#82 and it sounds like they're open to the idea as well. Will update here accordingly once I hear back.

@elfisher
Copy link

elfisher commented Aug 4, 2022

hey @phillbaker and @jackson-theisen, @bbarani and I are from the OpenSearch Project. We will look into creating a repo in the OpenSearch org for a terraform provider. Would one of you be willing to create a PR into the repo using this content once we have it setup?

@bbarani
Copy link

bbarani commented Aug 4, 2022

Thanks @elfisher . @phillbaker @jackson-theisen Do you have any preference on the repo name? Listing some options here.. terraform-provider-opensearch | terraform-provider-for-opensearch | opensearch-terraform-provider

Feel free to provide your recommendation

@jackson-theisen
Copy link
Author

jackson-theisen commented Aug 4, 2022

Hey @elfisher and @bbarani, thanks for being so quick to respond on this request! To add some context, I am currently not a contributor on this repo and have no affiliation to @phillbaker outside of this request. I'm certainly planning to get more involved as the incentive behind my request was actually so I could begin using this provider (TLDR; red tape around unofficial providers).

That said, it seems like @phillbaker is fully on board with the migration as long as I would handle the more administrative tasks around the process. I'll take point on submitting a PR to the repo that you guys create. The only changes that I'd plan to make would be naming updates and anything necessary to meet compatibility requirements with the official terraform registry (unlikely that any will be needed). Here's a couple references that i'll be using to validate the code before submitting the PR:

Regarding naming, terraform-provider-opensearch is the clear winner imo (@phillbaker should get final say) as that is the standard convention across the Terraform community. As I previously mentioned, I also think that the future of this provider should strictly focus on OpenSearch compatibility. While it currently supports both OpenSearch and ElasticSearch, it makes sense to deprecate support for anything specific to the latter as https://github.com/elastic/terraform-provider-ec will become the primary tool in that regard. @phillbaker please let me know if you're ok with this and I'll plan to include a note about the shift in focus in the README of the initial PR to the new repo.

@elfisher
Copy link

elfisher commented Aug 5, 2022

Regarding naming, terraform-provider-opensearch i

For what it is worth I also do like this name!

@phillbaker
Copy link
Owner

Hi all,

Regarding naming, yes terraform-provider-opensearch is the convention and we should stick with that.

the future of this provider should strictly focus on OpenSearch compatibility. While it currently supports both OpenSearch and ElasticSearch, it makes sense to deprecate support for anything specific to the latter as https://github.com/elastic/terraform-provider-ec will become the primary tool in that regard. @phillbaker please let me know if you're ok with this and I'll plan to include a note about the shift in focus in the README of the initial PR to the new repo.

Agreed here as well. I'll update this existing repository to deprecate opendistro/opensearch resources in favor of the new provider once we get it released. I'll continue to maintain elasticsearch 7.x resources in this repository.

Please let me know what I can do to help.

@bbarani
Copy link

bbarani commented Aug 19, 2022

Thanks for the update @phillbaker. I will update this thread once the repo terraform-provider-opensearch is created under OpenSearch organization.

@bbarani
Copy link

bbarani commented Oct 7, 2022

@phillbaker @jackson-theisen I apologize for the delay. terraform-provider-opensearch repo has been created under OpenSearch project. I will send the invitation to join the GitHub repo to your GitHub handle soon. Please let me know if you have any other questions. CC: @elfisher

@phillbaker
Copy link
Owner

Thanks, @bbarani, I received the invitation. It doesn't look like I have permissions, but please make sure that github actions is enabled for the repository.

@bbarani
Copy link

bbarani commented Oct 11, 2022

@phillbaker Thanks for accepting the invite. You should have necessary permission to view GitHub actions tab now.

@phillbaker
Copy link
Owner

Hi @bbarani, workflows don't seem to be enabled. Please see opensearch-project/terraform-provider-opensearch#7 - I can't access https://github.com/opensearch-project/terraform-provider-opensearch/settings/actions, but that's where it would be enabled.

@phillbaker
Copy link
Owner

@bbarani checking in here - this is preventing progress on getting the first PR merged.

As well, we'll need:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants