kustomize broken on Windows absolute paths

Created on 30 Jul 2020  路  3Comments  路  Source: kubernetes-sigs/kustomize

From powershell:

PS C:\Users\nick> kustomize build \Users\nick\scratch
PS C:\Users\nick> kustomize build .\scratch
PS C:\Users\nick> kustomize build C:\Users\nick\scratch
Error: unable to find one of 'kustomization.yaml', 'kustomization.yml' or 'Kustomization' in directory 'C:\Users\nick\AppData\Local\Temp\kustomize-089348767\repo'

The first two invocations work fine. The last invocation tries to create a temp Kustomize directory and then complains it doesn't exist? Why would it do this?

This seems to have broken recently AFAICT

PS C:\Users\nick> kustomize version
{Version:kustomize/v3.8.1 GitCommit:0b359d0ef0272e6545eda0e99aacd63aef99c4d0 BuildDate:2020-07-16T00:58:46Z GoOs:windows GoArch:amd64}
areapi kinbug prioritbacklog triagaccepted

Most helpful comment

I reproduced the bug locally, and I think this is a problem in hashicorp/go-getter and therefore, the "lite" version forked for kustomize, yujunz/go-getter

I submitted a PR to hashicorp go-getter to handle Windows-style absolute paths:
https://github.com/hashicorp/go-getter/pull/284

When applying this change locally, it fixes the problem.

If this gets merged, then I can submit the same PR (can I cherry-pick across repos? not sure) to yujunz/go-getter. Then update kustomize to use this new version and it should fix the issue.

/assign

All 3 comments

I've confirmed that this worked correctly in kustomize v3.4.0

This issue is still present in version 3.8.3

I reproduced the bug locally, and I think this is a problem in hashicorp/go-getter and therefore, the "lite" version forked for kustomize, yujunz/go-getter

I submitted a PR to hashicorp go-getter to handle Windows-style absolute paths:
https://github.com/hashicorp/go-getter/pull/284

When applying this change locally, it fixes the problem.

If this gets merged, then I can submit the same PR (can I cherry-pick across repos? not sure) to yujunz/go-getter. Then update kustomize to use this new version and it should fix the issue.

/assign

Was this page helpful?
0 / 5 - 0 ratings

Related issues

TechnicalMercenary picture TechnicalMercenary  路  3Comments

davidsbond picture davidsbond  路  3Comments

mgoodness picture mgoodness  路  4Comments

drekle picture drekle  路  4Comments

davidknezic picture davidknezic  路  3Comments