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.

Sign up for GitHub

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

Jump to bottom

[release_2.1] Write auto-created private data dirs to pytest dir instead of /tmp (#916) #941

Merged
merged 1 commit into from
Dec 14, 2021
Merged

[release_2.1] Write auto-created private data dirs to pytest dir instead of /tmp (#916) #941

merged 1 commit into from
Dec 14, 2021

Commits on Dec 14, 2021

  1. [release_2.1] Write auto-created private data dirs to pytest dir inst… 

    …ead of /tmp (ansible#916)

Write auto-created private data dirs to pytest dir instead of /tmp

I briefly discussed this option with @samdoran in ansible#901
After this patch, we create directories that look like:
/tmp/pytest-of-alancoding/pytest-0/popen-gw2/test_no_dst_all_dirs_src__dir_0/.ansible-runner-2fpvb4uo

This is as opposed to leaving around directories like /tmp/.ansible-runner-2fpvb4uo. There are a lot of these, like ~1,700 created with a single test run excluding the integration tests.
There are still a few directories created in /tmp after this, but it's a handful, not a flood.

Reviewed-by: Sam Doran <github@samdoran.com>
Reviewed-by: None <None>
(cherry picked from commit 7f9de03)

Co-authored-by: Alan Rominger <arominge@redhat.com>
    @AlanCoding @samdoran
    AlanCoding authored and samdoran committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    c5812cc View commit details
    Browse the repository at this point in the history