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

Issue 870487 link

Starred by 1 user
Status: Fixed
Owner:
█ lhchavez@chromium.org
Last visit 15 days ago
Closed: Aug 8
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

run_oci should keep stdin open if running from a TTY

Project Member Reported by lhchavez@chromium.org, Aug 2 
Otherwise containers that expect to read from stdin will die.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 4 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/e23a93a25df1476dc0e9eb276196ec3fcb46dd1b

commit e23a93a25df1476dc0e9eb276196ec3fcb46dd1b
Author: Luis Hector Chavez <lhchavez@google.com>
Date: Sat Aug 04 09:00:09 2018

run_oci: Also inherit stdin

This fixes containers that read from stdin.

BUG= chromium:870487 
TEST=Android container still boots

Change-Id: I7eeda21c2c3f9f39c3e4e259762f7ae622f2ac6b
Reviewed-on: https://chromium-review.googlesource.com/1162262
Commit-Ready: Luis Hector Chavez <lhchavez@chromium.org>
Tested-by: Luis Hector Chavez <lhchavez@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/e23a93a25df1476dc0e9eb276196ec3fcb46dd1b/run_oci/run_oci.cc

Comment 2 by lhchavez@chromium.org, Aug 8

Status: Fixed (was: Untriaged)
►

Sign in to add a comment