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

Add creation timeout parameter for resource cluster #135

Merged
merged 1 commit into from
Aug 2, 2022
Merged

Add creation timeout parameter for resource cluster #135

merged 1 commit into from
Aug 2, 2022

Conversation

egor-georgiev
Copy link
Contributor

As per Confluent documentation, provisioning of dedicated clusters can take up to 24 hours. However, provisioning of clusters failed at due to context deadline being exceeded after 20 minutes (as per tf documentation, it's a default). This PR by @borisnaydis tried to address the issue, but changes did not remediate it. This PR adds a configurable timeout parameter for creation of clusters, which defaults to 24 hours, but can be changed in the resource configuration (snippet below). Code is tested locally and with this change it is possible to provision larger clusters to confluent without facing a timeout at around 20 minutes.

resource "confluentcloud_kafka_cluster" "cluster" {
  name             = "test_increase_create_timeout"
  service_provider = "aws"
  region           = "us-east-1"
  availability     = "HIGH"
  environment_id   = "env-abc123"
  deployment = {
    sku = "DEDICATED"
  }
  cku = 10
  timeouts {
    create = "60m"
  }
}

@borisnaydis
Copy link
Contributor

Hi @Mongey. Did you have a chance to look at this PR? Does it need something before it can be merged?

@sudiptad2017
Copy link

@Mongey Is there a chance you can merge this PR and release it. We are facing the same issue when terraform times out if Confluent takes more than 20 minutes to provision a DEDICATED cluster.

@Mongey Mongey merged commit b80ed90 into Mongey:master Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants