From 7da4a992391230cfaad10fbf87dbb69f920f943d Mon Sep 17 00:00:00 2001 From: Jordan Borean Date: Thu, 13 Apr 2023 06:09:19 +1000 Subject: Improve Ansible.Basic.cs tempdir uniqueness (#80328) (#80331) * Improve Ansible.Basic.cs tempdir uniqueness The current tempdir naming scheme can result in the same name if the remote worker starts at the same time as another. By using the process id it should add enough uniqueness to avoid this situation. * Fix sanity issues * Fix up compile issue on older hosts (cherry picked from commit fb6b90fe4255e9995706905e2a9cde205648c0d2) --- changelogs/fragments/ansible-basic-tmpdir-uniqueness.yml | 2 ++ lib/ansible/module_utils/csharp/Ansible.Basic.cs | 3 ++- 2 files changed, 4 insertions(+), 1 deletion(-) create mode 100644 changelogs/fragments/ansible-basic-tmpdir-uniqueness.yml diff --git a/changelogs/fragments/ansible-basic-tmpdir-uniqueness.yml b/changelogs/fragments/ansible-basic-tmpdir-uniqueness.yml new file mode 100644 index 0000000000..0240949563 --- /dev/null +++ b/changelogs/fragments/ansible-basic-tmpdir-uniqueness.yml @@ -0,0 +1,2 @@ +bugfixes: +- Windows - Ensure the module temp directory contains more unique values to avoid conflicts with concurrent runs - https://github.com/ansible/ansible/issues/80294 diff --git a/lib/ansible/module_utils/csharp/Ansible.Basic.cs b/lib/ansible/module_utils/csharp/Ansible.Basic.cs index c68281ef1b..16480992e8 100644 --- a/lib/ansible/module_utils/csharp/Ansible.Basic.cs +++ b/lib/ansible/module_utils/csharp/Ansible.Basic.cs @@ -176,7 +176,8 @@ namespace Ansible.Basic } string dateTime = DateTime.Now.ToFileTime().ToString(); - string dirName = String.Format("ansible-moduletmp-{0}-{1}", dateTime, new Random().Next(0, int.MaxValue)); + string dirName = String.Format("ansible-moduletmp-{0}-{1}-{2}", dateTime, System.Diagnostics.Process.GetCurrentProcess().Id, + new Random().Next(0, int.MaxValue)); string newTmpdir = Path.Combine(baseDir, dirName); #if CORECLR DirectoryInfo tmpdirInfo = Directory.CreateDirectory(newTmpdir); -- cgit v1.2.1