-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Re-enable oc run --attach
verifications once the docker logs api fix is vendored
#11240
Comments
I'm assuming this is the same failure manifesting slightly differently.
|
@mfojtik not sure this is the PM team either, but it's not devex...something is running an image w/ oc run and then expecting to see some output in the container logs:
|
@liggitt upstream is flaking a ton lately on |
oc run --attach
verifications once the docker logs api fix is vendored
The fix for this merged upstream to docker yesterday. If everyone wants, I can do a PR for origin's vendored copies of docker/engine-api and kube to add back support for retrieving logs at the beginning of attach. Let me know if I should do this. |
if we don't, is this a regression in 1.4? |
Not in 1.4, no. I was a regression in 1.3 I think. |
Interesting, the output I have from the command vs the output I got from docker logs is different:
vs
We're checking the former, but the latter contains the value we're looking for. |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
/remove-lifecycle stale |
The text was updated successfully, but these errors were encountered: