Skip to content

Removes run-kube-local Target#552

Merged
arkodg merged 1 commit intoenvoyproxy:mainfrom
danehans:comment_out_kube_local
Oct 13, 2022
Merged

Removes run-kube-local Target#552
arkodg merged 1 commit intoenvoyproxy:mainfrom
danehans:comment_out_kube_local

Conversation

@danehans
Copy link
Copy Markdown
Contributor

xref: #256

Signed-off-by: danehans daneyonhansen@gmail.com

Signed-off-by: danehans <daneyonhansen@gmail.com>
@danehans danehans requested a review from a team as a code owner October 12, 2022 18:33
@danehans danehans added the area/ci CI and build related issues label Oct 12, 2022
@danehans danehans added this to the 0.2.0 milestone Oct 12, 2022
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #552 (e26668c) into main (2fb3ca8) will decrease coverage by 0.75%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #552      +/-   ##
==========================================
- Coverage   60.98%   60.22%   -0.76%     
==========================================
  Files          45       45              
  Lines        5564     5564              
==========================================
- Hits         3393     3351      -42     
- Misses       1968     2003      +35     
- Partials      203      210       +7     
Impacted Files Coverage Δ
internal/provider/kubernetes/httproute.go 59.13% <0.00%> (-6.96%) ⬇️
internal/provider/kubernetes/tlsroute.go 59.72% <0.00%> (-6.95%) ⬇️
internal/provider/kubernetes/gatewayclass.go 70.28% <0.00%> (-2.90%) ⬇️
internal/provider/kubernetes/gateway.go 49.23% <0.00%> (-1.54%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Copy Markdown
Member

@Xunzhuo Xunzhuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@arkodg arkodg merged commit 01a638c into envoyproxy:main Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci CI and build related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants