Skip to content
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.

Sign up for GitHub

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

Jump to bottom

echo foo | kubectl exec -i <pod> cat hangs instead of cleanly exiting #13394

Closed
ncdc opened this issue Aug 31, 2015 · 4 comments · Fixed by #13322
Closed

echo foo | kubectl exec -i <pod> cat hangs instead of cleanly exiting #13394

ncdc opened this issue Aug 31, 2015 · 4 comments · Fixed by #13322
Assignees
@ncdc
Labels
priority/backlog Higher priority than priority/awaiting-more-evidence. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery.

Comments

@ncdc
Copy link
Member

ncdc commented Aug 31, 2015

If you run the following

echo foo | kubectl exec -i <pod> cat

it will print out foo and hang instead of returning control to the user in the shell. This is due to various changes to how go-dockerclient handles attach and exec.

I am working on a fix and will have a PR soon.
@lavalamp
Copy link
Member

lavalamp commented Sep 1, 2015

dup of #13395?

@lavalamp lavalamp closed this as completed Sep 1, 2015
@ncdc
Copy link
Member Author

ncdc commented Sep 1, 2015

Not a dupe - they have different root causes.

@liggitt
Copy link
Member

liggitt commented Sep 1, 2015

one is non-interactive terminating stdin, the other is interactive non-terminating stdin... need different tests and handling

@liggitt liggitt reopened this Sep 1, 2015
@lavalamp lavalamp added priority/backlog Higher priority than priority/awaiting-more-evidence. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. labels Sep 1, 2015
@lavalamp
Copy link
Member

lavalamp commented Sep 1, 2015

Gotcha, thanks for clarification.

@ncdc ncdc mentioned this issue Sep 2, 2015
Merged
@ghost ghost closed this as completed in #13322 Sep 4, 2015
@ncdc ncdc mentioned this issue Oct 19, 2015
Closed
@Marcofeli Marcofeli mentioned this issue Jan 15, 2019
Closed
@seans3 seans3 mentioned this issue Mar 21, 2023
Closed
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ncdc ncdc

Labels
priority/backlog Higher priority than priority/awaiting-more-evidence. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Various exec fixes ncdc/kubernetes
3 participants
@ncdc @lavalamp @liggitt