github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

Merge pull request #8281 from hach-que/patch-1

Browse Source 
fix: 'failed to resolve symlink' error messaging
This commit is contained in:
Maksym Pavlenko 2023-03-16 15:23:19 -07:00 committed by GitHub
commit 63e45eb5d8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pkg/cri/opts/spec_windows_opts.go
View File

@ -67,9 +67,10 @@ func parseMount(osi osinterface.OS, mount *runtime.Mount) (*runtimespec.Mount, e
} }
} }
var err error var err error
originalSrc := src
src, err = osi.ResolveSymbolicLink(src) src, err = osi.ResolveSymbolicLink(src)
if err != nil { if err != nil {
return nil, fmt.Errorf("failed to resolve symlink %q: %w", src, err) return nil, fmt.Errorf("failed to resolve symlink %q: %w", originalSrc, err)
} }
// hcsshim requires clean path, especially '/' -> '\'. Additionally, // hcsshim requires clean path, especially '/' -> '\'. Additionally,
// for the destination, absolute paths should have the C: prefix. // for the destination, absolute paths should have the C: prefix.