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

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

Closed
Tracked by #1971
lblackstone opened this issue Oct 19, 2021 · 3 comments · Fixed by #3064
Closed
Tracked by #1971
Assignees
Labels
area/docs Improvements or additions to documentation area/helm area/yaml kind/enhancement Improvements or new features mro1 Monica's list of 1st tier overlay related issues resolution/fixed This issue was fixed

Comments

@lblackstone
Copy link
Member

lblackstone commented Oct 19, 2021

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, pulumi/pulumi#5642 is done, so we could implement the ready attribute workaround in all SDKs as an alternative.

Affected area/feature

API docs

@lblackstone
Copy link
Member Author

I think we should do pulumi/docs#6740 before this so that we don't duplicate efforts across multiple repos.

@EronWright
Copy link
Contributor

EronWright commented Apr 17, 2024

Update: the core problem has been fixed in the "v2" implementations of ConfigFile and ConfigGroup, see #1833 (comment).

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

@EronWright EronWright self-assigned this Jun 17, 2024
@EronWright
Copy link
Contributor

EronWright commented Jun 17, 2024

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:

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Improvements or additions to documentation area/helm area/yaml kind/enhancement Improvements or new features mro1 Monica's list of 1st tier overlay related issues resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants