Revert "Reland "shill: start running shill process tree in a minijail""
This reverts commit 8751e2a8474c895d11d56ca45e4f71e808b25c27.
Reason for revert: Reverting this so the revert can be merged back into M69. We believe we have
all the networking stuff fixed as of
but are doing this revert just to give us some breathing room by punting landing the shill
sandboxing from M69 to M70, in case there's anything thats still broken that we don't know about.
Original change's description:
> Reland "shill: start running shill process tree in a minijail"
> This is a reland of aad3747780f4f81a971878773b79635e1b3dc001
> The 2 known issues which caused problems the first time around have been
> addressed by CL:1130017 and CL:1130474. I've done some more thorough
> testing (including with elm board, which is arm processor and marvell wifi)
> and haven't discovered any new issues.
> Original change's description:
> > shill: start running shill process tree in a minijail
> > Kick off shill sandboxing (we'll see if it sticks). This simple change to
> > the init script as well as the autotest change to
> > security_SandboxedServices are the only 2 things that need to be
> > reverted to go back to running the shill process tree as root.
> > CQ-DEPEND=CL:1082958
> > BUG=chromium:649417
> > TEST=observed shill process tree running as non-root. no autotests
> > break.
> > Change-Id: Ice5ed2cbbc64281ba8f136e5157023ff355245d6
> > Reviewed-on: https://chromium-review.googlesource.com/1087540
> > Commit-Ready: Micah Morton <email@example.com>
> > Tested-by: Micah Morton <firstname.lastname@example.org>
> > Reviewed-by: Jorge Lucangeli Obes <email@example.com>
> Bug: chromium:649417
> Change-Id: I3fc5506ee8886db8d6a4d7d155216aba9e20b7f8
> Reviewed-on: https://chromium-review.googlesource.com/1132174
> Commit-Ready: ChromeOS CL Exonerator Bot <firstname.lastname@example.org>
> Tested-by: Micah Morton <email@example.com>
> Reviewed-by: Jorge Lucangeli Obes <firstname.lastname@example.org>
> Reviewed-by: Mike Frysinger <email@example.com>
# Not skipping CQ checks because original CL landed > 1 day ago.
Bug: chromium:649417, chromium:870076
Commit-Ready: Micah Morton <firstname.lastname@example.org>
Tested-by: Micah Morton <email@example.com>
Reviewed-by: Micah Morton <firstname.lastname@example.org>
Reviewed-by: Kirtika Ruchandani <email@example.com>
Reviewed-by: Brian Norris <firstname.lastname@example.org>
1 file changed