pulumi-kubernetes icon indicating copy to clipboard operation
pulumi-kubernetes copied to clipboard

Document dependsOn limitations for ComponentResource resources (Helm, YAML, Kustomize)

Open lblackstone opened this issue 4 years ago • 2 comments

Hello!

  • Vote on this issue by adding a 👍 reaction
  • If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

dependsOn doesn't await child resource creation for several resources in the k8s provider (helm.Chart, yaml.ConfigFile, yaml.ConfigGroup, kustomize.Directory). There is ongoing work to resolve this limitation, but we should document it more clearly in the meantime.

Related to #861

Edit:

Actually, https://github.com/pulumi/pulumi/issues/5642 is done, so we could implement the ready attribute workaround in all SDKs as an alternative.

Affected area/feature

API docs

lblackstone avatar Oct 19 '21 14:10 lblackstone

I think we should do https://github.com/pulumi/docs/issues/6740 before this so that we don't duplicate efforts across multiple repos.

lblackstone avatar Nov 02 '21 00:11 lblackstone

Update: the core problem has been fixed in the "v2" implementations of ConfigFile and ConfigGroup, see https://github.com/pulumi/pulumi-kubernetes/issues/1833#issuecomment-2061802888.

We'll leave this issue open until the problem is fixed for the other resources.

EronWright avatar Apr 17 '24 17:04 EronWright

Update: Chart "v4" is out and works great with dependsOn, likewise for Kustomize Directory "v2".

For Chart v3, a ready attribute was introduced for most SDKs and has a usage example in the apidocs:

  • NodeJS: https://github.com/pulumi/pulumi-kubernetes/pull/1364
  • Python: https://github.com/pulumi/pulumi-kubernetes/pull/1782
  • Go: https://github.com/pulumi/pulumi-kubernetes/pull/1784
  • .NET: https://github.com/pulumi/pulumi-kubernetes/pull/1785

The outstanding task appears to be to add info about dependsOn to the overlay resources. Update: we added links to the newer version and to the associated blog post.

EronWright avatar Jun 17 '24 18:06 EronWright