terraform-ibm-modules / terraform-ibm-icd-elasticsearch

Implements an instance of the IBM Cloud Databases for Elasticsearch service.
Apache License 2.0
0 stars 1 forks source link

Support using existing ES instance for ES DA #235

Closed akocbek closed 2 months ago

akocbek commented 3 months ago

As a user, I do not want to create ES DA in a stack, if ES instance already exists.

ocofaigh commented 3 months ago

@akocbek some further info was added in https://github.com/terraform-ibm-modules/terraform-ibm-icd-elasticsearch/issues/238

shemau commented 3 months ago

Does this even make sense?

I am interpreting this as a request for data source in terraform terms, versus a resource.

Of all the resources the DA creates, only access tags (ibm_resource_tag) and keys (ibm_resource_key) might possibly be helpful to extend the existing ES to a new stack.

This could equally apply to any of the persistent storage services (eg. object storage, secrets manager, key manager, cloud database).

I wonder if the RAG stack needs the logic, rather than the ES DA.

Is the best approach:

I think it is highly undesirable to have a DA that deploys nothing. So thinking out loud, maybe a couple of DAs, one to add access tags to a service and one to add resource keys to a service make sense with a stack that has the flavor to either use existing resources defined somewhere or provision all the required resources.

ocofaigh commented 3 months ago

It is needed. For example someone might want to enable the elser module on an existing instance, or create an index on an existing instance. Its also needed for the stack use case where there is no way currently to just ignore a member DA inside a stack

kierramarie commented 3 months ago

Currently blocked by this issue: https://github.com/IBM-Cloud/terraform-provider-ibm/issues/5397

ocofaigh commented 3 months ago

The provider crash is intermittent so its not a blocker

kierramarie commented 2 months ago

PR has been approved and merged. https://github.com/terraform-ibm-modules/terraform-ibm-icd-elasticsearch/pull/243/