[ 
https://issues.apache.org/jira/browse/YARN-8777?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16637151#comment-16637151
 ] 

Eric Yang commented on YARN-8777:
---------------------------------

[~Zian Chen] Docker exec -it requires a tty to be attached for program to 
produce text based GUI output.  For example, running top, or handling terminal 
resize event.  Therefore, some environment variables, such as TERM, and control 
characters are managed externally to the exec program.  The current patch 
doesn't implement pseudo terminal setup, this can prevent docker exec -it to 
fail if the Java side doesn't implement pseudo terminal.  I need to revise this 
patch to implement pseudo terminal setup in C side, it would be much easier 
than doing pseudo terminal in the Java.

> Container Executor C binary change to execute interactive docker command
> ------------------------------------------------------------------------
>
>                 Key: YARN-8777
>                 URL: https://issues.apache.org/jira/browse/YARN-8777
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Zian Chen
>            Assignee: Eric Yang
>            Priority: Major
>              Labels: Docker
>         Attachments: YARN-8777.001.patch, YARN-8777.002.patch, 
> YARN-8777.003.patch
>
>
> Since Container Executor provides Container execution using the native 
> container-executor binary, we also need to make changes to accept new 
> “dockerExec” method to invoke the corresponding native function to execute 
> docker exec command to the running container.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to