value-files with multiple files: JSON string of array? about helm HOT 8 CLOSED

@kierenj ya it's a bit tricky to pass arguments with actions, I wish it would just allow full yaml syntax and pass that along somehow to the action.

The code that does this is here: https://github.com/deliverybot/helm/blob/master/index.js#L81-L97

I believe what's happening is you have a trailing slash in the array which is causing it to be invalid JSON, if it's invalid the action just tries to interpret it as a string. If you remove the trailing slash everything should work :)

from helm.

This is what I'm doing:

    - uses: deliverybot/helm@v1
      with:
        helm: helm3
        release: foobar
        namespace: default
        chart: path/to/chart
        dry-run: true
        value-files: '["values.yaml", "values.production.yaml"]'

You shouldn't indent the values. If you really want to (eg. for readability) I believe this is valid YAML:

    - uses: deliverybot/helm@v1
      with:
        helm: helm3
        release: foobar
        namespace: default
        chart: path/to/chart
        dry-run: true
        value-files: >-
          [
            "values.yaml", 
            "values.production.yaml",
          ]

Here is helpful information on how to deal with strings that contain multiple new lines: https://yaml-multiline.info/

from helm.

Thanks @riker09 - I think the trailing slash would still cause an error.

from helm.

What trailing slash are we talking about? I can spot a trailing comma which may cause an error.

from helm.

Ya my bad I actually meant comma

from helm.

So in conclusion a perfectly valid configuration would look like this:

    - uses: deliverybot/helm@v1
      with:
        helm: helm3
        release: foobar
        namespace: default
        chart: path/to/chart
        dry-run: true
        value-files: >-
          [
            "values.yaml", 
            "values.production.yaml"
          ]

Notice the removed comma on the second to last line.

from helm.

There should be a clear example in the README. I've spent hours trying to figure out this problem with this action.

from helm.

@nodox feel free to contribute an example, will be happy to merge it.

from helm.