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

Make remote commands cancellable and remove panics #861

Merged
merged 1 commit into from
Mar 31, 2022

Conversation

kazk
Copy link
Member

@kazk kazk commented Mar 31, 2022

Make remote commands (attach, exec) cancellable like portforward (#854). Also add Error and remove panics.

@kazk kazk added the changelog-change changelog change category for prs label Mar 31, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #861 (b481014) into master (bc1f955) will decrease coverage by 0.04%.
The diff coverage is 68.75%.

@@            Coverage Diff             @@
##           master     #861      +/-   ##
==========================================
- Coverage   70.66%   70.62%   -0.05%     
==========================================
  Files          61       61              
  Lines        4255     4241      -14     
==========================================
- Hits         3007     2995      -12     
+ Misses       1248     1246       -2     
Impacted Files Coverage Δ
kube-client/src/api/remote_command.rs 67.74% <65.51%> (-2.63%) ⬇️
kube-client/src/lib.rs 93.75% <100.00%> (+0.04%) ⬆️

kube-client/src/api/remote_command.rs Show resolved Hide resolved
@clux clux added this to the 0.71.0 milestone Mar 31, 2022
Signed-off-by: kazk <kazk.dev@gmail.com>
@kazk kazk merged commit 8a33aac into kube-rs:master Mar 31, 2022
@kazk kazk deleted the refine-remote-command branch March 31, 2022 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-change changelog change category for prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants