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

Issue 788778 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

service unitests failing on debian after 4a91dec

Project Member Reported by tsepez@chromium.org, Nov 27 2017

Issue description

An external reporter reports:

As I said before, after this change
https://chromium-review.googlesource.com/c/chromium/src/+/688484, I'm
unable to run services_unittests locally for ChromeOS target with mus.

Some of the tests fail to fork -

I ran it like:
./out/CrosOzone/services_unittests --single-process-tests
--gtest_filter=ResourceCoordinatorTest.ResourceCoordinatorInstantiate

and get launch_posix.cc(379)] fork: Operation not permitted (1).
 
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5c2a4b6179273f240e66cde6d93e3e5a1933d466

commit 5c2a4b6179273f240e66cde6d93e3e5a1933d466
Author: Tom Sepez <tsepez@chromium.org>
Date: Tue Nov 28 00:36:53 2017

Revert passing sandbox type to service manager.

Reason for reversion
  https://bugs.chromium.org/p/chromium/issues/detail?id=788778

The functionality isn't required -- yet -- and breaks some tests. Try
to get the tests working again.

Bug:  788778 
Change-Id: I1caea69411c3210409b66e083c10221bb3244881
Reviewed-on: https://chromium-review.googlesource.com/790933
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#519479}
[modify] https://crrev.com/5c2a4b6179273f240e66cde6d93e3e5a1933d466/services/service_manager/service_manager.cc

Comment 3 by msi...@igalia.com, Nov 28 2017

Cc: toniki...@chromium.org msi...@igalia.com

Comment 4 by msi...@igalia.com, Nov 28 2017

Cc: -toniki...@chromium.org toniki...@igalia.com
Status: Fixed (was: Assigned)

Sign in to add a comment