Pewarta Proxy

📋 Server Request & Proxy Logs

Loading active proxy settings...
Developed by Khairul Adha (adhexa.id)
Mengambil log server...

⚙️ Pengaturan Proxy Target

Skip to content

gh-142654: show the clear error message when sampling on an unknown PID #142655

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.

Sign up for GitHub

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

Merged
pablogsal merged 10 commits into from
Dec 17, 2025
Merged

gh-142654: show the clear error message when sampling on an unknown PID #142655

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
7024bd1
gh-142654: show the clear error message when sampling on an unknown PID
kemingy Dec 13, 2025
b1a5082
only catch the RuntimeError, leave the PermissionError for downstream
kemingy Dec 13, 2025
cfea35b
catch the PermissionError for macOS
kemingy Dec 13, 2025
ab9c0c6
check if the pid is running before attaching
kemingy Dec 15, 2025
a1c66e5
resolve conflict
kemingy Dec 15, 2025
0a1f140
Merge branch 'main' into gh-142654
kemingy Dec 15, 2025
dcad05a
fix sampling test failure due to the _is_process_running check
kemingy Dec 15, 2025
4d55401
add custom sampling errors
kemingy Dec 16, 2025
5b5ad2f
use os.kill(pid, 0) for all the posix OS
kemingy Dec 16, 2025
7b4bd69
preserve tracebacks, account for permission error, small fixes
pablogsal Dec 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
catch the PermissionError for macOS 
Signed-off-by: Keming <kemingy94@gmail.com>
  • Loading branch information
@kemingy
kemingy committed Dec 13, 2025
commit cfea35b7824f8c6f6abb03ceb03baff68621c328
2 changes: 1 addition & 1 deletion Lib/test/test_profiling/test_sampling_profiler/test_integration.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -635,7 +635,7 @@ def test_sample_target_module(self):
)
class TestSampleProfilerErrorHandling(unittest.TestCase):
def test_invalid_pid(self):
with self.assertRaises(SystemExit):
with self.assertRaises((SystemExit, PermissionError)):
collector = PstatsCollector(sample_interval_usec=100, skip_idle=False)
profiling.sampling.sample.sample(-1, collector, duration_sec=1)

Expand Down
Loading