Skip to content
This repository has been archived by the owner on Sep 4, 2021. It is now read-only.

Commit

Permalink
host: Don't assign consoles to TTY jobs
Browse files Browse the repository at this point in the history
Signed-off-by: Lewis Marshall <lewis@lmars.net>
  • Loading branch information
lmars committed Jul 9, 2016
1 parent f26b6e2 commit 218e56e
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions host/libcontainer_backend.go
Expand Up @@ -679,17 +679,6 @@ func (l *LibcontainerBackend) Run(job *host.Job, runConfig *RunConfig, rateLimit
Args: []string{"/.containerinit", job.ID},
User: "root",
}
if job.Config.TTY {
console, err := process.NewConsole(0, 0)
if err != nil {
return err
}
// copy the console to /dev/null because if we don't
// do anything with the console, it will be destroyed
// by the garbage collector and the job will get a
// tty hangup.
go io.Copy(ioutil.Discard, console)
}
if err := c.Run(process); err != nil {
c.Destroy()
return err
Expand Down

0 comments on commit 218e56e

Please sign in to comment.