Skip to content

Conversation

@crashiura
Copy link

Add tty for container

This fix issue 365

@crashiura crashiura changed the title Add support tty feature: add support tty Jun 13, 2022
@funvit
Copy link
Contributor

funvit commented Jun 13, 2022

I am think this can break docker.go (c *DockerContainer) StartLogProducer...

@mdelapenya
Copy link
Member

I am think this can break docker.go (c *DockerContainer) StartLogProducer...

@funvit could you please elaborate a bit more about it?

@mdelapenya
Copy link
Member

@crashiura @funvit do you think that, after #454, this can be closed?

@crashiura
Copy link
Author

@mdelapenya Hi, I think yes

@mdelapenya mdelapenya closed this Oct 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants