tree 3be99fd3bb53c7a559473c23e2d20d4fa6dbd280
parent 3c900c1fbffb33507a04760a7d73ffff17f0bf3c
author Gwendal Grignou <gwendal@chromium.org> 1636133766 -0700
committer Commit Bot <commit-bot@chromium.org> 1639047534 +0000

hooks: oom_score: Remove false positive message

Message "Please remove ${package}from known_bad_oom" would be printed
even when the ${package} still have .conf files with oom score line
missing.
Fix return logic in check_oom() to always return 1 on missing score line
and adapt test in check() accordingly.

BUG=b:205304293
TEST=Check with chromeos-init the message is not printed anymore.
Remove chromeos-init for known bad oom list and observe error.

Change-Id: I0f15b870dd19cd6d805ae7b2b3a9843e79586c57
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosutils/+/3264358
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Gwendal Grignou <gwendal@chromium.org>
Commit-Queue: Gwendal Grignou <gwendal@chromium.org>