New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 681979 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 615389



Sign in to add a comment

schedtune governor causes platform_FilePerms test to fail

Project Member Reported by briannorris@chromium.org, Jan 17 2017

Issue description

Kevin release builders have been failing:

https://uberchromegw.corp.google.com/i/chromeos/builders/kevin-release/builds/776

It comes down to this:

01/17 07:51:51.237 ERROR|platform_FilePerms:0310| No expectations entry for "/sys/fs/cgroup/schedtune" with info "{'device': 'cgroup', 'type': 'cgroup', 'options': ['rw', 'nosuid', 'nodev', 'noexec', 'relatime', 'schedtune']}"
01/17 07:51:52.852 ERROR|          parallel:0026| child process failed

Looks like we missed the 'schedtune' mount option:

# mount | grep schedtune
cgroup on /sys/fs/cgroup/schedtune/chrome type cgroup (rw,nosuid,nodev,noexec,relatime,schedtune)
 

Comment 1 by dtor@google.com, Jan 17 2017

Ah, we do have it, but in different place. Meet to drop "chrome" from mount point in the test
Wait, did the mountpoint move in M-57 or something? M-56 has it at:

/sys/fs/cgroup/schedtune/chrome

But I see the test is complaining about:

/sys/fs/cgroup/schedtune
Yeah it moved in M-57, we need to backport those changes to 56 -- thought that was already under way?
Cc: adlr@chromium.org dshi@chromium.org abhishekbh@chromium.org
 Issue 681994  has been merged into this issue.
Project Member

Comment 6 by bugdroid1@chromium.org, Jan 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/c018197c8f4bd6843a0b2ca1f10157dbb3ff6e48

commit c018197c8f4bd6843a0b2ca1f10157dbb3ff6e48
Author: Brian Norris <briannorris@chromium.org>
Date: Tue Jan 17 21:43:42 2017

platform_FilePerms: this mountpoint moved

BUG= chromium:681979 
TEST=`test_that platform_FilePerms` on latest

Change-Id: I449096167267e220f4a8dc1e45f4b4ca275892f7
Signed-off-by: Brian Norris <briannorris@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/428551
Tested-by: Dmitry Torokhov <dtor@chromium.org>
Reviewed-by: Dmitry Torokhov <dtor@chromium.org>

[modify] https://crrev.com/c018197c8f4bd6843a0b2ca1f10157dbb3ff6e48/client/site_tests/platform_FilePerms/platform_FilePerms.py

Status: Fixed (was: Started)
Project Member

Comment 8 by bugdroid1@chromium.org, Jan 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/7f5b0f1434af5390946bd9f9424448d9d4442750

commit 7f5b0f1434af5390946bd9f9424448d9d4442750
Author: Bernie Thompson <bhthompson@google.com>
Date: Mon Jan 23 18:17:39 2017

Add kevin-paladin as experimental

BUG= chromium:681979 
TEST=None

Change-Id: Ia9b6d8a07145efbbfa4faa8093814ca6a16231f6
Reviewed-on: https://chromium-review.googlesource.com/431178
Commit-Ready: Bernie Thompson <bhthompson@chromium.org>
Tested-by: Bernie Thompson <bhthompson@chromium.org>
Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
Reviewed-by: Brian Norris <briannorris@chromium.org>

[modify] https://crrev.com/7f5b0f1434af5390946bd9f9424448d9d4442750/cbuildbot/config_dump.json
[modify] https://crrev.com/7f5b0f1434af5390946bd9f9424448d9d4442750/cbuildbot/waterfall_layout_dump.txt
[modify] https://crrev.com/7f5b0f1434af5390946bd9f9424448d9d4442750/cbuildbot/chromeos_config.py

Project Member

Comment 9 by bugdroid1@chromium.org, Jan 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/7f5b0f1434af5390946bd9f9424448d9d4442750

commit 7f5b0f1434af5390946bd9f9424448d9d4442750
Author: Bernie Thompson <bhthompson@google.com>
Date: Mon Jan 23 18:17:39 2017

Add kevin-paladin as experimental

BUG= chromium:681979 
TEST=None

Change-Id: Ia9b6d8a07145efbbfa4faa8093814ca6a16231f6
Reviewed-on: https://chromium-review.googlesource.com/431178
Commit-Ready: Bernie Thompson <bhthompson@chromium.org>
Tested-by: Bernie Thompson <bhthompson@chromium.org>
Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
Reviewed-by: Brian Norris <briannorris@chromium.org>

[modify] https://crrev.com/7f5b0f1434af5390946bd9f9424448d9d4442750/cbuildbot/config_dump.json
[modify] https://crrev.com/7f5b0f1434af5390946bd9f9424448d9d4442750/cbuildbot/waterfall_layout_dump.txt
[modify] https://crrev.com/7f5b0f1434af5390946bd9f9424448d9d4442750/cbuildbot/chromeos_config.py

Labels: VerifyIn-61

Comment 11 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment