Skip to content

Commit

Permalink
Merge branch '3006.x' into 3006_x_manjaro_arm_correct_os_family
Browse files Browse the repository at this point in the history
  • Loading branch information
dmurphy18 authored Aug 12, 2024
2 parents ba39288 + 092a79c commit dac222e
Show file tree
Hide file tree
Showing 37 changed files with 155 additions and 66 deletions.
1 change: 1 addition & 0 deletions changelog/66467.removed.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Remove psutil_compat.py file, which should have been removed when RHEL 6 EOL
1 change: 1 addition & 0 deletions changelog/66726.fixed.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Clean up multiprocessing file handles on minion
2 changes: 2 additions & 0 deletions requirements/pytest.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,5 @@ flaky
more-itertools
pyfakefs
trustme
pytest-skip-markers >= 1.5.2 ; python_version >= '3.8'
pytest-skip-markers <= 1.5.1 ; python_version < '3.8'
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.10/cloud.txt
Original file line number Diff line number Diff line change
Expand Up @@ -482,9 +482,10 @@ pytest-shell-utilities==1.8.0
# via
# -c requirements/static/ci/py3.10/linux.txt
# pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -c requirements/static/ci/py3.10/linux.txt
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.10/darwin.txt
Original file line number Diff line number Diff line change
Expand Up @@ -337,8 +337,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.10/freebsd.txt
Original file line number Diff line number Diff line change
Expand Up @@ -330,8 +330,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.10/linux.txt
Original file line number Diff line number Diff line change
Expand Up @@ -347,8 +347,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.10/windows.txt
Original file line number Diff line number Diff line change
Expand Up @@ -308,8 +308,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.11/cloud.txt
Original file line number Diff line number Diff line change
Expand Up @@ -446,9 +446,10 @@ pytest-shell-utilities==1.8.0
# via
# -c requirements/static/ci/py3.11/linux.txt
# pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -c requirements/static/ci/py3.11/linux.txt
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.11/darwin.txt
Original file line number Diff line number Diff line change
Expand Up @@ -308,8 +308,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.11/freebsd.txt
Original file line number Diff line number Diff line change
Expand Up @@ -307,8 +307,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.11/linux.txt
Original file line number Diff line number Diff line change
Expand Up @@ -324,8 +324,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.11/windows.txt
Original file line number Diff line number Diff line change
Expand Up @@ -304,8 +304,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.12/cloud.txt
Original file line number Diff line number Diff line change
Expand Up @@ -446,9 +446,10 @@ pytest-shell-utilities==1.8.0
# via
# -c requirements/static/ci/py3.12/linux.txt
# pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -c requirements/static/ci/py3.12/linux.txt
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.12/darwin.txt
Original file line number Diff line number Diff line change
Expand Up @@ -308,8 +308,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.12/freebsd.txt
Original file line number Diff line number Diff line change
Expand Up @@ -307,8 +307,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.12/linux.txt
Original file line number Diff line number Diff line change
Expand Up @@ -324,8 +324,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.12/windows.txt
Original file line number Diff line number Diff line change
Expand Up @@ -304,8 +304,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.7/cloud.txt
Original file line number Diff line number Diff line change
Expand Up @@ -530,9 +530,10 @@ pytest-shell-utilities==1.8.0
# via
# -c requirements/static/ci/py3.7/linux.txt
# pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.0 ; python_version < "3.8"
# via
# -c requirements/static/ci/py3.7/linux.txt
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.7/freebsd.txt
Original file line number Diff line number Diff line change
Expand Up @@ -371,8 +371,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.0 ; python_version < "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.7/linux.txt
Original file line number Diff line number Diff line change
Expand Up @@ -383,8 +383,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.0 ; python_version < "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.7/windows.txt
Original file line number Diff line number Diff line change
Expand Up @@ -323,8 +323,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.0 ; python_version < "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.8/cloud.txt
Original file line number Diff line number Diff line change
Expand Up @@ -516,9 +516,10 @@ pytest-shell-utilities==1.8.0
# via
# -c requirements/static/ci/py3.8/linux.txt
# pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -c requirements/static/ci/py3.8/linux.txt
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.8/freebsd.txt
Original file line number Diff line number Diff line change
Expand Up @@ -357,8 +357,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.8/linux.txt
Original file line number Diff line number Diff line change
Expand Up @@ -369,8 +369,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.8/windows.txt
Original file line number Diff line number Diff line change
Expand Up @@ -309,8 +309,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.9/cloud.txt
Original file line number Diff line number Diff line change
Expand Up @@ -518,9 +518,10 @@ pytest-shell-utilities==1.8.0
# via
# -c requirements/static/ci/py3.9/linux.txt
# pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -c requirements/static/ci/py3.9/linux.txt
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.9/darwin.txt
Original file line number Diff line number Diff line change
Expand Up @@ -366,8 +366,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.9/freebsd.txt
Original file line number Diff line number Diff line change
Expand Up @@ -359,8 +359,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.9/linux.txt
Original file line number Diff line number Diff line change
Expand Up @@ -369,8 +369,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
3 changes: 2 additions & 1 deletion requirements/static/ci/py3.9/windows.txt
Original file line number Diff line number Diff line change
Expand Up @@ -310,8 +310,9 @@ pytest-salt-factories==1.0.1
# via -r requirements/pytest.txt
pytest-shell-utilities==1.8.0
# via pytest-salt-factories
pytest-skip-markers==1.5.0
pytest-skip-markers==1.5.2 ; python_version >= "3.8"
# via
# -r requirements/pytest.txt
# pytest-salt-factories
# pytest-shell-utilities
# pytest-system-statistics
Expand Down
6 changes: 5 additions & 1 deletion salt/utils/platform.py
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,11 @@ def is_aarch64():
"""
Simple function to return if host is AArch64 or not
"""
return platform.machine().startswith("aarch64")
if is_darwin():
# Allow for MacOS Arm64 platform returning differently from Linux
return platform.machine().startswith("arm64")
else:
return platform.machine().startswith("aarch64")


def spawning_platform():
Expand Down
15 changes: 11 additions & 4 deletions salt/utils/process.py
Original file line number Diff line number Diff line change
Expand Up @@ -1169,10 +1169,17 @@ def add(self, proc):

def cleanup(self):
with self.lock:
for proc in self.processes:
if proc.is_alive():
continue
proc.join()
for proc in self.processes[:]:
proc.join(0.01)
if hasattr(proc, "exitcode"):
# Only processes have exitcode and a close method, threads
# do not.
if proc.exitcode is None:
continue
proc.close()
else:
if proc.is_alive():
continue
self.processes.remove(proc)
self.count -= 1
log.debug("Subprocess %s cleaned up", proc.name)
Expand Down
21 changes: 0 additions & 21 deletions salt/utils/psutil_compat.py

This file was deleted.

Loading

0 comments on commit dac222e

Please sign in to comment.