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

helm_release hangs with high concurrency #456

Closed
scott-kausler opened this issue Mar 31, 2020 · 4 comments
Closed

helm_release hangs with high concurrency #456

scott-kausler opened this issue Mar 31, 2020 · 4 comments
Labels

Comments

@scott-kausler
Copy link

Terraform Version

Terraform v0.12.20

Affected Resource(s)

  • helm_release

Terraform Configuration Files

provider "helm" {
  // Helm 3.
  version = "1.1.1"
}

variable "count_installs" {
  type    = number
  default = 1
}

variable "namespace" {
  type    = string
  default = "test"
}

data "helm_repository" "azure_samples" {
  name = "azure-samples"
  url = "https://azure-samples.github.io/helm-charts/"
}

resource "helm_release" "hello_world" {
  count=var.count_installs
  namespace = var.namespace
  name      = "hello-world-${count.index}"
  chart     = "azure-samples/aks-helloworld"
  repository = data.helm_repository.azure_samples.metadata[0].name

  set {
    name  = "title"
    value = "hello-world-${count.index}"
  }

  set {
    name  = "serviceName"
    value = "hello-world-${count.index}"
  }
}

Debug Output

https://gist.github.com/scott-kausler/81d1c38cbb6e49d1898e823402a94aa5

Expected Behavior

All releases should have been created

Actual Behavior

Terraform hangs when creating the resources.

Steps to Reproduce

  1. terraform apply -var 'count_installs=11'

If for some reason that works, destroy then bump up the number of count_installs.

Important Factoids

Most of the resources seem to be created:

$ kubectl get deployments -n test
NAME                            READY   UP-TO-DATE   AVAILABLE   AGE
acs-helloworld-hello-world-0    1/1     1            1           77s
acs-helloworld-hello-world-1    1/1     1            1           77s
acs-helloworld-hello-world-10   1/1     1            1           77s
acs-helloworld-hello-world-2    1/1     1            1           77s
acs-helloworld-hello-world-3    1/1     1            1           77s
acs-helloworld-hello-world-4    1/1     1            1           77s
acs-helloworld-hello-world-5    1/1     1            1           77s
acs-helloworld-hello-world-7    1/1     1            1           77s
acs-helloworld-hello-world-8    1/1     1            1           77s
acs-helloworld-hello-world-9    1/1     1            1           77s

However, helm does not show them in the release list:

$ helm list -n test
NAME	NAMESPACE	REVISION	UPDATED	STATUS	CHART	APP VERSION
@aareet
Copy link
Contributor

aareet commented Jun 3, 2020

Related to #458

@aareet
Copy link
Contributor

aareet commented Jul 2, 2020

@scott-kausler we've made some improvements in the latest release - can you please tell us if you are able to reproduce this issue with the latest release?

@scott-kausler
Copy link
Author

@aareet I was unable to reproduce this issue with the latest release. Thank you.

@ghost ghost removed the waiting-response label Jul 6, 2020
@ghost
Copy link

ghost commented Aug 6, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!

@ghost ghost locked and limited conversation to collaborators Aug 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants