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

Help to use version 5.4.0 of helm chart #405

Closed
leandromoreirati opened this issue Jul 9, 2024 · 1 comment · Fixed by #425
Closed

Help to use version 5.4.0 of helm chart #405

leandromoreirati opened this issue Jul 9, 2024 · 1 comment · Fixed by #425

Comments

@leandromoreirati
Copy link

Hi, Everyone!

How are you?

I'm testing upgrade of my atlantis chart version of the 4.13.0 to 5.4.0, I adjusted all needed parameters but when I execute the helm
I'm receiving the following error:

Error: template: atlantis/templates/statefulset.yaml:44:47: executing "atlantis/templates/statefulset.yaml" at <.Values.dnsPolicy>: dnsPolicy is not a method but has arguments

image

I commented the parameter dnsPolicy inte the values file, but error persists.

Anyone could be a a sugestion how solve this problemas, please?

I grateful any tip.

@patipat-paypay
Copy link

I face the same issue if I use helm client version v3.8.1. There is no issue when I use helm client version v3.14.3.
If you need to stick with the older version. You can try rearrange the code https://github.com/runatlantis/helm-charts/blob/main/charts/atlantis/templates/statefulset.yaml#L44 to

{{- if and (or .Values.dnsPolicy (and .Values.dnsPolicy .Values.dnsConfig)) (ne .Values.dnsPolicy "ClusterFirst") }}

tmessinis added a commit to tmessinis/atlantis-helm-charts that referenced this issue Sep 20, 2024
GMartinez-Sisti pushed a commit that referenced this issue Sep 28, 2024
* fix(templates): Fix and boolean on line 44 of statefulset.yaml

Issue: #405

* Update chart version to 5.5.2

* Increase the sizeLimit of the init-shared-path emptyDir to 300Mi

* Update readme with new default value for initConfig.sizeLimit

---------

Co-authored-by: Theo Messinis <theo.messinis@crowdstrike.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants