Skip to content

fix: Retry cgroup discovery after a failed /proc read - #21

Merged
Pijukatel merged 2 commits into
apify:masterfrom
Mantisus:fix-discovery-cache
Nov 9, 2026
Merged

Pijukatel merged 2 commits into
apify:masterfrom
Mantisus:fix-discovery-cache

Conversation

@Mantisus

@Mantisus Mantisus commented Nov 6, 2026

Copy link
Copy Markdown
Collaborator

Description

Discovery reads /proc/self/mountinfo and /proc/self/cgroup once and caches the answer. Any OSError or ValueError from those reads became "no mechanism here", and that answer was cached as well. One EMFILE on the first lookup therefore left every limit at None, with *_METRICS_UNAVAILABLE, until clear_cache(). That held even after file descriptors were free again.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Mantisus Mantisus self-assigned this Nov 6, 2026
@Mantisus
Mantisus requested a review from Pijukatel November 6, 2026 15:38
@Pijukatel
Pijukatel merged commit f599a20 into apify:master Nov 9, 2026
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants