-
Notifications
You must be signed in to change notification settings - Fork 291
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix access denied from psutil on macOS #1985
Conversation
704fe42
to
d6e3bca
Compare
.github/workflows/integration.yml
Outdated
- uses: actions/checkout@v2 | ||
- uses: actions/checkout@v4 | ||
- name: Install docker-compose | ||
run: sudo apt-get install -y docker-compose |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This didn't really work, and the issue is intermittent. Let's leave it out of this PR and see if a simple retry gets us past the issue for now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was trying to fix it fast, but right, dropping it for now, am not sure missing docker-compose is gonna be fixed by retrying :-D
d6e3bca
to
fe2cab3
Compare
the integration failure is unrelated |
On macOS dispatcher while trying to go through process list gets stuck on some of the system processes like launchd, logd, systemstats, etc. and quites unexpectedly with PermissionError and psutil.AccessDenied exceptions. Fixes: https://tracker.ceph.com/issues/67313 Signed-off-by: Kyr Shatskyy <[email protected]>
Fix unhandled exception psutilAccessDenied in process_matches_run(). Signed-off-by: Kyr Shatskyy <[email protected]>
fe2cab3
to
d218c61
Compare
On macOS dispatcher while trying to go through process list gets stuck
on some of the system processes like launchd, logd, systemstats, etc.
and quites unexpectedly with PermissionError and psutil.AccessDenied
exceptions.
Fixes: https://tracker.ceph.com/issues/67313