-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
function "lookup" not defined #5032
Comments
Hi, Using the repo:
Using the chart:
Deployed charts:
|
I'm getting the same issue with external-dns, I'm using microk8s' helm3 which is this version:
Trying to install the latest 4.5.3 version which supposedly fixes a breaking change with backwards compability: https://github.com/bitnami/charts/tree/master/bitnami/common. |
Ah, looks like this it's this 14a0042#r46011226 |
Hi, |
Hello @travisjeffery @davejhahn, After investigating the error, we found out that it should be related to the used Helm version. It seems Helm 3.0 is reporting issues with the |
@FraPazGal in my situation I can't because it's the version microk8s uses, they'd have to upgrade. |
Yeah looks like I'm still getting the lookup issue with latest external-dns. |
@travisjeffery could you try to manually update? |
I'm pretty sure I had the lock saying 1.3.5 at one point and it still wasn't working, I could be wrong but I can't check now. I had to pin to 4.5.1 and move on. |
Hello, Yes, the issue comes from the current Helm version used. We have decided to bump the minimum Helm version to 3.1 so that new functionalities can be added to our charts. We understand this may inconvenience some users but opted for this solution to maintain our products up-to-date and benefitting from the newer features. For more information, please check #5091 |
I'm on helm 3.5.2 and still getting the error:
Is there anything else that I need to update? |
@donofriov just check which common version is deployed. |
@Cellebyte How can I do that? I thought common was included with the external-dns install as a dependency? I tried adding the latest common dependency but still getting an error
|
@donofriov just download the helm chart manually from here -> https://charts.bitnami.com/bitnami/external-dns-4.8.0.tgz |
Hi, |
Yea definitely thanks for the help @Cellebyte unfortunately I wasn't able just to update the bitnami versions because we are also using flux (v1). I just updated to the latest flux deployment (1.21.2) and flux helm-operator deployment (1.2.0) and that actually resolved the issue. Now i'm able to use external-dns 4.8.0 and common 1.3.9 without any errors, specifically the lookup function missing. Thanks for following up @rafariossaa |
@donofriov Nice that it is now working :) |
Should I process to close this issue ? |
Hello , Warning FailedReleaseSync 3s (x3 over 4m35s) helm-operator synchronization of release 'external-dns' in namespace 'clust-mgmt' failed: installation failed: parse error in "external-dns/charts/common/templates/_secrets.tpl": template: external-dns/charts/common/templates/_secrets.tpl:84: function "lookup" not defined I have external-dns - 4.8.4 and common - 1.4.1 Is this issue resolve for lookup function . |
@Dhinesh-ATCI <- please update your flux it is using probably an old helm version. |
Thanks @Cellebyte . My flux was deploying a helm-operator with version v2 as default and when I changed to v3 it is working good and I able to view the external-dns deployment now . |
Since it seems to be working now, I'm closing this issue. |
Which chart:
Latest (1/14/2021)
Describe the bug
Error: parse error at (zookeeper/charts/common/templates/_secrets.tpl:82): function "lookup" not defined
To Reproduce
Deploy helm chart
Additional context
I just deployed yesterday and didn't have any issues. After getting this error, I forced version 6.0.0 of the Helm Chart and it worked. Not sure if something is wrong with the helm chart or if there are some breaking changes?
The text was updated successfully, but these errors were encountered: