pulumi-kubernetes
pulumi-kubernetes copied to clipboard
Incorrect Release example - value_yaml_files should be a sequence
Hello!
- Vote on this issue by adding a 👍 reaction
- To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already)
Issue details
The following is not reading the local values
file for a helm chart:
wordpress = Release(
"wordpress",
ReleaseArgs(
chart="wordpress",
repository_opts=RepositoryOptsArgs(
repo="https://charts.bitnami.com/bitnami",
),
value_yaml_files=pulumi.FileAsset("./wordpress-values.yaml"),
),
)
Steps to reproduce
- Try the above code snippet
Expected: Expect the values in the values file are applied to the helm deploymennt Actual: values file appears to be unread.
I think you want to specify the value_yaml_files as a sequence: [pulumi.FileAsset("./wordpress-values.yml")]
. Not entirely sure why this didn't cause a warning for you earlier though.
See https://github.com/pulumi/pulumi-kubernetes/blob/07bf517e59859739ea259802a337468cb01add29/sdk/python/pulumi_kubernetes/helm/v3/Release.py#L705-L722
Interesting, I tried specifying the file path either prefixed with ./
, or not, and also absolute. Never received any errors - yet they cant all have been correct.
Ah the example needs to be updated - sorry about that. I am not entirely sure how the python sdk managed to escape the obvious error here...