tests: Allow read-only root filesystem
https://crrev.com/c/1869501 introduced a check to test that the root dir
permissions were 755. This is too restrictive and doesn't work in Linux
distributions with a read-only root filesystem.
Since the referenced bug was about / being writable by all, we should
just error on that case.
BUG=chromium:1015450
TEST=CQ passes
Change-Id: I3455ac7f910c56c4c097adf89b58e647d8f78cd1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3613808
Tested-by: Henrique Ferreiro <hferreiro@igalia.com>
Commit-Queue: Henrique Ferreiro <hferreiro@igalia.com>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Auto-Submit: Henrique Ferreiro <hferreiro@igalia.com>
diff --git a/scripts/run_tests.py b/scripts/run_tests.py
index c9f78e0..d7fa566 100644
--- a/scripts/run_tests.py
+++ b/scripts/run_tests.py
@@ -71,10 +71,10 @@
# Check the environment. https://crbug.com/1015450
st = os.stat('/')
- if st.st_mode & 0o7777 != 0o755:
+ if st.st_mode & 0o007 != 0o005:
cros_build_lib.Die(
f'The root directory has broken permissions: {st.st_mode:o}\n'
- 'Fix with: sudo chmod 755 /')
+ 'Fix with: sudo chmod o+rx-w /')
if st.st_uid or st.st_gid:
cros_build_lib.Die(
f'The root directory has broken ownership: {st.st_uid}:{st.st_gid}'