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
cpFromPod: close writerStream synchronously #1396
cpFromPod: close writerStream synchronously #1396
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: fpm-peter The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Welcome @fpm-peter! |
/hold |
/unhold |
Looks like CI/CD tests are failing. |
I can see why the last person didn't finish the PR. With the fake websocket never actually connecting to anything, the promise cannot resolve. Just comparing the called command with the expected called command proves nothing besides that exec is working as intended which is already validated in the exec_test. The exec-websocket-return is ignored in cpFromPod since this also proves nothing but that a connection got established - which is the problem of the bug in the first place. |
@fpm-peter unfortunately we don't have e2e tests with a real kubernetes cluster. You could add them, similar to the ones in the Java client here: https://github.com/kubernetes-client/java/blob/master/.github/workflows/maven.yml#L67 But that is a pretty big change to force you to do to merge this PR. Perhaps the right answer is to remove this test if it isn't validating anything? |
The Kubernetes project currently lacks enough contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
This replicates #880 since #943 didn't solve the issue.