Ed Overton

Results 46 comments of Ed Overton

On 1.2.2, this test passes: ``` func Test_Afero_MkdirAll(t *testing.T) { fs := afero.NewMemMapFs() err := fs.MkdirAll("/this/is/a", 0755) require.NoError(t, err) validateExpectedPerms(t, fs, "drwxr-xr-x", "/this/is/a") validateExpectedPerms(t, fs, "----------", "/this", "/this/is") err =...

@kargakis, I'm confused. I posted a test case that shows problematic behavior in 1.2.2. If that test passes, it implies faulty behavior. Note how there are directories created with "---------"...

For what it's worth, this issue was resolved at or before afero v1.6.0.

Thanks for the quick response! I'm running 1.14.1.

Thanks, @dnephin - given the root issue, I'm fine if you'd like to close this one out. I appreciate the help!

@KnVerey and/or @mengqiy, I'm just checking in to see if you expect to be able to get to this one soon. I'm happy to run fresh comparisons against the current...

After thinking this through, perhaps a better route would be for me to abandon this PR and come at the memory issue from another direction. My idea is an initial...

As an aside, yes, there were significant performance improvements with v5.0.0 (#4791, #4944, #4809). Back to the issue at hand... Looking around a little bit, I suspect the performance issue...

I haven't had time to investigate anything recently, sorry. If you'd like, I can rebase my work from https://github.com/kubernetes-sigs/kustomize/issues/5084#issuecomment-1806477953 and put it up as a PR if you want to...

>> If you'd like, I can rebase my work from https://github.com/kubernetes-sigs/kustomize/issues/5084#issuecomment-1806477953 and put it up as a PR if you want to see whether that provides any help. > Yes,...