Repository: aurora Updated Branches: refs/heads/master b75dfb838 -> 6eda8c8af
Log process sampling failures with debug severity The observer's logs consist of lots of warnings about being unable to find PIDs. This is expected when running with the PID isolator, or when checkpoints are out of date (e.g. after processes were killed by the OOM). W0116 14:42:54.694221 3253 process_collector_psutil.py:75] Error during process sampling: psutil.NoSuchProcess process no longer exists (pid=27727) W0116 14:42:54.717905 3253 process_collector_psutil.py:42] Error during process sampling [pid=10960]: psutil.NoSuchProcess process no longer exists (pid=10960) W0116 14:42:54.718089 3253 process_collector_psutil.py:75] Error during process sampling: psutil.NoSuchProcess process no longer exists (pid=10960) W0116 14:42:54.718245 3253 process_collector_psutil.py:42] Error during process sampling [pid=10026]: psutil.NoSuchProcess process no longer exists (pid=10026) W0116 14:42:54.718334 3253 process_collector_psutil.py:75] Error during process sampling: psutil.NoSuchProcess process no longer exists (pid=10026) This change adopts the proposal of David Robinson to decrease the severity level to debug. Bugs closed: AURORA-1541 Reviewed at https://reviews.apache.org/r/55578/ Project: http://git-wip-us.apache.org/repos/asf/aurora/repo Commit: http://git-wip-us.apache.org/repos/asf/aurora/commit/6eda8c8a Tree: http://git-wip-us.apache.org/repos/asf/aurora/tree/6eda8c8a Diff: http://git-wip-us.apache.org/repos/asf/aurora/diff/6eda8c8a Branch: refs/heads/master Commit: 6eda8c8af70a83da3fd6f8fc2de4aa539ad44199 Parents: b75dfb8 Author: Stephan Erb <s...@apache.org> Authored: Tue Jan 17 21:53:18 2017 +0100 Committer: Stephan Erb <s...@apache.org> Committed: Tue Jan 17 21:53:18 2017 +0100 ---------------------------------------------------------------------- .../python/apache/thermos/monitoring/process_collector_psutil.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/aurora/blob/6eda8c8a/src/main/python/apache/thermos/monitoring/process_collector_psutil.py ---------------------------------------------------------------------- diff --git a/src/main/python/apache/thermos/monitoring/process_collector_psutil.py b/src/main/python/apache/thermos/monitoring/process_collector_psutil.py index 4a1b159..3000e95 100644 --- a/src/main/python/apache/thermos/monitoring/process_collector_psutil.py +++ b/src/main/python/apache/thermos/monitoring/process_collector_psutil.py @@ -39,7 +39,7 @@ def process_to_sample(process): threads = process.num_threads() return ProcessSample(rate, user, system, rss, vms, nice, status, threads) except (AccessDenied, NoSuchProcess) as e: - log.warning('Error during process sampling [pid=%s]: %s' % (process.pid, e)) + log.debug('Error during process sampling [pid=%s]: %s' % (process.pid, e)) return ProcessSample.empty() @@ -72,7 +72,7 @@ class ProcessTreeCollector(object): new_samples[self._pid] = parent_sample except (IOError, PsutilError) as e: - log.warning('Error during process sampling: %s' % e) + log.debug('Error during process sampling: %s' % e) self._sample = ProcessSample.empty() self._rate = 0.0