summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatt Clay <matt@mystile.com>2023-02-03 14:22:57 -0800
committerGitHub <noreply@github.com>2023-02-03 14:22:57 -0800
commitb3307f1aa738372d0fb4f5ee5aaed6b3fa436bd6 (patch)
tree89a85693e82653b3c4f002c2e61ccc557c05a7d5
parent639e3e30d7765546d4b337da562edeab4349df89 (diff)
downloadansible-b3307f1aa738372d0fb4f5ee5aaed6b3fa436bd6.tar.gz
Remove unused stdlib imports from module_utils (#79905)
-rw-r--r--changelogs/fragments/unused-imports-module-utils-stdlib.yml4
-rw-r--r--lib/ansible/module_utils/common/file.py5
-rw-r--r--lib/ansible/module_utils/compat/selectors.py1
-rw-r--r--lib/ansible/module_utils/facts/network/iscsi.py1
-rw-r--r--lib/ansible/module_utils/facts/network/nvme.py1
-rw-r--r--lib/ansible/module_utils/yumdnf.py1
6 files changed, 4 insertions, 9 deletions
diff --git a/changelogs/fragments/unused-imports-module-utils-stdlib.yml b/changelogs/fragments/unused-imports-module-utils-stdlib.yml
new file mode 100644
index 0000000000..99f4294aa7
--- /dev/null
+++ b/changelogs/fragments/unused-imports-module-utils-stdlib.yml
@@ -0,0 +1,4 @@
+minor_changes:
+ - >
+ Remove unused Python stdlib imports from module_utils which were not present for backwards compatibility in:
+ common.file, compat.selectors, facts.network.iscsi, facts.network.nvme, yumdnf
diff --git a/lib/ansible/module_utils/common/file.py b/lib/ansible/module_utils/common/file.py
index 045565be63..a92c88ab3b 100644
--- a/lib/ansible/module_utils/common/file.py
+++ b/lib/ansible/module_utils/common/file.py
@@ -4,15 +4,10 @@
from __future__ import (absolute_import, division, print_function)
__metaclass__ = type
-import errno
import os
import stat
import re
-import pwd
-import grp
import time
-import shutil
-import traceback
import fcntl
import sys
diff --git a/lib/ansible/module_utils/compat/selectors.py b/lib/ansible/module_utils/compat/selectors.py
index 93ffc62629..0d26ea8ab2 100644
--- a/lib/ansible/module_utils/compat/selectors.py
+++ b/lib/ansible/module_utils/compat/selectors.py
@@ -35,7 +35,6 @@ _BUNDLED_METADATA = {"pypi_name": "selectors2", "version": "1.1.1", "version_con
# Fix use of OSError exception for py3 and use the wrapper of kqueue.control so retries of
# interrupted syscalls work with kqueue
-import os.path
import sys
import types
diff --git a/lib/ansible/module_utils/facts/network/iscsi.py b/lib/ansible/module_utils/facts/network/iscsi.py
index 2bb9383423..ef5ac398a9 100644
--- a/lib/ansible/module_utils/facts/network/iscsi.py
+++ b/lib/ansible/module_utils/facts/network/iscsi.py
@@ -19,7 +19,6 @@ from __future__ import (absolute_import, division, print_function)
__metaclass__ = type
import sys
-import subprocess
import ansible.module_utils.compat.typing as t
diff --git a/lib/ansible/module_utils/facts/network/nvme.py b/lib/ansible/module_utils/facts/network/nvme.py
index febd0abb38..1d759566c9 100644
--- a/lib/ansible/module_utils/facts/network/nvme.py
+++ b/lib/ansible/module_utils/facts/network/nvme.py
@@ -19,7 +19,6 @@ from __future__ import (absolute_import, division, print_function)
__metaclass__ = type
import sys
-import subprocess
import ansible.module_utils.compat.typing as t
diff --git a/lib/ansible/module_utils/yumdnf.py b/lib/ansible/module_utils/yumdnf.py
index 27bc8ba2d2..7eb9d5fc2b 100644
--- a/lib/ansible/module_utils/yumdnf.py
+++ b/lib/ansible/module_utils/yumdnf.py
@@ -15,7 +15,6 @@ __metaclass__ = type
import os
import time
import glob
-import tempfile
from abc import ABCMeta, abstractmethod
from ansible.module_utils.six import with_metaclass