Skip to content

Conversation

@wswsmao
Copy link
Contributor

@wswsmao wswsmao commented Oct 15, 2025

#2113 introduced prefetch-list, but it only supports specific filenames. This commit adds wildcard support for the prefetch-list option, allowing users to flexibly select files to prefetch using wildcards.

Comment on lines 190 to 201
for name := range r.allPaths {
matched, err := doublestar.Match(pattern, name)
if err != nil {
return matchedCount, err
}
if !matched {
continue
}
if err := r.Record(name); err != nil {
continue
}
matchedCount++
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you move this to the optimize.go so that the new dependency isn't added to the root gomod?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Signed-off-by: abushwang <abushwang@tencent.com>
Copy link
Member

@ktock ktock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@ktock ktock merged commit baee5d1 into containerd:main Oct 17, 2025
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants