terraform-provider-datadog icon indicating copy to clipboard operation
terraform-provider-datadog copied to clipboard

Feature Request: add datadog_integration_confluent_cloud

Open miguelaferreira opened this issue 2 years ago • 1 comments

It would be very useful for my team to terraform the integration between DataDog and Confluent Cloud. Following the DataDog integration documentation, the new resource would need to allow for a Confluent Cloud API key, and a set of managed resources.

Affected Resource(s)

Please list the resources as a list, for example:

  • datadog_integration_confluent_cloud

Terraform Configuration Files

resource "datadog_integration_confluent_cloud" {
  api_key    = confluent_api_key.example.id
  api_secret = confluent_api_key.example.id

  managed_resource {
    id   = confluent_kafka_cluster.example.id
    type = "Cluster"
    tags = ["connect_cluster2"]
  }

  managed_resource {
    id   = confluent_connector.example.id
    type = "Connector"
    tags = ["datagen", "foo:bar"]
  }
}

miguelaferreira avatar Sep 13 '22 10:09 miguelaferreira

I'll be able to implement it once https://github.com/DataDog/datadog-api-client-go/issues/1677 is ✅ .

linouk23 avatar Sep 19 '22 07:09 linouk23

@linouk23 Have you started any implementation? Thanks.

jcunhafonte avatar Dec 05 '22 10:12 jcunhafonte

@jcunhafonte my focus has shifted to other tasks unfortunately but I can see there's some ongoing work here: https://github.com/DataDog/terraform-provider-datadog/pull/1682

linouk23 avatar Dec 21 '22 22:12 linouk23

Is there any ongoing work for this? Also, I would like to know if there's any kind of guide to contributing to this project, in case someone else wants to implement this feature.

One last question, is possible to configure the integration using the API? I don't see any example in the docs https://docs.datadoghq.com/api/latest/confluent-cloud/

Thank you

Conacious avatar Feb 15 '23 08:02 Conacious

Is there any ongoing work for this? Also, I would like to know if there's any kind of guide to contributing to this project, in case someone else wants to implement this feature.

One last question, is possible to configure the integration using the API? I don't see any example in the docs https://docs.datadoghq.com/api/latest/confluent-cloud/

Thank you

@Conacious You can use various Datadog API Clients to achieve this or directly with the REST API too:

  • https://github.com/DataDog/datadog-api-client-python/blob/master/examples/v2/confluent-cloud/UpdateConfluentResource.py
  • https://github.com/DataDog/datadog-api-client-typescript/tree/master/examples/v2/confluent-cloud
  • More: https://github.com/orgs/DataDog/repositories?q=datadog+api&type=all&language=&sort=

jcunhafonte avatar Feb 16 '23 20:02 jcunhafonte

@Conacious

I think there is some work in progress from @skarimo on https://github.com/DataDog/terraform-provider-datadog/pull/1734 about this :eyes:

pichouk avatar Feb 23 '23 13:02 pichouk

Reading the various notes, do we really need to wait for #1734 as that is about updating existing confluent resources, is not the real blocker #1776 ?

once that's in place this can start as a new resource not an existing one. Or am I miss reading #1734 ?

Marcus-James-Adams avatar Mar 01 '23 14:03 Marcus-James-Adams

I se that this has now been completed, but when I look at the terraform datadog resource page I can not see a resource for this

Marcus-James-Adams avatar Apr 24 '23 13:04 Marcus-James-Adams

This was merged and needs a release, which will happen soon.

therve avatar Apr 24 '23 13:04 therve

@Marcus-James-Adams as a follow up, I can see these 2 new resources:

  • https://registry.terraform.io/providers/DataDog/datadog/latest/docs/resources/integration_confluent_account
  • https://registry.terraform.io/providers/DataDog/datadog/latest/docs/resources/integration_confluent_resource

Thanks everyone!

linouk23 avatar Mar 26 '24 04:03 linouk23