Port platform_FilePerms to Tast |
|
Issue descriptionI'm porting the platform_FilePerms Autotest test to Tast. The Autotest test both checks permission/ownership of various directories and parses /etc/mtab and /var/log/mount_options.log and compares their mounts against a baseline. I'm planning to split it into separate security.MountPerms and security.Mtab tests, since it's really checking two different things and since it's pretty long.
,
Dec 18
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/tast-tests/+/3bc5626cb6fb2fa05b36c3e8c50d552192374eb8 commit 3bc5626cb6fb2fa05b36c3e8c50d552192374eb8 Author: Daniel Erat <derat@chromium.org> Date: Tue Dec 18 08:42:43 2018 tast-tests: Add security.SystemDirs. Add a new security.SystemDirs test that replicates logic from the platform_FilePerms Autotest test to check ownership and permissions of system directories. BUG=chromium:915539 TEST=ran new test on various DUTs Change-Id: I88d66df91d0a343fc1a5d3d4860aa96943368c47 Reviewed-on: https://chromium-review.googlesource.com/1378919 Commit-Ready: Dan Erat <derat@chromium.org> Tested-by: Dan Erat <derat@chromium.org> Reviewed-by: Dan Erat <derat@chromium.org> Reviewed-by: Shuhei Takahashi <nya@chromium.org> Reviewed-by: Hidehiko Abe <hidehiko@chromium.org> [add] https://crrev.com/3bc5626cb6fb2fa05b36c3e8c50d552192374eb8/src/chromiumos/tast/local/bundles/cros/security/system_dirs.go
,
Dec 28
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/tast-tests/+/0833f1856e1d52995e822f8734d56ad20a3a4a03 commit 0833f1856e1d52995e822f8734d56ad20a3a4a03 Author: Daniel Erat <derat@chromium.org> Date: Fri Dec 28 21:01:26 2018 tast-tests: Add security.Mtab. Add a new security.Mtab test that replicates logic from the platform_FilePerms Autotest test to compare /etc/mtab and /var/log/mount_options.log against a baseline. BUG=chromium:915539 TEST=ran on DUTs with and without rootfs verification, and with user session started and stopped CQ-DEPEND=CL:1388087 Change-Id: Idb8a9db8062cc2afc3647b22e8e9798ea17d7e82 Reviewed-on: https://chromium-review.googlesource.com/1378920 Commit-Ready: Dan Erat <derat@chromium.org> Tested-by: Dan Erat <derat@chromium.org> Reviewed-by: Shuhei Takahashi <nya@chromium.org> [add] https://crrev.com/0833f1856e1d52995e822f8734d56ad20a3a4a03/src/chromiumos/tast/local/bundles/cros/security/mtab.go
,
Dec 28
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/tast-tests/+/0c3bcdc1b000a6cc911bbd27d8a07ee6edfdeb96 commit 0c3bcdc1b000a6cc911bbd27d8a07ee6edfdeb96 Author: Daniel Erat <derat@chromium.org> Date: Fri Dec 28 21:01:25 2018 tast-tests: Add security.SystemDirs to CQ. Add the security.SystemDirs local test to the Chrome OS Commit Queue. BUG=chromium:915539 TEST=no failures over last 14 days on release builders Change-Id: Ia732d43b5ca0e5ff20bd9daa5283cd0b2c75305a Reviewed-on: https://chromium-review.googlesource.com/1390948 Commit-Ready: Dan Erat <derat@chromium.org> Tested-by: Dan Erat <derat@chromium.org> Reviewed-by: Shuhei Takahashi <nya@chromium.org> [modify] https://crrev.com/0c3bcdc1b000a6cc911bbd27d8a07ee6edfdeb96/src/chromiumos/tast/local/bundles/cros/security/system_dirs.go
,
Jan 12
Would you mind checking if security.Mtab seems to be stable on release builders now? :-) Thanks!
,
Jan 13
Passed except one board/model. (The details can be found at b:122518140). |
|
►
Sign in to add a comment |
|
Comment 1 by bugdroid1@chromium.org
, Dec 18