-
Notifications
You must be signed in to change notification settings - Fork 604
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
api: add support for containers that require TTY #1370
Conversation
Current coverage is 55.15% (diff: 100%)@@ master #1370 diff @@
==========================================
Files 80 80
Lines 12641 12638 -3
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
- Hits 6977 6970 -7
- Misses 4698 4701 +3
- Partials 966 967 +1
|
LGTM |
17 days old, no updates, needs rebase. Still want to push this through @stevvooe ? |
@dperny See moby/moby#25644 (comment) from @crosbymichael on the support of TTYs. |
I repoened this. I couldn't find a better alternative to having generic support for a system level console in normal containers are this point. Sorry for slowing this PR down. |
Signed-off-by: Stephen J Day <stephen.day@docker.com>
c9fc8ba
to
9293079
Compare
LGTM |
LGTM @stevvooe: Once this is merged, will you be opening a docker/docker PR to add support for it in the CLI and executor? |
I don't have any plans to. If it is not implemented by code freeze, I can take this on. |
Signed-off-by: Stephen J Day stephen.day@docker.com