-
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
test/end-to-end/core:183: oc status
output fragmented across oc run --attach
#9309
Comments
This seems to be more like a CLI issue. @Kargakis can you look this up? If it turns to be CLI and not platform specific, please re-assign to @fabianofranz |
I'm not sure if it's CLI -- I guess it's possible there are cases where |
Confirm I've seen this more than once, same log. |
Before or after rebase? Looks like before. |
@smarterclayton i don't think this is related to the other attach flake... but rebase could make this worse. any suggestion who would be the best candidate to look it up? |
I was seeing this before the rebase, using go-dockerclient. |
@ncdc output of attach still being seen as truncated at times (see https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_integration/2734/console). is there a known race between a process exiting and the output being sent? |
It is known, yes. I don't know why it's happening. |
@stevekuznetsov is this related to #9371? |
@Kargakis not that I can tell, no. In this case output is missing and it is missing from the command |
The test has been refactored ever since - can we close this? |
I'm not seeing any refactors here, what do you mean? |
ugh, nvmd |
@liggitt no, and I imagine it will be a while. Kube has to switch from using the now-deprecated docker/engine-api back to docker/docker, and then Origin will have to pick up the change as well. |
Do we consider logs to be critical enough function that we need to do extraordinary things to fix this before we release? |
If logs aren't reliable you can't script with them. They were previously
scriptable, so that's a problem.
…On Mon, Jan 23, 2017 at 9:49 AM, Jordan Liggitt ***@***.***> wrote:
Do we consider logs to be critical enough function that we need to do
extraordinary things to fix this before we release?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#9309 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABG_p97p4pNFLzgBu8sJ7o4nYIlvM5pvks5rVL3qgaJpZM4I0s_v>
.
|
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 |
Looks like a manifestation of our favorite /close |
Seeing this in AWS-based image.
We see some of the output from
oc status
... but not all of it?@deads2k @ncdc thoughts?
The text was updated successfully, but these errors were encountered: