yujunjun/haystack
1
0
Fork 0
mirror of https://github.com/deepset-ai/haystack.git synced 2025-12-02 18:06:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix ResourceWarning: unclosed file when using telemetry (#8884)

Browse Source
This commit is contained in:
matthias plasser 2025-02-20 10:41:35 +01:00 committed by GitHub
parent 8cafcddb00
commit 7c17ca0405
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
haystack/telemetry/_environment.py
View File

@ -16,6 +16,11 @@ logger = logging.getLogger(__name__)
_IS_DOCKER_CACHE = None
def _str_in_any_line_of_file(s: str, path: str) -> bool:
with open(path) as f:
return any(s in line for line in f)
def _in_podman() -> bool:
"""
Check if the code is running in a Podman container.
@ -41,7 +46,7 @@ def _has_docker_cgroup_v1() -> bool:
This only works with cgroups v1.
"""
path = "/proc/self/cgroup" # 'self' should be always symlinked to the actual PID
return os.path.isfile(path) and any("docker" in line for line in open(path))
return os.path.isfile(path) and _str_in_any_line_of_file("docker", path)
def _has_docker_cgroup_v2() -> bool:
@ -51,7 +56,7 @@ def _has_docker_cgroup_v2() -> bool:
inspired from: https://github.com/jenkinsci/docker-workflow-plugin/blob/master/src/main/java/org/jenkinsci/plugins/docker/workflow/client/DockerClient.java
"""
path = "/proc/self/mountinfo" # 'self' should be always symlinked to the actual PID
return os.path.isfile(path) and any("/docker/containers/" in line for line in open(path))
return os.path.isfile(path) and _str_in_any_line_of_file("/docker/containers/", path)
def _is_containerized() -> Optional[bool]: