Set file on windows

[henkaa]

Me and my colleague are trying to get helmfile working for our helm charts, but we both run into trouble when using files in folders in the set file section.
We are both running helm and helmfile on a Windows 10 system (have tried both cmd and Powershell with same result) and both helm and helmfile is installed and maintained via scoop. helmfile is latest version.

Example from the helmfile.yaml

releases:
  - name: timespan-rest
    namespace: cda2-test
    installed: true
    createNamespace: true
    chart: cda2/timespan-rest
    values:
      - image:
          tag: "develop"
    set:
      - name: applicationyaml
        file: local/timespan-application.yml

When I run lint on it I get this output

ARGS:
0: helm (4 bytes)
1: lint (4 bytes)
2: C:\Users\henand2\AppData\Local\Temp\helmfile3968896767\cda2-test\timespan-rest\cda2\timespan-rest\latest\timespan-rest (118 bytes)
3: --namespace (11 bytes)
4: cda2-test (9 bytes)
5: --values (8 bytes)
6: C:\Users\henand2\AppData\Local\Temp\helmfile2343919300\cda2-test-timespan-rest-values-574f6f4f6d (96 bytes)
7: --set-file (10 bytes)
8: applicationyaml=local\timespan-application.yml (46 bytes)

ERROR:
exit status 1

EXIT STATUS
1

STDERR:
Error: failed parsing --set-file data: open localtimespan-application.yml: The system cannot find the file specified.

COMBINED OUTPUT:
Error: failed parsing --set-file data: open localtimespan-application.yml: The system cannot find the file specified.

---

There seems to be a "transformation" of slashes made, and on windows, one backslash is not enough. We have tried using following variants, all with the same outcome:
local/timespan-application.yml
local\/timespan-application.yml
local\timespan-application.yml
local\\timespan-application.yml

Are we doing something wrong or have we run into a bug here?

[yxxhero]

@henkaa could you try to use abs path?

[henkaa]

Tested an abs patch and it seems to work with forward slashes and "double" backslash (to make it a single backslash). But it would be ideal if relative path also work because it would mean less work for our team members.

[yxxhero]

@henkaa please give me more info about the path.

[henkaa]

Hope this is more (and wanted) information. Otherwise, just state more details of what you need :)
Here is an example. The file timespan-application.yml is the applications property-file and is written as a configMap to the kubernetes.
(sorry for the missing prefix spaces on each line)

Here is a working block (with abs path)
- name: timespan-rest
namespace: cda2-test
installed: true
createNamespace: true
chart: cda2/timespan-rest
values:
- image:
tag: "develop"
set:
- name: applicationyaml
file: C:\versionshantering\git\gitlab\cyb-sec-sol\katalog\products\cda-2\helmfile\local\timespan-application.yml

Preferebly we would like this format, since it is easier for us coworkers and would also work on our repositories CD/CI.
- name: timespan-rest
namespace: cda2-test
installed: true
createNamespace: true
chart: cda2/timespan-rest
values:
- image:
tag: "develop"
set:
- name: applicationyaml
file: local/timespan-application.yml

Our CI/CD is a GitLab repository, just for info.

Here is the wanted format, but as text file, so you have the complete file. I obscured our repository link (you will not be able to reach it anyway. timespan-application.yml is a regular Spring-Boot application properties file.
helmfile.txt

[yxxhero]

@henkaa it's ok. thanks.

[yxxhero]

@henkaa please show your directory structure.

[henkaa]

Basicly it is two folders.
One with the helmfile (where I execute >helmfile lint)

And one with the subfolder

All the other files in helmfile directory is there since we need to have all config files in the same directory to work with relative paths at the moment. And as you notice, there are a lot of other files, so the example you got earlier is just a part of the helmfile, but it is isolated to not depend of the other files at all.

And oc, on my teammates computer, he has this in another folder but the internal folder structure is same. This also applies for GitLab oc.

[yxxhero]

@henkaa BTW. please provider the debug log by adding the --debug flag.

[yxxhero]

@henkaa I don't have the windows os. so I need the log so much.

[henkaa]

It was sleeping hours here in Sweden, hence no replay. Will fix the logs :)

[yxxhero]

@henkaa so sorry.

[henkaa]

No worries :)
Here is the log (once again, obscured the repository link for security reasons, the rest of the log is intact).
You notice the error on row 73-74 in log. when we use relative path it seems to translate to single backslash.
helmfile.log

[yxxhero]

@henkaa if you run helm lint directly. did it work?

[henkaa]

When I use helm directly and files are in another folder than "Working folder", I also have to use double backslashes for it to work.

As you can see from the log file attached, it fails when we use only one backslash. For two backslash or one forward slash, it works.
helm.log

[yxxhero]

@henkaa fixed. could you try to use the latest version by building the main branch?

[yxxhero]

@henkaa looking forward to your feedback.

[henkaa]

Stupid question, but how do I build it? I have never worked with go-builds :)

[yxxhero]

@henkaa could you use docker? if so, you can use canary image: docker pull ghcr.io/helmfile/helmfile:canary

[henkaa]

Have docker desktop. But I have a question here.
It says to bind a volume to helm at ${HOME}/.helm or ${HOME}/.config/helm (depending on 2 or 3) but on my system, there is none of these dirs. I run helm via scoop. Should I point to helm install directory in scoop maybe? And it seems to be a v3 via scoop.

And just do be sure. When running via Docker. Are we guaranteed that this tests the Windows path correctly?

[yxxhero]

@henkaa yeah. it's a question. please wait for the latest binary. Thanks.

[henkaa]

I got curious, so I actually installed go via scoop and made a build.. first I verified with the regular helmfile.exe (via Scoop) that the error existed in my example. (and it was). Then I ran the same lint, but with the built version of helmfile.exe and then the linting had no problems finding my config file via the relative path :)

So it seems the fix works :) Looking forward on an official build for this fix.

[yxxhero]

@henkaa thanks so much. coming soon.

[yxxhero]

@henkaa 0.155.0 released. please enjoy it. if have any new questions. please post a new issue.

[henkaa]

Any idea on when it arrives on Scoop?

[yxxhero]

waiting https://github.com/ScoopInstaller/Main to auto-update.