Code Monkey home page Code Monkey logo

Comments (9)

yxxhero avatar yxxhero commented on September 28, 2024 2

@Marwen-TAALLAH please format your code by code.

from helm.

Marwen-TAALLAH avatar Marwen-TAALLAH commented on September 28, 2024

@Marwen-TAALLAH please format your code by code.

Done !

from helm.

banjoh avatar banjoh commented on September 28, 2024

@Marwen-TAALLAH is there a step to process the values.yaml file first before passing it to helm, or is the file passed in as-is in the description? If so, I think helm will report an error because that file does not look like valid YAML

master:
  ${ indent(2, redis_affinity) }
  ${ indent(2, redis_nodeSelector) }
  ${ indent(2, redis_default_tolerations) }
  persistence:
    enabled: false

Specifically, the master value is a mixture of a map/dictionary and a scaler (string).

I'm not aware of helm supporting templating for the values file

from helm.

Marwen-TAALLAH avatar Marwen-TAALLAH commented on September 28, 2024

@Marwen-TAALLAH is there a step to process the values.yaml file first before passing it to helm, or is the file passed in as-is in the description? If so, I think helm will report an error because that file does not look like valid YAML

master:
  ${ indent(2, redis_affinity) }
  ${ indent(2, redis_nodeSelector) }
  ${ indent(2, redis_default_tolerations) }
  persistence:
    enabled: false

Specifically, the master value is a mixture of a map/dictionary and a scaler (string).

I'm not aware of helm supporting templating for the values file

@banjoh The values.yaml file is causing a problem only when the redis chart is called as a module, when executinf the terraform apply locally, the file doesn't cause an issue !!

from helm.

banjoh avatar banjoh commented on September 28, 2024

Are you able to create a reproducer chart, say by creating a chart using helm create foo then adding your values to the default values file? If you are using subcharts, then setup your test accordingly. It will be easier to troubleshoot the issue if terraform and any other external tool is absent.

from helm.

Marwen-TAALLAH avatar Marwen-TAALLAH commented on September 28, 2024

@banjoh I am able to create the chart with helm directly without terraform, I am also able to create the chart with terraform, all of that with the exact same values.yaml I have provided.
The problem occured when I tried to call the terraform code as a module.

from helm.

banjoh avatar banjoh commented on September 28, 2024

@Marwen-TAALLAH to rule out terraform as being the problem here, could you provide the helm commands that terraform successfully templates your helm chart with?

I cannot run terraform, so I'm not able to ascertain whether this values file is valid as you say.

Am looking to see the output of helm template <your-chart> --values values.yaml

from helm.

github-actions avatar github-actions commented on September 28, 2024

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

from helm.

JuryA avatar JuryA commented on September 28, 2024

@banjoh: I'm not aware of helm supporting templating for the values file

Helm does not offer direct support for this. Still, it is a frequent practice—otherwise, avoiding the repetition of the same values would often be impossible (or rather, possible, but highly inefficient). However, this issue is a complete mess—mixing Terraform with Helm, and the task does not make any sense. Even my most bizarre 'Helm sadism' (referring to overly complex and overengineered solutions), such as a complete AST parser implemented purely using Go Templates in Helm, would work better than the convoluted solution this person is trying to assemble here (no offense).

@Marwen-TAALLAH I have no idea what ${ ... } means—Terraform? Why isn't it described better? This does not look like a Helm issue... Your generated YAML is corrupted somehow. If it is not valid YAML, you must fix that first...

from helm.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.