Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

win_fetch - improve test time by not scanning Win dir #81884

Merged
merged 1 commit into from Oct 4, 2023

Conversation

jborean93
Copy link
Contributor

SUMMARY

A part of the fetch operation is to call stat on the path specified. For Windows this means it'll get the size of the target which can be an expensive operation for directories with lots of children. This changes the test so it uses an empty dir to speed up the test times.

ISSUE TYPE
  • Test Pull Request

@ansibot ansibot added test This PR relates to tests. needs_triage Needs a first human triage before being processed. labels Oct 4, 2023
@mattclay mattclay removed the needs_triage Needs a first human triage before being processed. label Oct 4, 2023
@jborean93 jborean93 merged commit dfc6258 into ansible:devel Oct 4, 2023
62 checks passed
@jborean93 jborean93 deleted the win-fetch-size branch October 4, 2023 14:57
jborean93 added a commit to jborean93/ansible that referenced this pull request Oct 4, 2023
jborean93 added a commit to jborean93/ansible that referenced this pull request Oct 4, 2023
jborean93 added a commit to jborean93/ansible that referenced this pull request Oct 4, 2023
jborean93 added a commit to jborean93/ansible that referenced this pull request Oct 4, 2023
mattclay pushed a commit that referenced this pull request Oct 4, 2023
mattclay pushed a commit that referenced this pull request Oct 4, 2023
mattclay pushed a commit that referenced this pull request Oct 4, 2023
mattclay pushed a commit that referenced this pull request Oct 4, 2023
@ansible ansible locked and limited conversation to collaborators Dec 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
test This PR relates to tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants