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.

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

Regression: kubectl logs --follow is broken #13811

Closed
stefwalter opened this issue Sep 10, 2015 · 10 comments
Closed

Regression: kubectl logs --follow is broken #13811

stefwalter opened this issue Sep 10, 2015 · 10 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@stefwalter
Copy link
Contributor

This command is broken on git master:

kubectl logs --follow <pod>

It just hangs forever rather than producing any output. It used to work in kubernetes 1.0

@stefwalter
Copy link
Contributor Author

Reproduce like so:

$ wget -O test.json 'https://bugzilla.redhat.com/attachment.cgi?id=1072268'
$ kubectl create -f test.json
$ kubectl get pods # get name and wait until running
$ kubectl logs --follow mock-ffctj
^C

The above just hangs. It should instead produce output:

$ kubectl logs --follow mock-tpo3j
HelloMessage.
HelloMessage.
HelloMessage.
HelloMessage.
HelloMessage.
HelloMessage.
HelloMessage.
HelloMessage.
...

docker logs --follow on the same machine works:

[root@falcon kubernetes]# docker logs --follow 248896a80890
HelloMessage.
HelloMessage.
HelloMessage.
HelloMessage.
HelloMessage.
...

@stefwalter
Copy link
Contributor Author

@eparis @ingvagabund @liggitt CC

@roberthbailey
Copy link
Contributor

@jlowdermilk CC as well

@roberthbailey
Copy link
Contributor

It sounds like we don't have any regression tests for this feature; we should add one as we fix it.

@stefwalter
Copy link
Contributor Author

The Cockpit integration tests caught this bug. So yes, testing upstream would be great.

@mikedanese
Copy link
Member

@kubernetes/kubectl

@mikedanese mikedanese added kind/bug Categorizes issue or PR as related to a bug. team/cluster labels Sep 10, 2015
@smarterclayton
Copy link
Contributor

Do we know what the problem is?

@liggitt
Copy link
Member

liggitt commented Sep 17, 2015

Did #13864 not resolve it?

@feiskyer
Copy link
Member

#13864 has fixed kubectl logs --follow broken problem.

@liggitt
Copy link
Member

liggitt commented Sep 21, 2015

@stefwalter closing as fixed in master

@liggitt liggitt closed this as completed Sep 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

6 participants